Re: [PATCH 2/2 v2] boost: add support for additional boost libs

["André Draszik" <git@andred.net>]

On Di, 2016-09-27 at 02:26 +0000, Huang, Jie (Jackie) wrote:
> 
> > 
> > -----Original Message-----
> > From: André Draszik [mailto:git@andred.net]
> > Sent: Monday, September 26, 2016 4:54 PM
> > To: Huang, Jie (Jackie); openembedded-core@lists.openembedded.org
> > Subject: Re: [OE-core] [PATCH 2/2 v2] boost: add support for additional
> > boost libs
> > 
> > On Mo, 2016-09-26 at 15:56 +0800, jackie.huang@windriver.com wrote:
> > > 
> > > From: Jackie Huang <jackie.huang@windriver.com>
> > > 
> > > * Added libs:
> > >   - container
> > >   - context
> > >   - coroutine
> > >   - exception
> > >   - graph_parallel
> > >   - locale
> > >   - math
> > >   - mpi
> > >   - wave
> > > 
> > > * Add PACKAGECONFIG to add proper dependencies for:
> > >   graph_parallel, locale, and mpi.
> > > 
> > > * boost-mpi depends on mpich which is in meta-oe,
> > >   and boost-graph_parallel depends on boost-mpi,
> > >   so they are disabled by default, but can be enabled
> > >   in a distro that needs them.
> > > 
> > > * context and coroutine are added only for x86 and powerpc.
> > > 
> > > Signed-off-by: Jackie Huang <jackie.huang@windriver.com>
> > > ---
> > >  meta/recipes-support/boost/boost.inc | 33
> > > ++++++++++++++++++++++++++++++-
> > > --
> > >  1 file changed, 30 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/meta/recipes-support/boost/boost.inc b/meta/recipes-
> > > support/boost/boost.inc
> > > index 5696b6a..7637a4e 100644
> > > --- a/meta/recipes-support/boost/boost.inc
> > > +++ b/meta/recipes-support/boost/boost.inc
> > > @@ -8,11 +8,14 @@ ARM_INSTRUCTION_SET_armv5 = "arm"
> > >  BOOST_LIBS = "\
> > >  	atomic \
> > >  	chrono \
> > > +	container \
> > >  	date_time \
> > > +	exception \
> > >  	filesystem \
> > >  	graph \
> > >  	iostreams \
> > >  	log \
> > > +	math \
> > >  	program_options \
> > >  	random \
> > >  	regex \
> > > @@ -22,12 +25,28 @@ BOOST_LIBS = "\
> > >  	timer \
> > >  	test \
> > >  	thread \
> > > +	wave \
> > >  	"
> > > 
> > > -# optional boost-python library
> > > -PACKAGECONFIG ??= ""
> > > +# only supported by x86 and powerpc
> > > +BOOST_LIBS_append_x86 = " context coroutine"
> > > +BOOST_LIBS_append_x86-64 = " context coroutine"
> > > +BOOST_LIBS_append_powerpc = " context coroutine"
> > > +
> > > +# optional libraries
> > > +PACKAGECONFIG ??= "locale"
> > > +PACKAGECONFIG[locale] = ",,icu"
> > > +PACKAGECONFIG[graph_parallel] = ",,,boost-mpi mpich"
> > > +PACKAGECONFIG[mpi] = ",,mpich"
> > >  PACKAGECONFIG[python] = ",,python python3"
> > > -BOOST_LIBS += "${@bb.utils.contains('PACKAGECONFIG', 'python',
> > > 'python
> > > python3', '', d)}"
> > > +
> > > +BOOST_LIBS += "\
> > > +    ${@bb.utils.contains('PACKAGECONFIG', 'locale', 'locale', '', d)}
> > > \
> > > +    ${@bb.utils.contains('PACKAGECONFIG', 'graph_parallel',
> > > 'graph_parallel mpi', \
> > > +                         bb.utils.contains('PACKAGECONFIG', 'mpi',
> > > 'mpi',
> > > '', d), d)} \
> > > +    ${@bb.utils.contains('PACKAGECONFIG', 'python', 'python python3',
> > > '',
> > > d)} \
> > > +"
> > 
> > Rather than having two ways to define what packages to build,
> > PACKAGECONFIG
> > and BOOST_LIBS, where each of them only supports a different subset of
> 
> I'm not a fan of this way, I just keep using the way it was.
> 
> > 
> > boost's libraries, can we just have one, PACKAGECONFIG, and have it
> > support
> > all of the potential libraries?
> 
> Yes, I think it's possible, but it will most likely end up with many empty
> PACKAGECONFIG definitions like:
> PACKAGECONFIG[atomic] = ",,,"
> PACKAGECONFIG[chrono] = ",,,"
> PACKAGECONFIG[container] = ",,,"
> PACKAGECONFIG[date_time] = ",,,"
> PACKAGECONFIG[exception] = ",,,"
> 
> If you think it's a better way and no other objection, I think I will do
> that in a
> separate commit as follow-up.

I don't think you'd need empty PACKAGECONFIG[flag] definitions, see
base.bbclass.

Cheers,
Andre'