]> git.lyx.org Git - lyx.git/blobdiff - src/LaTeX.C
float2string #4 (Spacing)
[lyx.git] / src / LaTeX.C
index 70510fff61cf246430f3496f4563f688858f1a4c..8609c3ed53dc20f92c5167fc0305b71f8574d928 100644 (file)
@@ -1,78 +1,93 @@
-/* This file is part of
- * ====================================================== 
- * 
- *           LyX, The Document Processor        
- *          Copyright 1995 Matthias Ettrich
- *           Copyright 1995-2000 The LyX Team.
+/**
+ * \file LaTeX.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           This file is Copyright 1996-2000
- *           Lars Gullik Bjønnes
+ * \author Alfredo Braunstein
+ * \author Lars Gullik Bjønnes
+ * \author Jean-Marc Lasgouttes
+ * \author Angus Leeming
+ * \author Dekel Tsur
  *
- * ====================================================== 
+ * Full author contact details are available in file CREDITS.
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-#include <fstream>
-
-#include "support/filetools.h"
 #include "LaTeX.h"
-#include "support/FileInfo.h"
-#include "debug.h"
-#include "support/lyxlib.h"
-#include "support/syscall.h"
-#include "support/syscontr.h"
-#include "support/path.h"
-#include "support/LRegex.h"
-#include "support/LSubstring.h"
 #include "bufferlist.h"
-#include "minibuffer.h"
 #include "gettext.h"
+#include "lyxrc.h"
+#include "debug.h"
+#include "DepTable.h"
+#include "support/filetools.h"
+#include "support/FileInfo.h"
+#include "support/tostr.h"
+#include "support/lstrings.h"
+#include "support/lyxlib.h"
+#include "support/systemcall.h"
+#include "support/os.h"
+
+#include <boost/regex.hpp>
+
+#include <fstream>
+
+using lyx::support::AbsolutePath;
+using lyx::support::bformat;
+using lyx::support::ChangeExtension;
+using lyx::support::contains;
+using lyx::support::FileInfo;
+using lyx::support::findtexfile;
+using lyx::support::getcwd;
+using lyx::support::OnlyFilename;
+using lyx::support::prefixIs;
+using lyx::support::QuoteName;
+using lyx::support::rtrim;
+using lyx::support::split;
+using lyx::support::subst;
+using lyx::support::suffixIs;
+using lyx::support::Systemcall;
+using lyx::support::unlink;
+using lyx::support::trim;
+
+namespace os = lyx::support::os;
+
+using boost::regex;
+using boost::smatch;
+
+
+#ifndef CXX_GLOBAL_CSTD
+using std::sscanf;
+#endif
 
-using std::ifstream;
-using std::getline;
 using std::endl;
+using std::getline;
+using std::string;
+using std::ifstream;
+using std::set;
+using std::vector;
 
 // TODO: in no particular order
 // - get rid of the extern BufferList and the call to
 //   BufferList::updateIncludedTeXfiles, this should either
 //   be done before calling LaTeX::funcs or in a completely
 //   different way.
-// - the bibtex command options should be supported.
 // - the makeindex style files should be taken care of with
 //   the dependency mechanism.
 // - makeindex commandline options should be supported
 // - somewhere support viewing of bibtex and makeindex log files.
 // - we should perhaps also scan the bibtex log file
-// - we should perhaps also scan the bibtex log file
 
 extern BufferList bufferlist;
 
-struct texfile_struct {
-       LaTeX::TEX_FILES file;
-       char const * extension;
-};
-
-static
-const texfile_struct all_files[] = {
-       { LaTeX::AUX, ".aux"},
-       { LaTeX::BBL, ".bbl"},
-       { LaTeX::DVI, ".dvi"},
-       { LaTeX::GLO, ".glo"},
-       { LaTeX::IDX, ".idx"},
-       { LaTeX::IND, ".ind"},
-       { LaTeX::LOF, ".lof"},
-       { LaTeX::LOA, ".loa"},
-       { LaTeX::LOG, ".log"},
-       { LaTeX::LOT, ".lot"},
-       { LaTeX::TOC, ".toc"},
-       { LaTeX::LTX, ".ltx"},
-       { LaTeX::TEX, ".tex"}
-};
+namespace {
 
+string runMessage(unsigned int count)
+{
+       return bformat(_("Waiting for LaTeX run number %1$s"), tostr(count));
+}
+
+} // anon namespace
 
 /*
  * CLASS TEXERRORS
@@ -81,21 +96,42 @@ const texfile_struct all_files[] = {
 void TeXErrors::insertError(int line, string const & error_desc,
                            string const & error_text)
 {
-        Error newerr(line, error_desc, error_text);
+       Error newerr(line, error_desc, error_text);
        errors.push_back(newerr);
 }
 
+
+bool operator==(Aux_Info const & a, Aux_Info const & o)
+{
+       return a.aux_file == o.aux_file &&
+               a.citations == o.citations &&
+               a.databases == o.databases &&
+               a.styles == o.styles;
+}
+
+
+bool operator!=(Aux_Info const & a, Aux_Info const & o)
+{
+       return !(a == o);
+}
+
+
 /*
  * CLASS LaTeX
  */
 
