Re: [igt-dev] [PATCH i-g-t v2 12/12] runner/meson.build: fix minimal build

[Mauro Carvalho Chehab <mauro.chehab@linux.intel.com>]

On Fri, 31 Mar 2023 12:39:19 +0200
Mauro Carvalho Chehab <mauro.chehab@linux.intel.com> wrote:

> On Fri, 31 Mar 2023 11:13:19 +0200
> Kamil Konieczny <kamil.konieczny@linux.intel.com> wrote:
> 
> > Hi Mauro,
> > 
> > On 2023-03-30 at 07:42:21 +0200, Mauro Carvalho Chehab wrote:  
> > > From: Mauro Carvalho Chehab <mchehab@kernel.org>
> > > 
> > > Minimal pipeline s now failing:    
> > ------------------ ^
> > s/s/is/
> >   
> > > 	https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/38824636
> > > 
> > > This doesn't seem to be related to the patches at the series from
> > > where the issue was reported, but addressing it is simple enough:
> > > just remove the error and change the logic to properly validate
> > > the possible options.
> > > 
> > > Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> > > ---
> > >  runner/meson.build | 6 +-----
> > >  1 file changed, 1 insertion(+), 5 deletions(-)
> > > 
> > > diff --git a/runner/meson.build b/runner/meson.build
> > > index dadfc75f0ca7..c62303ce35c5 100644
> > > --- a/runner/meson.build
> > > +++ b/runner/meson.build
> > > @@ -24,11 +24,7 @@ if liboping.found()
> > >  	runner_c_args += '-DHAVE_OPING=1'
> > >  endif
> > >  
> > > -if not build_tests and jsonc.found()
> > > -	error('Building test runner requires building tests')
> > > -endif    
> > 
> > imho:
> > if jsonc.found() and not build_tests 
> > 
> > is more clear what we need here.  
> 
> IMO, this is less clear ;-) I mean, refusing to run because jsonc is 
> installed and -Dtests=false sounds a very bad idea.
> 
> What we want instead is to have the runner built only if build_tests
> is selected (and the required jsonc library is installed). 

Btw, this error can be seen vanilla upstream if you have json-c installed
on some machine and run:

	$ meson -Dtests=disabled build-err
	The Meson build system
	Version: 0.53.2

	...
	
	Run-time dependency json-c found: YES 0.13.1
	...
	
	runner/meson.build:28:1: ERROR: Problem encountered: Building test runner requires building tests
	
	A full log can be found at /home/mchehab/freedesktop-igt/build-err/meson-logs/meson-log.txt

Regards,
Mauro