[PATCH libdrm] meson: add configuration summary

Eric Engestrom eric.engestrom at imgtec.com
Wed Feb 28 15:11:20 UTC 2018


On Tuesday, 2018-02-27 12:30:48 -0800, Eric Anholt wrote:
> Dylan Baker <dylan at pnwbakers.com> writes:
> 
> > [ Unknown signature status ]
> > Quoting Eric Engestrom (2018-02-27 03:11:07)
> >> The message block printed is the same as the one in configure.ac
> >> 
> >> Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
> >> ---
> >>  meson.build | 17 +++++++++++++++++
> >>  1 file changed, 17 insertions(+)
> >> 
> >> diff --git a/meson.build b/meson.build
> >> index bd00cdc2cae9f0749180..ab6f881755935968b822 100644
> >> --- a/meson.build
> >> +++ b/meson.build
> >> @@ -373,3 +373,20 @@ if with_man_pages
> >>  endif
> >>  subdir('data')
> >>  subdir('tests')
> >> +
> >> +message('')
> >> +message('@0@ will be compiled with:'.format(meson.project_name()))
> >> +message('')
> >> +message('  libkms         @0@'.format(with_libkms))
> >> +message('  Intel API      @0@'.format(with_intel))
> >> +message('  vmwgfx API     @0@'.format(with_vmwgfx))
> >> +message('  Radeon API     @0@'.format(with_radeon))
> >> +message('  AMDGPU API     @0@'.format(with_amdgpu))
> >> +message('  Nouveau API    @0@'.format(with_nouveau))
> >> +message('  OMAP API       @0@'.format(with_omap))
> >> +message('  EXYNOS API     @0@'.format(with_exynos))
> >> +message('  Freedreno API  @0@ (kgsl: @1@)'.format(with_freedreno, with_freedreno_kgsl))
> >> +message('  Tegra API      @0@'.format(with_tegra))
> >> +message('  VC4 API        @0@'.format(with_vc4))
> >> +message('  Etnaviv API    @0@'.format(with_etnaviv))
> >> +message('')
> >> -- 
> >> Cheers,
> >>   Eric
> >> 
> >
> > This one is certainly simple enough that we can use a single message call and a
> > ''' string :)
> 
> But then you end up with 13 @n@ values and when someone wants to put
> something earlier in the list for some sorting reason, then they need to
> renumber the rest.  This is much nicer.

I have to agree here. Dylan, why did you want to avoid multiple
`message()`? They're not expensive afaict, so I'm not sure what the gain
would be?

I also had a try, and multiline messages only get a `Message:` prefix on
the first line, so combining them would result in unpredictable vertical
alignments. Unless there's a good reason to merge them, I'll keep them
separate.


More information about the dri-devel mailing list