This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106617/

Ship it!

Ok, let's see how well this will work :)

- Michel


On September 27th, 2012, 8:37 p.m., Eugene Shalygin wrote:

Review request for Kile and Michel Ludwig.
By Eugene Shalygin.

Updated Sept. 27, 2012, 8:37 p.m.

Description

Kile rerun bibtool every time the user changes some .tex file. This is not a problem on itself, but then it reruns latex. And latex run might take a long time.

The patch tries to eliminate bibtool (and thus latex) runs under some conditions.

It suppose that we run bibtool only in the following cases:
		1. Biblatex said that we have to (in this case bibToolInLaTexOutput is not empty)
		2. At least one of the .bib has modification date younger then .bbl file
		3. We have undefined citations and any of source files (including .bib and .tex) is younger then .bbl. If .bbl file is younger then all of them, then next generation will not change anything

To achive that, it adds check for "undefined citation" latex message and changes LaTeX::updateBibs() in a way that allows to check only bib ddependencies or all dependencies.
Also, it filters biblatex rerun messages when citation is missing in the database: there is no need to just rerun latex, it will not change anything. Before Kile was launching latex 3 times in a row without any effect.

Testing

Manual testing

Diffs

  • src/kilestdtools.h (d8ee576)
  • src/kilestdtools.cpp (6a2ae95)

View Diff