[PATCH] test: Do not build expo and cedit test if no SDL

Tom Rini trini at konsulko.com
Wed Oct 15 16:15:35 CEST 2025


On Wed, Oct 15, 2025 at 03:25:38PM +0200, Kory Maincent wrote:
> Hello Tom,
> 
> On Mon, 6 Oct 2025 15:44:36 -0600
> Tom Rini <trini at konsulko.com> wrote:
> 
> > On Mon, Oct 06, 2025 at 02:38:10PM -0600, Simon Glass wrote:
> > > Hi Kory,
> > > 
> > > On Mon, 6 Oct 2025 at 10:45, Kory Maincent <kory.maincent at bootlin.com>
> > > wrote:  
> > > >
> > > > expo and cedit tests depend on the host having the SDL library.
> > > > Build these tests only if VIDEO_SANDBOX_SDL config is enabled.
> > > >
> > > > Signed-off-by: Kory Maincent <kory.maincent at bootlin.com>
> > > > ---
> > > >  test/boot/Makefile | 2 ++
> > > >  1 file changed, 2 insertions(+)  
> > > 
> > > Reviewed-by: Simon Glass <sjg at chromium.org>
> > > 
> > > We should be careful that a future CI change doesn't cause these tests
> > > to stop running.  
> > 
> > Given how HOST_HAS_SDL is defined, CI will be fine.
> 
> What is the status on this fix? I see that it is still not merged. Is there a
> reason for this?

It's on my list to pick up very soon. The general window for applying
patches is about 2 weeks to allow time for feedback.

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


More information about the U-Boot mailing list