The patch could be put in as a blank option or now with a "README.Fedora8" file in the root. The readme would give instructions to change the Options.lb file to include the -Wl,--build-id=none option. At some point in the future we could change it around and make it so that the default is the option with a "README.build-id" about removing it. This would hopefully be a mute point, at some future time, as V2 would give way to V3 (which does not have this problem).
I still feel the Options.lb patch is the best and cleanest way to go at this point. The only sticking point is that the majority of people are not on Fedora8 yet and do not "need" this patch. I think my suggestion above solves this problem. The only issue I can see is if the change to add the option sneaks into the tree with a future patch file before we want to make the switch. This will just take some watchfullness on our part to make sure it does not happen.
I can put together a patch file with the changes and also the added README.Fedora8 file, if this is OK...
/*********************
Marc Karasek MTS Sun Microsystems mailto:marc.karasek@sun.com ph:770.360.6415 *********************/
ron minnich wrote:
On Dec 16, 2007 4:42 PM, Marc Karasek Marc.Karasek@sun.com wrote:
In order to enable/disable it you just have to go into the src/config/Options.lb and change the BUILDID_OPTION to be blank for older ld versions / non-x86 builds and leave it as "-Wl,--build-id=none" for the newer ld versions.
There is no way we can ask users to do this. We never intended for users to modify src/config/Options.lb as a normal part of making a build work.
The Options.lb file defines a set of config variables, and sets reasonable default values for them, and for the most part these values are mainboard and build related. For the most part, all these settings work across all distros.
I don't want to ask people to start tweaking this file.
We have to find a better way. Let's keep thinking.
Thanks!
ron