sed errors in rc5

[GoogleCodeExporter]

Report from a user:

I am using version 2.2.0-rc5, and I kept getting errors from sed,
something like:

   sed: -e expression #12, char 6: unknown option to `s'

Turns out it was sed expressions trying to do s/// on pieces of string
that themselves contain slashes (file names with paths).

Original issue reported on code.google.com by shiblon on 3 Nov 2010 at 1:15

[GoogleCodeExporter]

The attached patch tried to fix the problem by introducing a new escape 
function, but we can just use sed's own ability to use characters other than / 
as delimiters for substitution commands, like !.  So, patch attached.

Original comment by shiblon on 3 Nov 2010 at 1:19

Attachments:

• rc6.patch

[GoogleCodeExporter]

Should be fixed in r300efe421645.

Original comment by shiblon on 3 Nov 2010 at 1:20

[GoogleCodeExporter]

Updated further in r4a8c95830000 .

Original comment by shiblon on 3 Nov 2010 at 6:50

• Changed state: Started

[GoogleCodeExporter]

Another update: rb577f05e031b

For some reason, it was choking on the double-backslash escaping for /.  I 
cannot, for the life of me, figure out why I used \\. to escape . before, but 
it appears to work.  The same trick (i.e., \\/) does not work for /.

Original comment by shiblon on 3 Nov 2010 at 7:59

[GoogleCodeExporter]

Original comment by shiblon on 31 Dec 2010 at 7:46

• Changed state: Fixed