Re: [PATCH 0/3] siteinfo.bbclass re-merge

[Tom Rini <tom_rini@mentor.com>]

On 07/13/2011 12:42 PM, Mark Hatle wrote:
> On 7/13/11 2:05 PM, Tom Rini wrote:
>> Hey all,
>>
>> This is the second of two pull requests.  This doesn't depend on the first
>> request, it's just splitting changes up.  There's three changes here.  First,
>> site/ix86-common was setting the wrong value for unsigned int.  Second,
>> change perl to use SITEINFO variables rather than functions.  Third, bring
>> in the various changes in siteinfo.bbclass that have been in oe.dev.  This
>> last part will, I hope, also make dealing with multilib-specific siteinfo
>> stuff easier (also alternative ABIs like x32).
> 
> Note, this is NOT an objection to the changes.  However, one thing to keep in mind.
> 
> (In fact I like the changes being suggested)
> 
> We have a dynamic site-configuation mechanism already in place.  So it's
> important that we don't duplicate things within the site/* files.
> 
> Currently the recipes: glibc, zlib, eglibc, and uclibc contain dynamic
> site_config files.  (See ./recipes-core/glibc/site_config/* for an example)
> 
> The output of these files get placed into a target arch specific site config
> directory: ${STAGING_DATADIR}/${TARGET_SYS}_config_site.d

I want to confirm that these files are still generated, found and
working.  But maybe part 4 of my series will be talking about some of
this stuff a bit more and if that's really the best method for the goal.

-- 
Tom Rini
Mentor Graphics Corporation