Hi Tom,
On Thu, 19 Oct 2023 at 07:57, Tom Rini trini@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 trini@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 sjg@chromium.org Reviewed-by: Tom Rini trini@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