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

[Kory Maincent]

On Wed, 15 Oct 2025 08:15:35 -0600
Tom Rini <trini at konsulko.com> wrote:

> 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.

Oh ok. Didn't know about that. I am used to the Linux merge process. Sorry for
the noise!

Regards,
-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com