-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
create dummy images in advance for doxygen not to complain #11614
Conversation
Starting build on |
I see the idea... you are creating empty pictures first just to remove the warnings? |
Exactly. Just a workaround, in order to be able to focus on the other real warnings. |
I see the idea. I will try locally |
The |
You are looking only for |
And by the way, I cannot reproduce locally these warnings. For example:
On the log-ouput you pointed we can see a warning about |
Starting build on |
Good point. Changed. |
Maybe it would be useful to add to the Jenkins build a step that cleans the git repository on sftnight before starting the build? To get fresh builds there too? |
Build failed on windows10/cxx14. Failing tests: |
For example using the git clean command within https://github.com/root-project/rootspi/blob/master/rdoc/preparesource.sh: |
It is more tricky than I thought. Actually I see the error "sometimes" on fresh builds only. That's not clear yet ... |
Maybe it's just a multi-threading thing? The output lines are ordered differently each time because the threads write to stdout in different order. If you Search for "latex5.C", don't you see always a warning line (every time in different position): |
I thought of it, that's why I do not use the option |
20388c1
to
fdd198a
Compare
Starting build on |
Do we still want this? It does not seem really safe... |
This Pull request:
Changes or fixes:
Currently, building the documentation issues 3700 warnings, see https://lcgapp-services.cern.ch/root-jenkins/view/ROOT/job/root-makedoc-master/lastBuild/parsed_console/
With the current hack, we would go down to just 2700.
By the way, it would be nice to fine-tune the parser: see root-project/jenkins-pipelines#10
Checklist:
See related discussion doxygen/doxygen#9044