Hi Paolo,
>>>> Rainer Orth <[email protected]> writes:
>>>>
>>>>> diff --git a/libstdc++-v3/scripts/extract_symvers.in
>>>>> b/libstdc++-v3/scripts/extract_symvers.in
>>>>> --- a/libstdc++-v3/scripts/extract_symvers.in
>>>>> +++ b/libstdc++-v3/scripts/extract_symvers.in
>>>>> @@ -49,9 +49,12 @@ SunOS)
>>>>> if readelf --help | grep -- --wide > /dev/null; then
>>>>> readelf="$readelf --wide"
>>>>> fi
>>>>> + # Omit _DYNAMIC etc. for consistency with extract_symvers.pl, only
>>>>> + # present on Solaris.
>>>>> ${readelf} ${lib} |\
>>>>> sed -e 's/ \[<other>: [A-Fa-f0-9]*\] //' -e '/\.dynsym/,/^$/p;d' |\
>>>>> egrep -v ' (LOCAL|UND) ' |\
>>>>> + egrep -v
>>>>> '(_DYNAMIC|_GLOBAL_OFFSET_TABLE_|_PROCEDURE_LINKAGE_TABLE_|_edata|_end|_etext)$'
>>>>> |\
>>>> Shouldn't the regexp also be anchored (with a space) at the beginning?
>>> This works indeed. I was a bit worried if a space was guaranteed at
>>> that position, rather than general whitespace. Consider the patch
>>> amended accordingly.
>> Any word on this patch? Is has remained unreviewed for a week.
> Sorry about the delay. If you can convince me that the change is guaranteed
> to affect only Solaris, then you will not need a specific approval.
Unless those symbols were explicitly exported in config/abi/pre/gnu*.ver
(which they are not, otherwise the x86_64-unknown-linux-bootstrap would
have shown a failure), they couldn't make it into libstdc++.so. They
only occur on Solaris because gld explicitly adds them itself.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
