anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-11-09 18:31:04 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Adapt to new ChkTeX return values.

Browse Source 
As of v. 1.7.7, chktex has four exit values. Only consider the program
failed with EXIT_FAILURE (1). This is backwards compatible to chktex
up to v. 1.7.5 and later patched versions included in TeXLive, where
there was the distinction EXIT_FAILURE (program failed) and EXIT_SUCCESS
(program successfully run, with or without something to report).

Note that ChkTeX v. 1.7.5 and 1.7.6 vanilla (as included in MikTeX) also
returned EXITE_FAILURE if ChkTeX found something to report.

We do not, and never did, support this case.

Fixes: #9989 (after ChkTeX 1.7.7. is released).
This commit is contained in:
Juergen Spitzmueller 2018-02-17 11:25:28 +01:00
parent ac1d6af008
commit 0d806799aa
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/Chktex.cpp
View File

@ -34,16 +34,22 @@ Chktex::Chktex(string const & chktex, string const & f, string const & p)
int Chktex::run(TeXErrors &terr)
{
// run bibtex
// run chktex
string log = onlyFileName(changeExtension(file, ".log"));
string tmp = cmd + " -q -v0 -b0 -x " + file + " -o " + log;
Systemcall one;
int result = one.startscript(Systemcall::Wait, tmp);
if (result == 0) {
result = scanLogFile(terr);
} else {
// ChkTeX (as of v. 1.7.7) has the following return values:
// 0 = EXIT_SUCCESS : program run successfully, nothing to report
// 1 = EXIT_FAILURE : program run unsucessfully
// 2 = EXIT_WARNINGS : program run successfully, only warnings to report
// 3 = EXIT_ERRORS : program run successfully, errors to report
// We only check for EXIT_FAILURE here, since older versions of ChkTeX
// returned 0 also in case 2 and 3.
if (result == EXIT_FAILURE)
result = -1;
}
else
result = scanLogFile(terr);
return result;
}