Need to add "thm" to rm_ext list

yochju / latex-makefile

Automatically exported from code.google.com/p/latex-makefile

Other

0 stars 0 forks source link

Need to add "thm" to rm_ext list #59

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

rm_ext needs to include "thm", since this is one of the files latex
produces that needs to be cleaned up on a "make clean".

Original issue reported on code.google.com by timanda...@gmail.com on 4 Dec 2009 at 6:22

GoogleCodeExporter commented 9 years ago

I presume that this comes from the theorem environment?

Original comment by shiblon on 4 Dec 2009 at 1:28

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

Can you try out r8808a6b24640 and let me know if that fixes it for you?

I've long been annoyed that I have this explicit blacklist of file extensions 
for cleaning, 
so I think I have a somewhat more general solution in place for it now that 
reads the 
OUTPUT lines from the .fls file.  Give it a whirl!

Original comment by shiblon on 4 Dec 2009 at 1:39

Changed state: Started

GoogleCodeExporter commented 9 years ago

Whoa, that was a quick response!

Yes, the .thm files come from the theorem environment. I use ntheorem, that 
mentions
the .thm files:
http://www.tex.ac.uk/tex-archive/macros/latex/contrib/ntheorem/ntheorem.pdf

r8808a6b24640 works great. Thanks for fixing this and doing it the proper way 
too
(and not just the easy way).

And thanks for creating and maintaining this really well written Makefile, I 
find it
highly useful, as do the people I recommended it to!

Original comment by timanda...@gmail.com on 4 Dec 2009 at 4:22

GoogleCodeExporter commented 9 years ago

Good stuff!  I'm probably going to add it to rm_ext too, just for kicks and 
grins, but I'm 
glad that the more general solution works in this case.

Thanks for the kind words, too.  Don't hesitate to file issues as you see them.

I'll post an uploaded version after I get issue 36 resolved (the change that 
fixed the thm 
problem was in a fix for that, sort of - I just made it cleaner for you).  I'll 
update the bug 
when that's done.  Meanwhile, the trunk should work fine.

Original comment by shiblon on 4 Dec 2009 at 4:28

GoogleCodeExporter commented 9 years ago

Cleaning out my issues box.  This fix went into the official version some time 
ago, so 
marking as verified.

Original comment by shiblon on 9 Mar 2010 at 8:51

Changed state: Verified