]> git.lyx.org Git - lyx.git/blobdiff - src/LaTeX.C
Fix deleting of paragraphs after undo (fix #236).
[lyx.git] / src / LaTeX.C
index 387e33f061b90d7175bc1d3f89c101a9162ecbe9..054a7796a1f0303226765709a58e4dd6dc9a1c9a 100644 (file)
@@ -3,9 +3,9 @@
  * 
  *           LyX, The Document Processor        
  *          Copyright 1995 Matthias Ettrich
- *           Copyright 1995-2000 The LyX Team.
+ *           Copyright 1995-2001 The LyX Team.
  *
- *           This file is Copyright 1996-2000
+ *           This file is Copyright 1996-2001
  *           Lars Gullik Bjønnes
  *
  * ====================================================== 
 #endif
 #include <fstream>
 
-#include "support/filetools.h"
 #include "LaTeX.h"
-#include "support/FileInfo.h"
+#include "bufferlist.h"
+#include "gettext.h"
+#include "lyxfunc.h"
 #include "debug.h"
-#include "support/lyxlib.h"
-#include "support/syscall.h"
-#include "support/syscontr.h"
-#include "support/path.h"
+#include "support/filetools.h"
+#include "support/FileInfo.h"
 #include "support/LRegex.h"
 #include "support/LSubstring.h"
-#include "bufferlist.h"
-#include "minibuffer.h"
-#include "gettext.h"
-#include "lyx_gui_misc.h"
+#include "support/lstrings.h"
+#include "support/lyxlib.h"
+#include "support/systemcall.h"
+#include "support/os.h"
+#include "support/path.h"
+#include <cstdio>  // sscanf
 
 using std::ifstream;
 using std::getline;
@@ -74,8 +75,12 @@ LaTeX::LaTeX(string const & latex, string const & f, string const & p)
 {
        num_errors = 0;
        depfile = file + ".dep";
-       if (prefixIs(cmd, "pdf")) // Do we use pdflatex ?
+       if (prefixIs(cmd, "pdf")) // Do we use pdflatex ?
                depfile += "-pdf";
+               output_file = ChangeExtension(file,".pdf");
+       } else {
+               output_file = ChangeExtension(file,".dvi");
+       }
 }
 
 
@@ -106,7 +111,7 @@ void LaTeX::deleteFilesOnError() const
 }
 
 
-int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
+int LaTeX::run(TeXErrors & terr, LyXFunc * lfun)
        // We know that this function will only be run if the lyx buffer
        // has been changed. We also know that a newly written .tex file
        // is always different from the previous one because of the date
@@ -152,12 +157,19 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
                head.read(depfile);
                // Update the checksums
                head.update();
-               if (!head.sumchange()) {
+               // Can't just check if anything has changed because it might have aborted
+               // on error last time... in which cas we need to re-run latex
+               // and collect the error messages (even if they are the same).
+               if (!FileInfo(output_file).exist()) {
+                       lyxerr[Debug::DEPEND]
+                               << "re-running LaTeX because output file doesn't exist." << endl;
+               } else if (!head.sumchange()) {
                        lyxerr[Debug::DEPEND] << "return no_change" << endl;
                        return NO_CHANGE;
+               } else {
+                       lyxerr[Debug::DEPEND]
+                               << "Dependency file has changed" << endl;
                }
-               lyxerr[Debug::DEPEND]
-                       << "Dependency file has changed" << endl;
 
                if (head.extchanged(".bib") || head.extchanged(".bst"))
                        run_bibtex = true;
@@ -173,8 +185,13 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
                bibtex_info_old = scanAuxFiles(aux_file);
 
        ++count;
-       lyxerr[Debug::LATEX] << "Run #" << count << endl; 
-       WriteStatus(minib, string(_("LaTeX run number ")) + tostr(count));
+       lyxerr[Debug::LATEX] << "Run #" << count << endl;
+       if (lfun) {
+               ostringstream str;
+               str << _("LaTeX run number") << ' ' << count;
+               lfun->dispatch(LFUN_MESSAGE, str.str().c_str());
+       }
+       
        this->operator()();
        scanres = scanLogFile(terr);
        if (scanres & ERROR_RERUN) {
@@ -207,7 +224,10 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
        if (head.haschanged(OnlyFilename(ChangeExtension(file, ".idx")))) {
                // no checks for now
                lyxerr[Debug::LATEX] << "Running MakeIndex." << endl;
-               WriteStatus(minib, _("Running MakeIndex."));
+               if (lfun) {
+                       lfun->dispatch(LFUN_MESSAGE, _("Running MakeIndex."));
+               }
+               
                rerun = runMakeIndex(OnlyFilename(ChangeExtension(file, ".idx")));
        }
 
@@ -219,14 +239,18 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
                // tags is found -> run bibtex and set rerun = true;
                // no checks for now
                lyxerr[Debug::LATEX] << "Running BibTeX." << endl;
-               WriteStatus(minib, _("Running BibTeX."));
+               if (lfun) {
+                       lfun->dispatch(LFUN_MESSAGE, _("Running BibTeX."));
+               }
+               
                updateBibtexDependencies(head, bibtex_info);
                rerun |= runBibTeX(bibtex_info);
-       } else if (!had_depfile)
+       } else if (!had_depfile) {
                /// If we run pdflatex on the file after running latex on it,
                /// then we do not need to run bibtex, but we do need to
                /// insert the .bib and .bst files into the .dep-pdf file.
                updateBibtexDependencies(head, bibtex_info);
+       }
        
        // 1
        // we know on this point that latex has been run once (or we just
