Skip to content

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

Merged
merged 2 commits into from
Oct 31, 2018

Conversation

cmonr
Copy link
Contributor

@cmonr cmonr commented Oct 30, 2018

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

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 30, 2018

travis-ci/docs — Local docs testing has passed

Is that correct, we are up to date? :-)

@iriark01
Copy link
Contributor

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.

@cmonr
Copy link
Contributor Author

cmonr commented Oct 30, 2018

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?

@cmonr
Copy link
Contributor Author

cmonr commented Oct 30, 2018

travis-ci/docs — Local docs testing has passed
Is that correct, we are up to date? :-)

@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.

Copy link
Contributor

@melwee01 melwee01 left a 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!

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 31, 2018

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.

Note, @adbridge Auto reviewers should use "mbed docs" team

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 31, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

Build : SUCCESS

Build number : 3511
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8593/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

@adbridge
Copy link
Contributor

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.

@iriark01 Shall i delete the mbed-os-docs team from GitHub then to avoid confusion ?

@iriark01
Copy link
Contributor

@adbridge yes please

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 31, 2018

/morph export-build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

@cmonr cmonr merged commit d8e4dae into ARMmbed:master Oct 31, 2018
@cmonr cmonr removed the needs: CI label Oct 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Travis doxygen binary could use an update
7 participants