yochju / latex-makefile

Automatically exported from code.google.com/p/latex-makefile
Other
0 stars 0 forks source link

sed scripts confused by "redefining \!" #88

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I have been compiling a rather long tex file using the memoir class. My .log 
file happens to contain the line

LaTeX Info: Redefining \! on input line 10866.

Apparently, one of the sed scripts used by the makefile incorrectly parses it 
as an error; not sure, but I think the problem is in this regexp:

-e 's/.*\(\n\{0,\}! .*\)/$(C_ERROR)\1$(C_RESET)/p' \

on line 1671 in 2.2.0-rc1

Original issue reported on code.google.com by f.pol...@gmail.com on 5 Jul 2010 at 8:51

GoogleCodeExporter commented 9 years ago
Wow, that's a great bug.  I'll see what I can do to fix it :)

Original comment by shiblon on 8 Jul 2010 at 4:10

GoogleCodeExporter commented 9 years ago
On line 1641, try inserting the following:

-e '/LaTeX Info: Redefining \\!/d'

And let me know if it works.

Also, if you can attach a minimal example of a file that triggers this, that 
would be extremely helpful.

Original comment by shiblon on 8 Jul 2010 at 5:34

GoogleCodeExporter commented 9 years ago
r7393671ae568 has the change in it in case you don't want to patch it yourself.

Original comment by shiblon on 8 Jul 2010 at 5:37

GoogleCodeExporter commented 9 years ago
The modification doesn't seem to fix it.
A minimal example is:

\documentclass{memoir}
\begin{document}
This is a minimal example
\end{document}

Original comment by f.pol...@gmail.com on 10 Jul 2010 at 1:01

GoogleCodeExporter commented 9 years ago
r9ad120ea66db is another attempt to fix this more generally.  Thanks for the 
minimal example - I'll give that a go.

Original comment by shiblon on 20 Jul 2010 at 5:20

GoogleCodeExporter commented 9 years ago
I just tested the new revision on your minimal example, and it appears to work, 
thankfully :)

Original comment by shiblon on 20 Jul 2010 at 5:23

GoogleCodeExporter commented 9 years ago

Original comment by shiblon on 28 Jul 2010 at 2:32