From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Wed, 28 Nov 2007 19:03:52 GMT
Subject: Re: [PATCH] Replace instances of export VAR=VAL with VAR=VAL; export VAR
Message-ID: <Pine.LNX.4.64.0711281903330.27959@racer.site>
In-Reply-To: <7vr6ia5hoi.fsf@gitster.siamese.dyndns.org>

Hi,

On Wed, 28 Nov 2007, Junio C Hamano wrote:

> Johannes Sixt <j.sixt@viscovery.net> writes:
> 
> > Johannes Schindelin schrieb:
> > ...
> >>> Recently there was a report that \n in the substitution side of s/// is not
> >>> supported by all seds :-(
> >>
> >> Okay, how about replacing the line with
> >>
> >> +			s/.*/GIT_'$uid'_NAME='\''&'\''\
> >> +export GIT_'$uid'_NAME/p
> >>
> >> Hmm?  (It works here.)
> >
> > This looks good. The other case I'm refering to was also solved in this way.
> 
> That looks ugly to me.
> 
> Is there a particular reason to force linebreak when a semicolon would
> do?

D'oh.  Of course.  You want me to resend?

Ciao,
Dscho