ethereum / yellowpaper

The "Yellow Paper": Ethereum's formal specification
Creative Commons Attribution Share Alike 4.0 International
1.65k stars 511 forks source link

To do: add to build.sh to pick up other errors, e.g. LaTeX errors, etc. #631

Open jamesray1 opened 6 years ago

jamesray1 commented 6 years ago

Fix the existing warnings and errors first then do this in separate PRs, You could also use regular expressions to capture the whole line.

E.g. roughly in decreasing order of severity (I've copied and pasted whole lines in some cases and others just the general error/warning):

Warning--I didn't find a database entry for "ECDSAcerticom" % the citation will show as a ? in the paper.
Package natbib Warning: Citation `ECDSAcerticom' on page 22 undefined on input 
line 1661. % only if this occurs in every or the last pass.
[6pdfTeX warning (ext4): destination with the same identifier (name{L__B}) has 
been already used, duplicate ignored
% The rest are all non-critical errors.
LaTeX Font Warning: Command \small invalid in math mode on input line
Underfull \vbox
Overfull \hbox
Overfull \vbox
Underfull \hbox
LaTeX Font Warning: Command \tiny invalid in math mode on input line
Package multicol Warning: I moved some lines to the next page.
(multicol)                Footnotes on page 5 might be wrong on input line 403.
jamesray1 commented 6 years ago

Wait until #295 is merged to avoid merge conflicts.

jamesray1 commented 6 years ago

Related: https://github.com/ethereum/yellowpaper/issues/541

pirapira commented 6 years ago

295 has been merged.

jamesray1 commented 6 years ago

That's great, although obviously there are still the above errors to detect.