@@ -247,8 +271,12 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
                        << "Dep. file has changed or rerun requested" << endl;
                lyxerr[Debug::LATEX]
                        << "Run #" << count << endl;
-               WriteStatus(minib,
-                           string(_("LaTeX run number ")) + tostr(count));
+               if (lfun) {
+                       ostringstream str;
+                       str << _("LaTeX run number") << ' ' << count;
+                       lfun->dispatch(LFUN_MESSAGE, str.str().c_str());
+               }
+               
                this->operator()();
                scanres = scanLogFile(terr);
                if (scanres & ERRORS) {
@@ -275,7 +303,10 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
        if (head.haschanged(OnlyFilename(ChangeExtension(file, ".idx")))) {
                // no checks for now
                lyxerr[Debug::LATEX] << "Running MakeIndex." << endl;
-               WriteStatus(minib, _("Running MakeIndex."));
+               if (lfun) {
+                       lfun->dispatch(LFUN_MESSAGE, _("Running MakeIndex."));
+               }
+               
                rerun = runMakeIndex(OnlyFilename(ChangeExtension(file, ".idx")));
        }
        
@@ -296,7 +327,12 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
                rerun = false;
                ++count;
                lyxerr[Debug::LATEX] << "Run #" << count << endl;
-               WriteStatus(minib, string(_("LaTeX run number ")) + tostr(count));
+               if (lfun) {
+                       ostringstream str;
+                       str << _("LaTeX run number") << ' ' << count;
+                       lfun->dispatch(LFUN_MESSAGE, str.str().c_str());
+               }
+               
                this->operator()();
                scanres = scanLogFile(terr);
                if (scanres & ERRORS) {
@@ -322,8 +358,8 @@ int LaTeX::operator()()
 #else // cmd.exe (OS/2) causes SYS0003 error at "/dev/null"
        string tmp = cmd + ' ' + file + " > nul";
 #endif
-        Systemcalls one;
-       return one.startscript(Systemcalls::System, tmp);
+        Systemcall one;
+       return one.startscript(Systemcall::Wait, tmp);
 }
 
 
@@ -339,8 +375,8 @@ bool LaTeX::runMakeIndex(string const & f)
        // to come for a later time. (0.13 perhaps?)
        string tmp = "makeindex -c -q ";
        tmp += f;
-       Systemcalls one;
-       one.startscript(Systemcalls::System, tmp);
+       Systemcall one;
+       one.startscript(Systemcall::Wait, tmp);
        return true;
 }
 
