Hi Tom, On Thu, 19 Oct 2023 at 07:57, Tom Rini <tr...@konsulko.com> wrote: > > On Thu, Oct 19, 2023 at 08:00:00AM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Wed, 18 Oct 2023 at 20:32, Tom Rini <tr...@konsulko.com> wrote: > > > > > > On Mon, Oct 16, 2023 at 04:28:23PM -0600, Simon Glass wrote: > > > > > > > Now that everything is working, add a test to make sure that this > > > > builds correctly. > > > > > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > > > Reviewed-by: Tom Rini <tr...@konsulko.com> > > > > --- > > > > > > > > Changes in v3: > > > > - Rebase on Tom's LONGHELP series > > > > - Correct 'of' typo > > > > > > > > test/py/tests/test_sandbox_opts.py | 20 ++++++++++++++++++++ > > > > 1 file changed, 20 insertions(+) > > > > create mode 100644 test/py/tests/test_sandbox_opts.py > > > > > > This is not doing the equivalent of: > > > make sandbox_config > > > sed -i -e 's/CONFIG_CMDLINE=y/CONFIG_CMDLINE=n/' .config > > > make oldconfig > > > make all > > > > Buildman should do that automatically...is something failing? > > Yes. https://source.denx.de/u-boot/u-boot/-/jobs/716843 fails, but > shouldn't, and it's only the no LTO job, so maybe there's some conflict > there?
For me this was the EFI patch to make it depend on net. Perhaps you hit a different one? I am seeing build errors with ~CMDLINE ~LTO with your series. Regards, Simon
