Hi,
On 03/06/2013 12:08 PM, Rainer Orth wrote:
Rainer Orth <r...@cebitec.uni-bielefeld.de> writes:
Andreas Schwab <sch...@suse.de> writes:
Rainer Orth <r...@cebitec.uni-bielefeld.de> 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.
Paolo.