@@ -385,6 +421,7 @@ void LaTeX::scanAuxFile(string const & file, Aux_Info & aux_info)
        LRegex reg4("\\\\@input\\{([^}]+)\\}");
 
        while (getline(ifs, token)) {
+               token = strip(token, '\r');
                if (reg1.exact_match(token)) {
                        LRegex::SubMatches const & sub = reg1.exec(token);
                        string data = LSubstring(token, sub[1].first,
@@ -437,7 +474,7 @@ void LaTeX::updateBibtexDependencies(DepTable & dep,
        // remove all ".bib" and ".bst" files.
        dep.remove_files_with_extension(".bib");
        dep.remove_files_with_extension(".bst");
-       string aux = OnlyFilename(ChangeExtension(file, ".aux"));
+       //string aux = OnlyFilename(ChangeExtension(file, ".aux"));
 
        for (vector<Aux_Info>::const_iterator it = bibtex_info.begin();
             it != bibtex_info.end(); ++it) {
@@ -469,8 +506,8 @@ bool LaTeX::runBibTeX(vector<Aux_Info> const & bibtex_info)
 
                string tmp = "bibtex ";
                tmp += OnlyFilename(ChangeExtension(it->aux_file, string()));
-               Systemcalls one;
-               one.startscript(Systemcalls::System, tmp);
+               Systemcall one;
+               one.startscript(Systemcall::Wait, tmp);
        }
        // Return whether bibtex was run
        return result;
@@ -519,8 +556,20 @@ int LaTeX::scanLogFile(TeXErrors & terr)
                                }
                        } else if (contains(token, "run BibTeX")) {
                                retval |= UNDEF_CIT;
-                       } else if (contains(token, "Rerun LaTeX.")) {
-                               // at least longtable.sty might use this.
+                       } else if (contains(token, "Rerun LaTeX") ||
+                                  contains(token, "Rerun to get")) {
+                               // at least longtable.sty and bibtopic.sty
+                               // might use this.
+                               lyxerr[Debug::LATEX]
+                                       << "We should rerun." << endl;
+                               retval |= RERUN;
+                       }
+               } else if (prefixIs(token, "(")) {
+                       if (contains(token, "Rerun LaTeX") ||
+                           contains(token, "Rerun to get")) {
+                               // Used by natbib
+                               lyxerr[Debug::LATEX]
+                                       << "We should rerun." << endl;
                                retval |= RERUN;
                        }
                } else if (prefixIs(token, "! ")) {
@@ -607,10 +656,15 @@ void LaTeX::deplog(DepTable & head)
 
        string const logfile = OnlyFilename(ChangeExtension(file, ".log"));
 
-       LRegex reg1(")* *\\(([^ \\)]+).*");
+       LRegex reg1("\\)* *\\(([^ )]+).*");
        LRegex reg2("File: ([^ ]+).*");
        LRegex reg3("No file ([^ ]+)\\..*");
        LRegex reg4("\\\\openout[0-9]+.*=.*`([^ ]+)'\\..*");
+       // If an index should be created, MikTex does not write a line like
+       //    \openout# = 'sample,idx'.
+       // but intstead only a line like this into the log:
+       //   Writing index file sample.idx
+       LRegex reg5("Writing index file ([^ ]+).*");
        LRegex unwanted("^.*\\.(aux|log|dvi|bbl|ind|glo)$");
        
        ifstream ifs(logfile.c_str());
@@ -623,6 +677,7 @@ void LaTeX::deplog(DepTable & head)
                string foundfile;
                string token;
                getline(ifs, token);
+               token = strip(token, '\r');
                if (token.empty()) continue;
                
                if (reg1.exact_match(token)) {
@@ -641,10 +696,17 @@ void LaTeX::deplog(DepTable & head)
                        LRegex::SubMatches const & sub = reg4.exec(token);
                        foundfile = LSubstring(token, sub[1].first,
                                               sub[1].second);
+               } else if (reg5.exact_match(token)) {
+                       LRegex::SubMatches const & sub = reg5.exec(token);
+                       foundfile = LSubstring(token, sub[1].first,
+                                              sub[1].second);
                } else {
                        continue;
                }
 
+               // convert from native os path to unix path
+               foundfile = os::internal_path(foundfile);
+               
                lyxerr[Debug::DEPEND] << "Found file: " 
                                      << foundfile << endl;
                
@@ -697,4 +759,7 @@ void LaTeX::deplog(DepTable & head)
                                << "Not a file or we are unable to find it."
                                << endl;
        }
+
+       // Make sure that the main .tex file is in the dependancy file.
+       head.insert(OnlyFilename(file), true);
 }