Re: [PATCH] Fix up libstdc++ build breakage with ldbl-extra.ver

2012-03-01 Thread Rainer Orth
Paolo Carlini writes: > Thanks Jakub for taking care of this. Indeed, and sorry for the breakage. Nothing of the sort turned up on x86_64-linux testing. Rainer -- - Rainer Orth, Center for Biotechnology, Biel

Re: [PATCH] Fix up libstdc++ build breakage with ldbl-extra.ver

2012-02-28 Thread Paolo Carlini
Thanks Jakub for taking care of this. Paolo.

Re: [PATCH] Fix up libstdc++ build breakage with ldbl-extra.ver

2012-02-28 Thread Paolo Bonzini
Il 28/02/2012 10:19, Jakub Jelinek ha scritto: >>> > > - $(EGREP) -v '#(#| |$$)' $@.tmp | \ >>> > > + $(EGREP) -v '^[ ]*#(#| |$$)' $@.tmp | \ >> > >> > I don't know this part very well, so I wonder why you have to remove >> > comments at all... hence I wonder if sed 's/##.*//;

Re: [PATCH] Fix up libstdc++ build breakage with ldbl-extra.ver

2012-02-28 Thread Jakub Jelinek
On Tue, Feb 28, 2012 at 09:57:38AM +0100, Paolo Bonzini wrote: > Il 28/02/2012 09:50, Jakub Jelinek ha scritto: > > - $(EGREP) -v '#(#| |$$)' $@.tmp | \ > > + $(EGREP) -v '^[ ]*#(#| |$$)' $@.tmp | \ > > I don't know this part very well, so I wonder why you have to remove > comments at

Re: [PATCH] Fix up libstdc++ build breakage with ldbl-extra.ver

2012-02-28 Thread Paolo Bonzini
Il 28/02/2012 09:50, Jakub Jelinek ha scritto: > - $(EGREP) -v '#(#| |$$)' $@.tmp | \ > + $(EGREP) -v '^[ ]*#(#| |$$)' $@.tmp | \ I don't know this part very well, so I wonder why you have to remove comments at all... hence I wonder if sed 's/##.*//;s/# .*//;s/#$//' (alternation i

[PATCH] Fix up libstdc++ build breakage with ldbl-extra.ver

2012-02-28 Thread Jakub Jelinek
Hi! On Mon, Feb 27, 2012 at 03:28:32PM -0800, Benjamin Kosnik wrote: > On Tue, 28 Feb 2012 00:12:33 +0100 > Jakub Jelinek wrote: > > > and $(EGREP) -v '#(#| |$$)' just throws away the whole > > };# Appended to version file. > > line. I wonder if > > sed -e 's/#[# $].*$//' > > wouldn't be better