[LinuxBIOS] [Fwd: Re: Patch file for Fedora 8 compile problems]
Marc Karasek
Marc.Karasek at Sun.COM
Mon Dec 17 16:45:54 CET 2007
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 at sun.com
ph:770.360.6415
*********************/
ron minnich wrote:
> On Dec 16, 2007 4:42 PM, Marc Karasek <Marc.Karasek at 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
>
More information about the coreboot
mailing list