-LaTeX::LaTeX(string const & latex, string const & f, string const & p)
-               : cmd(latex), file(f), path(p)
+LaTeX::LaTeX(string const & latex, OutputParams const & rp,
+            string const & f, string const & p)
+       : cmd(latex), file(f), path(p), runparams(rp)
 {
-       tex_files = NO_FILES;
-       file_count = sizeof(all_files) / sizeof(texfile_struct);
        num_errors = 0;
        depfile = file + ".dep";
+       if (prefixIs(cmd, "pdf")) { // Do we use pdflatex ?
+               depfile += "-pdf";
+               output_file = ChangeExtension(file,".pdf");
+       } else {
+               output_file = ChangeExtension(file,".dvi");
+       }
 }
 
 
@@ -106,45 +142,45 @@ void LaTeX::deleteFilesOnError() const
        // What files do we have to delete?
 
        // This will at least make latex do all the runs
-       ::unlink(depfile.c_str());
+       unlink(depfile);
 
        // but the reason for the error might be in a generated file...
 
-       string ofname = OnlyFilename(file);
+       string const ofname = OnlyFilename(file);
 
        // bibtex file
-       string bbl = ChangeExtension(ofname, ".bbl");
-       ::unlink(bbl.c_str());
+       string const bbl = ChangeExtension(ofname, ".bbl");
+       unlink(bbl);
 
        // makeindex file
-       string ind = ChangeExtension(ofname, ".ind");
-       ::unlink(ind.c_str());
-       
+       string const ind = ChangeExtension(ofname, ".ind");
+       unlink(ind);
+
        // Also remove the aux file
-       string aux = ChangeExtension(ofname, ".aux");
-       ::unlink(aux.c_str());
+       string const aux = ChangeExtension(ofname, ".aux");
+       unlink(aux);
 }
 
 
