search

shiblon / latex-makefile

A Makefile for LaTeX - drop it in, type make, and magic happens.
Other
185 stars 30 forks source link

sed errors in rc5 #109

Closed shiblon closed 8 years ago

shiblon commented 8 years ago

Originally reported on Google Code with ID 96

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).

Reported by shiblon on 2010-11-03 13:15:01

shiblon commented 8 years ago 
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.

Reported by shiblon on 2010-11-03 13:19:07

shiblon commented 8 years ago 
Should be fixed in r300efe421645.

Reported by shiblon on 2010-11-03 13:20:32

shiblon commented 8 years ago 
Updated further in r4a8c95830000 .

Reported by shiblon on 2010-11-03 18:50:57

shiblon commented 8 years ago 
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 /.

Reported by shiblon on 2010-11-03 19:59:34

shiblon commented 8 years ago

Reported by shiblon on 2010-12-31 19:46:22