Chris K

> Really nice! It's fairly large so looking over everything might take time but so far this looks very promising. The gist is to strip out the image building and...

I recommend the use of `cmds/boot/boot` for local/GRUB boots instead. We should probably move `localboot` to `exp` to mark it deprecated.

Yeah I’m giving it a rewrite based on his comments On Fri, Feb 9, 2024 at 12:48 ron minnich ***@***.***> wrote: > ***@***.**** approved this pull request. > > —...

Ready for another review @rminnich

I would like to eventually replace goshsmall with goshliner once I've confirmed that our internal use cases can work with goshliner. It doesn't seem like it adds that much size...

Please don't merge this PR yet @10000TB, I asked Steven to share the use case and the code using this with me so I can comment better. I haven't heard...

> > Please don't merge this PR yet @10000TB, I asked Steven to share the use case and the code using this with me so I can comment better. I...

Among other things, https://github.com/u-root/mkuimage is fixing this (tho this one is easy).

Hm I mean in the meantime you can call New(WithReaderAt(dtb), WithFileName(...)) I suppose. The package is usable on non-linux, only that file wouldn't be compiled there. It's called fdt_linux I...

I figured they wouldn't be too hard, but I'm fine with just having -h and -r work the same way.