On Thu, Aug 22, 2013 at 9:51 AM, Paul Eggleton <paul.eggle...@linux.intel.com> wrote: > On Thursday 22 August 2013 11:20:46 Richard Purdie wrote: >> On Thu, 2013-08-22 at 10:07 +0100, Paul Eggleton wrote: >> > On Wednesday 21 August 2013 22:58:55 Saul Wold wrote: >> > > Marko Lindqvist (3): >> > > texinfo: correct dont-depend-on-help2man.patch >> > > SDL2: add recipe >> > > boost: fix build when PARALLEL_MAKE is not set >> > > >> > > Otavio Salvador (1): >> > > distro_features_check.bbclass: Allow checking of required/conflicting >> > > >> > > features >> > >> > I should have asked this earlier - why does this need to be a separate >> > bbclass? The code doesn't do anything if the variables aren't set, so it >> > can't be because it could have an impact on existing setups that don't >> > set them. >> >> FWIW I do have a worry about the shear weight of the core these days as >> we're building up piles of anonymous python for example. In many ways I >> therefore like the idea of having more classes which get included when >> they're needed. > > For something like this though it just increases complexity for users - unlike > other optional class-supported functionality it's not even clear from the > variable name that you'll need to inherit the class before they can be used.
Both ways works for me. So do the choice and I update the patch. -- Otavio Salvador O.S. Systems http://www.ossystems.com.br http://code.ossystems.com.br Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750 _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
