[Buildroot] [RFC v3 2/4] meson: change from global to per package cross-compilation.conf

[Peter Seiderer <ps.report@gmx.net>]

Hello Thomas,

On Sun, 19 Aug 2018 19:15:11 +0200, Peter Seiderer <ps.report@gmx.net> wrote:

> Hello Thomas,
> 
> 
> On Sun, 19 Aug 2018 15:22:06 +0200, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:
> 
> > Hello,
> > 
> > On Sun, 19 Aug 2018 01:15:12 +0200, Peter Seiderer wrote:
> >   
> > > The CFLAGS are ignored, e.g. with 'LIBDRM_CONF_ENV += CFLAGS=-DAO_NO_SPARC_V9' in package/libdrm/libdrm.mk:
> > > 
> > > 	$ make libdrm V=1    
> > 
> > It seems weird, no? It is documented to be working, and even at runtime
> > it tells you "I am going to use those additional CFLAGS". Isn't this a
> > bug or a misuse of meson from us ?
> >   
> 
> Bug or feature of the cross-compilation.conf usage? Will take a look at
> the meson sources....

Seems to be intentional, see mesonbuild/backend/backends.py#L542 ([1]):

        if not target.is_cross:
            # Compile args added from the env: CFLAGS/CXXFLAGS, etc. We want these
            # to override all the defaults, but not the per-target compile args.
commands += self.environment.coredata.get_external_args(compiler.get_language())

With the 'if not target.is_cross' added with this commit [2] fixing [3] which
suggests adding CROSS_*FLAGS...

Regards,
Peter

[1] https://github.com/mesonbuild/meson/blob/master/mesonbuild/backend/backends.py#L542
[2] https://github.com/mesonbuild/meson/commit/b595cda4ed0ca699da3052a6bd30ba7d1dae1124
[3] https://github.com/mesonbuild/meson/issues/1772

> 
> Regards,
> Peter
> 
> > I would really prefer to understand what is going on here instead of
> > having to generate per-package .conf files.
> > 
> > Thanks!
> > 
> > Thomas  
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot