Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > On Wed, 28 Nov 2007, Johannes Sixt wrote: > >> Johannes Schindelin schrieb: >> > - s/.*/export GIT_'$uid'_NAME='\''&'\''/p >> > + s/.*/GIT_'$uid'_NAME='\''&'\''\nexport >> > GIT_'$uid'_NAME/p >> >> 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.) Do we really want to replace code which is not as far as I know known to fail under any of the supported shells with something which quite possibly relies on particular sed implementations (and we have fewer portability guarantees or even experience about sed than about the shell, right?)? -- David Kastrup, Kriemhildstr. 15, 44793 Bochum - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
