-
Notifications
You must be signed in to change notification settings - Fork 3k
Build and run latest released doxygen in Travis CI #8593
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
Conversation
Tweaked doxygen failure command
Is that correct, we are up to date? :-) |
Guys just as a comment, the team is "mbed docs". "mbed-os-docs" has just me, and I have no right to manage that team so it won't be up to date. |
...huh. Well, until that gets fixed, @AnotherButler @melwee01 would y'all mind taking a look? |
@0xc0170 To be more specific, the job is now up to date. There are still doxygen issues, but they won't block CI until we can get them sorted out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me, but I don't really know what I'm looking at!
Note, @adbridge Auto reviewers should use "mbed docs" team |
/morph build |
Build : SUCCESSBuild number : 3511 Triggering tests/morph test |
Exporter Build : FAILUREBuild number : 3130 |
@iriark01 Shall i delete the mbed-os-docs team from GitHub then to avoid confusion ? |
@adbridge yes please |
Test : SUCCESSBuild number : 3297 |
/morph export-build |
Exporter Build : SUCCESSBuild number : 3134 |
Description
It was discovered yesterday that Travis CI is not running the latest released version of doxygen, since the latest released vesrion in Ubuntu Trusty is 1.8.6, not 1.8.14. Further, Travis CI does not have a PPA/distro that has the latest version.
The latest vesrion discovers issues in the repo that the older version does not.
Fixes #8574
The fix
This fix locally builds and installs the latest released version inside Travis CI. A potential future optimization could be to store this binary elsewhere instead of building it every time, but as of now the build step does not take long.
Future work
Once doxygen issues are fixed, the command that will fail the build on new errors can be enabled.
Pull request type