-int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
+int LaTeX::run(TeXErrors & terr)
        // 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
        // in it. However it seems safe to run latex (at least) on time each
        // time the .tex file changes.
 {
-       int scanres = LaTeX::NO_ERRORS;
+       int scanres = NO_ERRORS;
        unsigned int count = 0; // number of times run
        num_errors = 0; // just to make sure.
-       const unsigned int MAX_RUN = 6;
+       unsigned int const MAX_RUN = 6;
        DepTable head; // empty head
        bool rerun = false; // rerun requested
-       
+
        // The class LaTeX does not know the temp path.
-       bufferlist.updateIncludedTeXfiles(GetCWD());
-       
+       bufferlist.updateIncludedTeXfiles(getcwd(), runparams);
+
        // Never write the depfile if an error was encountered.
-       
+
        // 0
        // first check if the file dependencies exist:
        //     ->If it does exist
@@ -160,60 +196,71 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
        //             run latex once (we need to run latex once anyway) and
        //             remake the dependency file.
        //
+
        FileInfo fi(depfile);
+       bool had_depfile = fi.exist();
        bool run_bibtex = false;
-       if (fi.exist()) {
+       string aux_file = OnlyFilename(ChangeExtension(file, "aux"));
+
+       if (had_depfile) {
+               lyxerr[Debug::DEPEND] << "Dependency file exists" << endl;
                // Read the dep file:
-               head.read(depfile);
+               had_depfile = head.read(depfile);
+       }
+
+       if (had_depfile) {
                // Update the checksums
                head.update();
-               
-               lyxerr[Debug::DEPEND] << "Dependency file exists" << endl;
-               if (head.sumchange()) {
-                       ++count;
+               // 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]
-                               << "Dependency file has changed" << endl;
-                       lyxerr[Debug::LATEX]
-                               << "Run #" << count << endl; 
-                       minib->Set(string(_("LaTeX run number ")) + tostr(count));
-                       minib->Store();
-                       this->operator()();
-                       scanres = scanLogFile(terr);
-                       if (scanres & LaTeX::ERRORS) {
-                               deleteFilesOnError();
-                               return scanres; // return on error
-                       }
-                       
-                       run_bibtex = scanAux(head);
-                       if (run_bibtex)
-                               lyxerr[Debug::DEPEND]
-                                       << "Bibtex demands rerun" << endl;
-               } else {
+                               << "re-running LaTeX because output file doesn't exist." << endl;
+               } else if (!head.sumchange()) {
                        lyxerr[Debug::DEPEND] << "return no_change" << endl;
-                       return LaTeX::NO_CHANGE;
+                       return NO_CHANGE;
+               } else {
+                       lyxerr[Debug::DEPEND]
+                               << "Dependency file has changed" << endl;
                }
-       } else {
-               ++count;
+
+               if (head.extchanged(".bib") || head.extchanged(".bst"))
+                       run_bibtex = true;
+       } else
                lyxerr[Debug::DEPEND]
-                       << "Dependency file does not exist" << endl;
-               
-               lyxerr[Debug::LATEX]
-                       << "Run #" << count << endl;
-               head.insert(file, true);
-               minib->Set(string(_("LaTeX run number ")) + tostr(count));
-               minib->Store();
-               this->operator()();
+                       << "Dependency file does not exist, or has wrong format" << endl;
+
+       /// We scan the aux file even when had_depfile = false,
+       /// because we can run pdflatex on the file after running latex on it,
+       /// in which case we will not need to run bibtex again.
+       vector<Aux_Info> bibtex_info_old;
+       if (!run_bibtex)
+               bibtex_info_old = scanAuxFiles(aux_file);
+
+       ++count;
+       lyxerr[Debug::LATEX] << "Run #" << count << endl;
+       message(runMessage(count));
+
+       startscript();
+       scanres = scanLogFile(terr);
+       if (scanres & ERROR_RERUN) {
+               lyxerr[Debug::LATEX] << "Rerunning LaTeX" << endl;
+               startscript();
                scanres = scanLogFile(terr);
-               if (scanres & LaTeX::ERRORS) {
-                       deleteFilesOnError();
-                       return scanres; // return on error
-               }
-               
        }
 
+       if (scanres & ERRORS) {
+               deleteFilesOnError();
+               return scanres; // return on error
+       }
+
+       vector<Aux_Info> const bibtex_info = scanAuxFiles(aux_file);
+       if (!run_bibtex && bibtex_info_old != bibtex_info)
+               run_bibtex = true;
+
        // update the dependencies.
        deplog(head); // reads the latex log
-       deptex(head); // checks for latex files
        head.update();
 
        // 0.5
@@ -222,31 +269,33 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
        // And if there were undefined citations or changes in references
        // the .aux file is checked for signs of bibtex. Bibtex is then run
        // if needed.
-       
+
        // run makeindex
        if (head.haschanged(OnlyFilename(ChangeExtension(file, ".idx")))) {
                // no checks for now
                lyxerr[Debug::LATEX] << "Running MakeIndex." << endl;
-               minib->Set(_("Running MakeIndex."));
-               minib->Store();
-               rerun = runMakeIndex(OnlyFilename(ChangeExtension(file, ".idx")));
+               message(_("Running MakeIndex."));
+               rerun = runMakeIndex(OnlyFilename(ChangeExtension(file, ".idx")), runparams);
        }
 
        // run bibtex
-       if (scanres & LaTeX::UNDEF_CIT
-           || scanres & LaTeX::RERUN
-           || run_bibtex) {
+       // if (scanres & UNDEF_CIT || scanres & RERUN || run_bibtex)
+       if (scanres & UNDEF_CIT || run_bibtex) {
                // Here we must scan the .aux file and look for
                // "\bibdata" and/or "\bibstyle". If one of those
                // tags is found -> run bibtex and set rerun = true;
                // no checks for now
                lyxerr[Debug::LATEX] << "Running BibTeX." << endl;
-               minib->Set(_("Running BibTeX."));
-               minib->Store();
-               rerun = runBibTeX(OnlyFilename(ChangeExtension(file, ".aux")), 
-                                 head);
+               message(_("Running BibTeX."));
+               updateBibtexDependencies(head, bibtex_info);
+               rerun |= runBibTeX(bibtex_info);
+       } 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
        // returned) and the question now is to decide if we need to run
@@ -266,15 +315,14 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
                        << "Dep. file has changed or rerun requested" << endl;
                lyxerr[Debug::LATEX]
                        << "Run #" << count << endl;
-               minib->Set(string(_("LaTeX run number ")) + tostr(count));
-               minib->Store();
-               this->operator()();
+               message(runMessage(count));
+               startscript();
                scanres = scanLogFile(terr);
-               if (scanres & LaTeX::ERRORS) {
+               if (scanres & ERRORS) {
                        deleteFilesOnError();
                        return scanres; // return on error
                }
-               
+
                // update the depedencies
                deplog(head); // reads the latex log
                head.update();
@@ -289,16 +337,15 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
        // again to make sure everything is redone correctly.
        // Also there should be no need to run the external programs any
        // more after this.
-       
+
        // run makeindex if the <file>.idx has changed or was generated.
        if (head.haschanged(OnlyFilename(ChangeExtension(file, ".idx")))) {
                // no checks for now
                lyxerr[Debug::LATEX] << "Running MakeIndex." << endl;
-               minib->Set(_("Running MakeIndex."));
-               minib->Store();
-               rerun = runMakeIndex(OnlyFilename(ChangeExtension(file, ".idx")));
+               message(_("Running MakeIndex."));
+               rerun = runMakeIndex(OnlyFilename(ChangeExtension(file, ".idx")), runparams);
        }
-       
+
        // 2
        // we will only run latex more if the log file asks for it.
        // or if the sumchange() is true.
@@ -309,22 +356,21 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
        //     -> rerun not asked for:
        //             just return (fall out of bottom of func)
        //
-       while ((head.sumchange() || rerun || (scanres & LaTeX::RERUN)) 
+       while ((head.sumchange() || rerun || (scanres & RERUN))
               && count < MAX_RUN) {
                // Yes rerun until message goes away, or until
                // MAX_RUNS are reached.
                rerun = false;
                ++count;
                lyxerr[Debug::LATEX] << "Run #" << count << endl;
-               minib->Set(string(_("LaTeX run number ")) + tostr(count));
-               minib->Store();
-               this->operator()();
+               message(runMessage(count));
+               startscript();
                scanres = scanLogFile(terr);
-               if (scanres & LaTeX::ERRORS) {
+               if (scanres & ERRORS) {
                        deleteFilesOnError();
                        return scanres; // return on error
                }
-               
+
                // keep this updated
                head.update();
        }
@@ -336,161 +382,151 @@ int LaTeX::run(TeXErrors & terr, MiniBuffer * minib)
 }
 
 
-int LaTeX::operator()()
+int LaTeX::startscript()
 {
-#ifndef __EMX__
-       string tmp = cmd + ' ' + QuoteName(file) + " > /dev/null";
-#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);
+       string tmp = cmd + ' ' + QuoteName(file) + " > " + os::nulldev();
+       Systemcall one;
+       return one.startscript(Systemcall::Wait, tmp);
 }
 
 
-bool LaTeX::runMakeIndex(string const & f)
+bool LaTeX::runMakeIndex(string const & f, OutputParams const & runparams)
 {
        lyxerr[Debug::LATEX] << "idx file has been made,"
                " running makeindex on file "
                             <<  f << endl;
-
-       // It should be possible to set the switches for makeindex
-       // sorting style and such. It would also be very convenient
-       // to be able to make style files from within LyX. This has
-       // to come for a later time. (0.13 perhaps?)
-       string tmp = "makeindex -c -q ";
-       tmp += f;
-       Systemcalls one;
-       one.startscript(Systemcalls::System, tmp);
+       string tmp = lyxrc.index_command + " ";
+       tmp = subst(tmp, "$$lang", runparams.document_language);
+       tmp += QuoteName(f);
+       Systemcall one;
+       one.startscript(Systemcall::Wait, tmp);
        return true;
 }
 
 
-bool LaTeX::scanAux(DepTable & dep)
+vector<Aux_Info> const
+LaTeX::scanAuxFiles(string const & file)
 {
-       // if any of the bib file has changed we don't have to
-       // check the .aux file.
-       if (dep.extchanged(".bib")
-           || dep.extchanged(".bst")) return true;
-       
-       string aux = OnlyFilename(ChangeExtension(file, ".aux"));
-       ifstream ifs(aux.c_str());
+       vector<Aux_Info> result;
+
+       result.push_back(scanAuxFile(file));
+
+       for (int i = 1; i < 1000; ++i) {
+               string file2 = ChangeExtension(file, "") + '.' + tostr(i)
+                       + ".aux";
+               FileInfo fi(file2);
+               if (!fi.exist())
+                       break;
+               result.push_back(scanAuxFile(file2));
+       }
+       return result;
+}
+
+
+Aux_Info const LaTeX::scanAuxFile(string const & file)
+{
+       Aux_Info result;
+       result.aux_file = file;
+       scanAuxFile(file, result);
+       return result;
+}
+
+
+void LaTeX::scanAuxFile(string const & file, Aux_Info & aux_info)
+{
+       lyxerr[Debug::LATEX] << "Scanning aux file: " << file << endl;
+
+       ifstream ifs(file.c_str());
        string token;
-       LRegex reg1("\\\\bibdata\\{([^}]+)\\}");
-       LRegex reg2("\\\\bibstyle\\{([^}]+)\\}");
+       static regex const reg1("\\\\citation\\{([^}]+)\\}");
+       static regex const reg2("\\\\bibdata\\{([^}]+)\\}");
+       static regex const reg3("\\\\bibstyle\\{([^}]+)\\}");
+       static regex const reg4("\\\\@input\\{([^}]+)\\}");
+
        while (getline(ifs, token)) {
-               if (reg1.exact_match(token)) {
-                       LRegex::SubMatches sub = reg1.exec(token);
-                       string data = LSubstring(token, sub[1].first,
-                                                sub[1].second);
-                       string::size_type b;
-                       do {
-                               b = data.find_first_of(',', 0);
-                               string l;
-                               if (b == string::npos)
-                                       l = data;
-                               else {
-                                       l = data.substr( 0, b - 0);
-                                       data.erase(0, b + 1);
-                               }
-                               string full_l =
-                                       findtexfile(
-                                               ChangeExtension(l, "bib"), "bib");
-                               if (!full_l.empty()) {
-                                       if (!dep.exist(full_l))
-                                               return true;
-                               }
-                       } while (b != string::npos);
-               } else if (reg2.exact_match(token)) {
-                       LRegex::SubMatches sub = reg2.exec(token);
-                       string style = LSubstring(token, sub[1].first,
-                                                 sub[1].second);
+               token = rtrim(token, "\r");
+               smatch sub;
+               if (regex_match(token, sub, reg1)) {
+                       string data = sub.str(1);
+                       while (!data.empty()) {
+                               string citation;
+                               data = split(data, citation, ',');
+                               lyxerr[Debug::LATEX] << "Citation: "
+                                                    << citation << endl;
+                               aux_info.citations.insert(citation);
+                       }
+               } else if (regex_match(token, sub, reg2)) {
+                       string data = sub.str(1);
+                       // data is now all the bib files separated by ','
+                       // get them one by one and pass them to the helper
+                       while (!data.empty()) {
+                               string database;
+                               data = split(data, database, ',');
+                               database = ChangeExtension(database, "bib");
+                               lyxerr[Debug::LATEX] << "BibTeX database: `"
+                                                    << database << '\'' << endl;
+                               aux_info.databases.insert(database);
+                       }
+               } else if (regex_match(token, sub, reg3)) {
+                       string style = sub.str(1);
                        // token is now the style file
                        // pass it to the helper
-                       string full_l =
-                               findtexfile(
-                                       ChangeExtension(style, "bst"),
-                                       "bst");
-                       if (!full_l.empty()) {
-                               if (!dep.exist(full_l))
-                                       return true;
-                       }
+                       style = ChangeExtension(style, "bst");
+                       lyxerr[Debug::LATEX] << "BibTeX style: `"
+                                            << style << '\'' << endl;
+                       aux_info.styles.insert(style);
+               } else if (regex_match(token, sub, reg4)) {
+                       string const file2 = sub.str(1);
+                       scanAuxFile(file2, aux_info);
                }
        }
-       return false;
 }
 
 
-bool LaTeX::runBibTeX(string const & f, DepTable & dep)
+void LaTeX::updateBibtexDependencies(DepTable & dep,
+                                    vector<Aux_Info> const & bibtex_info)
 {
        // Since a run of Bibtex mandates more latex runs it is ok to
-       // remove all ".bib" and ".bst" files, it is also required to
-       // discover style and database changes.
+       // remove all ".bib" and ".bst" files.
        dep.remove_files_with_extension(".bib");
        dep.remove_files_with_extension(".bst");
-       ifstream ifs(f.c_str());
-       string token;
-       bool using_bibtex = false;
-       LRegex reg1("\\\\bibdata\\{([^}]+)\\}");
-       LRegex reg2("\\\\bibstyle\\{([^}]+)\\}");
-       while (getline(ifs, token)) {
-               if (reg1.exact_match(token)) {
-                       using_bibtex = true;
-                       LRegex::SubMatches const & sub = reg1.exec(token);
-                       string data = LSubstring(token, sub[1].first,
-                                                sub[1].second);
-                       // data is now all the bib files separated by ','
-                       // get them one by one and pass them to the helper
-                       string::size_type b;
-                       do {
-                               b = data.find_first_of(',', 0);
-                               string l;
-                               if (b == string::npos)
-                                       l = data;
-                               else {
-                                       l = data.substr(0, b - 0);
-                                       data.erase(0, b + 1);
-                               }
-                               string full_l = 
-                                       findtexfile(
-                                               ChangeExtension(l, "bib"),
-                                               "bib");
-                               lyxerr[Debug::LATEX] << "Bibtex database: `"
-                                                    << full_l << "'" << endl;
-                               if (!full_l.empty()) {
-                                       // add full_l to the dep file.
-                                       dep.insert(full_l, true);
-                               }
-                       } while (b != string::npos);
-               } else if (reg2.exact_match(token)) {
-                       using_bibtex = true;
-                       LRegex::SubMatches const & sub = reg2.exec(token);
-                       string style = LSubstring(token, sub[1].first,
-                                                 sub[1].second);
-                       // token is now the style file
-                       // pass it to the helper
-                       string full_l = 
-                               findtexfile(
-                                       ChangeExtension(style, "bst"),
-                                       "bst");
-                       lyxerr[Debug::LATEX] << "Bibtex style: `"
-                                            << full_l << "'" << endl;
-                       if (!full_l.empty()) {
-                               // add full_l to the dep file.
-                               dep.insert(full_l, true);
-                       }
+       //string aux = OnlyFilename(ChangeExtension(file, ".aux"));
+
+       for (vector<Aux_Info>::const_iterator it = bibtex_info.begin();
+            it != bibtex_info.end(); ++it) {
+               for (set<string>::const_iterator it2 = it->databases.begin();
+                    it2 != it->databases.end(); ++it2) {
+                       string file = findtexfile(*it2, "bib");
+                       if (!file.empty())
+                               dep.insert(file, true);
+               }
+
+               for (set<string>::const_iterator it2 = it->styles.begin();
+                    it2 != it->styles.end(); ++it2) {
+                       string file = findtexfile(*it2, "bst");
+                       if (!file.empty())
+                               dep.insert(file, true);
                }
        }
-       if (using_bibtex) {
-               // run bibtex and
-               string tmp = "bibtex ";
-               tmp += OnlyFilename(ChangeExtension(file, string()));
-               Systemcalls one;
-               one.startscript(Systemcalls::System, tmp);
-               return true;
+}
+
+
+bool LaTeX::runBibTeX(vector<Aux_Info> const & bibtex_info)
+{
+       bool result = false;
+       for (vector<Aux_Info>::const_iterator it = bibtex_info.begin();
+            it != bibtex_info.end(); ++it) {
+               if (it->databases.empty())
+                       continue;
+               result = true;
+
+               string tmp = lyxrc.bibtex_command + " ";
+               tmp += QuoteName(OnlyFilename(ChangeExtension(it->aux_file, string())));
+               Systemcall one;
+               one.startscript(Systemcall::Wait, tmp);
        }
-       // bibtex was not run.
-       return false;
+       // Return whether bibtex was run
+       return result;
 }
 
 
@@ -506,7 +542,7 @@ int LaTeX::scanLogFile(TeXErrors & terr)
        string token;
        while (getline(ifs, token)) {
                lyxerr[Debug::LATEX] << "Log line: " << token << endl;
-               
+
                if (token.empty())
                        continue;
 
@@ -534,8 +570,22 @@ int LaTeX::scanLogFile(TeXErrors & terr)
                                    && contains(token, "undefined")) {
                                        retval |= UNDEF_CIT;
                                }
-                       } else if (contains(token, "Rerun LaTeX.")) {
-                               // at least longtable.sty might use this.
+                       } else if (contains(token, "run BibTeX")) {
+                               retval |= UNDEF_CIT;
+                       } 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 (token[0] == '(') {
+                       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, "! ")) {
@@ -558,6 +608,8 @@ int LaTeX::scanLogFile(TeXErrors & terr)
                        if (prefixIs(tmp, "l.")) {
                                // we have a latex error
                                retval |=  TEX_ERROR;
+                               if (contains(desc, "Package babel Error: You haven't defined the language"))
+                                       retval |= ERROR_RERUN;
                                // get the line number:
                                int line = 0;
                                sscanf(tmp.c_str(), "l.%d", &line);
@@ -597,7 +649,7 @@ int LaTeX::scanLogFile(TeXErrors & terr)
                                retval |= TEX_WARNING;
                        } else if (contains(token, "Rerun to get citations")) {
                                // Natbib seems to use this.
-                               retval |= RERUN;
+                               retval |= UNDEF_CIT;
                        } else if (contains(token, "No pages of output")) {
                                // A dvi file was not created
                                retval |= NO_OUTPUT;
@@ -612,106 +664,129 @@ int LaTeX::scanLogFile(TeXErrors & terr)
 }
 
 
+namespace {
+
+void handleFoundFile(string const & ff, DepTable & head)
+{
+       // convert from native os path to unix path
+       string const foundfile = os::internal_path(trim(ff));
+
+       lyxerr[Debug::DEPEND] << "Found file: " << foundfile << endl;
+
+       // Ok now we found a file.
+       // Now we should make sure that this is a file that we can
+       // access through the normal paths.
+       // We will not try any fancy search methods to
+       // find the file.
+
+       // (1) foundfile is an
+       //     absolute path and should
+       //     be inserted.
+       if (AbsolutePath(foundfile)) {
+               lyxerr[Debug::DEPEND] << "AbsolutePath file: "
+                                     << foundfile << endl;
+               // On initial insert we want to do the update at once
+               // since this file can not be a file generated by
+               // the latex run.
+               if (FileInfo(foundfile).exist())
+                       head.insert(foundfile, true);
+
+               return;
+       }
+
+       string const onlyfile = OnlyFilename(foundfile);
+
+       // (2) foundfile is in the tmpdir
+       //     insert it into head
+       if (FileInfo(onlyfile).exist()) {
+               static regex unwanted("^.*\\.(aux|log|dvi|bbl|ind|glo)$");
+               if (regex_match(onlyfile, unwanted)) {
+                       lyxerr[Debug::DEPEND]
+                               << "We don't want "
+                               << onlyfile
+                               << " in the dep file"
+                               << endl;
+               } else if (suffixIs(onlyfile, ".tex")) {
+                       // This is a tex file generated by LyX
+                       // and latex is not likely to change this
+                       // during its runs.
+                       lyxerr[Debug::DEPEND]
+                               << "Tmpdir TeX file: "
+                               << onlyfile
+                               << endl;
+                       head.insert(onlyfile, true);
+               } else {
+                       lyxerr[Debug::DEPEND]
+                               << "In tmpdir file:"
+                               << onlyfile
+                               << endl;
+                       head.insert(onlyfile);
+               }
+       } else
+               lyxerr[Debug::DEPEND]
+                       << "Not a file or we are unable to find it."
+                       << endl;
+}
+
+} // anon namespace
+
+
 void LaTeX::deplog(DepTable & head)
 {
        // This function reads the LaTeX log file end extracts all the external
        // files used by the LaTeX run. The files are then entered into the
        // dependency file.
 
-       string logfile = OnlyFilename(ChangeExtension(file, ".log"));
+       string const logfile = OnlyFilename(ChangeExtension(file, ".log"));
 
-       LRegex reg1(")* *\\(([^ ]+).*");
-       LRegex reg2("File: ([^ ]+).*");
+       static regex reg1(".*\\([^)]+.*");
+       static regex reg2("File: ([^ ]+).*");
+       static regex reg3("No file ([^ ]+)\\..*");
+       static regex 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
+       static regex reg5("Writing index file ([^ ]+).*");
 
        ifstream ifs(logfile.c_str());
        while (ifs) {
                // Ok, the scanning of files here is not sufficient.
                // Sometimes files are named by "File: xxx" only
                // So I think we should use some regexps to find files instead.
-               // "(\([^ ]+\)"   should match the "(file " variant
+               // "(\([^ ]+\)"   should match the "(file " variant, note
+               // that we can have several of these on one line.
                // "File: \([^ ]+\)" should match the "File: file" variant
-               string foundfile;
+
                string token;
                getline(ifs, token);
-               if (token.empty()) continue;
-               
-               if (reg1.exact_match(token)) {
-                       LRegex::SubMatches const & sub = reg1.exec(token);
-                       foundfile = LSubstring(token, sub[1].first,
-                                              sub[1].second);
-               } else if (reg2.exact_match(token)) {
-                       LRegex::SubMatches const & sub = reg2.exec(token);
-                       foundfile = LSubstring(token, sub[1].first,
-                                              sub[1].second);
-               } else {
+               token = rtrim(token, "\r");
+               if (token.empty())
                        continue;
-               }
 
-               lyxerr[Debug::DEPEND] << "Found file: " 
-                                    << foundfile << endl;
-               
-               // Ok now we found a file.
-               // Now we should make sure that this is a file that we can
-               // access through the normal paths.
-               // We will not try any fance search methods to
-               // find the file.
-               
-               // (1) foundfile is an
-               //     absolute path and should
-               //     be inserted.
-               if (AbsolutePath(foundfile)) {
-                       lyxerr[Debug::DEPEND] << "AbsolutePath file: " 
-                                             << foundfile << endl;
-                       // On inital insert we want to do the update at once
-                       // since this file can not be a file generated by
-                       // the latex run.
-                       head.insert(foundfile, true);
-                       continue;
-               }
+               smatch sub;
+
+               if (regex_match(token, sub, reg1)) {
+                       static regex reg1_1("\\(([^()]+)");
+                       smatch what;
+                       string::const_iterator first = token.begin();
+                       string::const_iterator end = token.end();
 
-               // (2) foundfile is in the tmpdir
-               //     insert it into head
-               if (FileInfo(OnlyFilename(foundfile)).exist()) {
-                       if (suffixIs(foundfile, ".aux")) {
-                               lyxerr[Debug::DEPEND] << "We don't want "
-                                                    << OnlyFilename(foundfile)
-                                                    << " in the dep file"
-                                                    << endl;
-                       } else if (suffixIs(foundfile, ".tex")) {
-                               // This is a tex file generated by LyX
-                               // and latex is not likely to change this
-                               // during its runs.
-                               lyxerr[Debug::DEPEND] << "Tmpdir TeX file: "
-                                                    << OnlyFilename(foundfile)
-                                                    << endl;
-                               head.insert(foundfile, true);
-                       } else {
-                               lyxerr[Debug::DEPEND] << "In tmpdir file:"
-                                                    << OnlyFilename(foundfile)
-                                                    << endl;
-                               head.insert(OnlyFilename(foundfile));
+                       while (regex_search(first, end, what, reg1_1)) {
+                               first = what[0].second;
+                               handleFoundFile(what.str(1), head);
                        }
-                       continue;
+               } else if (regex_match(token, sub, reg2)) {
+                       handleFoundFile(sub.str(1), head);
+               } else if (regex_match(token, sub, reg3)) {
+                       handleFoundFile(sub.str(1), head);
+               } else if (regex_match(token, sub, reg4)) {
+                       handleFoundFile(sub.str(1), head);
+               } else if (regex_match(token, sub, reg5)) {
+                       handleFoundFile(sub.str(1), head);
                }
-               lyxerr[Debug::DEPEND]
-                       << "Not a file or we are unable to find it."
-                       << endl;
        }
-}
 
-
-void LaTeX::deptex(DepTable & head)
-{
-       int except = AUX|LOG|DVI|BBL|IND|GLO; 
-       string tmp;
-       FileInfo fi;
-       for (int i = 0; i < file_count; ++i) {
-               if (!(all_files[i].file & except)) {
-                       tmp = OnlyFilename(ChangeExtension(file,
-                                                          all_files[i].extension));
-                       lyxerr[Debug::DEPEND] << "deptex: " << tmp << endl;
-                       if (fi.newFile(tmp).exist())
-                               head.insert(tmp);
-               }
-       }
+       // Make sure that the main .tex file is in the dependancy file.
+       head.insert(OnlyFilename(file), true);
 }