shiblon / latex-makefile

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

sed scripts confused by "redefining \!" #101

Closed shiblon closed 8 years ago

shiblon commented 8 years ago

Originally reported on Google Code with ID 88

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

Reported by f.poloni on 2010-07-05 08:51:04

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

Reported by shiblon on 2010-07-08 16:10:58

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

Reported by shiblon on 2010-07-08 17:34:14

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

Reported by shiblon on 2010-07-08 17:37:41

shiblon commented 8 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}

Reported by f.poloni on 2010-07-10 13:01:27

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

Reported by shiblon on 2010-07-20 17:20:31

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

Reported by shiblon on 2010-07-20 17:23:25

shiblon commented 8 years ago

Reported by shiblon on 2010-07-28 14:32:18