[PATCH v3 32/32] sandbox: Add a test for disabling CONFIG_CMDLINE

[Tom Rini]

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 at 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 at chromium.org>
> > > Reviewed-by: Tom Rini <trini at 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?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231019/d325fe42/attachment.sig>