Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes:

> 
> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
> 
> | >>>>> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes:
> | 
> | Lars> I'll leave that one for J-M. In trunk I fixed it with an '\r'
> | 
> | Well I am not sure what to do. Can't we just add the right property to
> | configure.ac?
> 
> No (at least not now). This was a bug before the conversion and is a
> bug after the conversion. Just now it is visible.

There are several flavours of sed out there on Windows. HSed, SSed, GNU sed.

   s/\r//
is, in general, not portable. It's a GNU-ism and is NOT covered by the POSIX
spec. I have absolutely no idea whether it will work with a non-GNU sed. It's
certainly flagged up as a non-portable construct by Laurent Vogel's sedcheck.sed
script (http://lvogel.free.fr/sed.htm)

So, make the change by all means, but be aware that "here be dragons".

Regards,
Angus




Reply via email to