X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FLaTeX.C;h=8609c3ed53dc20f92c5167fc0305b71f8574d928;hb=701b99ecd70ac472aa53c8b2317af44def4f9670;hp=70510fff61cf246430f3496f4563f688858f1a4c;hpb=c5d617710f7bc0dec3140a9134a3ddcae535fc26;p=lyx.git diff --git a/src/LaTeX.C b/src/LaTeX.C index 70510fff61..8609c3ed53 100644 --- a/src/LaTeX.C +++ b/src/LaTeX.C @@ -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 -#ifdef __GNUG__ -#pragma implementation -#endif -#include - -#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 + +#include + +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 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 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 .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 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 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 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::const_iterator it = bibtex_info.begin(); + it != bibtex_info.end(); ++it) { + for (set::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::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 const & bibtex_info) +{ + bool result = false; + for (vector::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); }