Skip to content

Commit

Permalink
fix doxygen test on Travis
Browse files Browse the repository at this point in the history 
it seems that doxygen 1.18.13 is not behaving the same as 1.1811 (Travis one).
one might need to remove some spaces in the signature as they are different depending on Doxygen's version
  • Loading branch information
@3nids
3nids committed Aug 6, 2017
1 parent 7946908 commit 518ffe3
Show file tree
Hide file tree
Showing 4 changed files with 241 additions and 230 deletions.
5 changes: 3 additions & 2 deletions .ci/travis/code_layout/install.sh
View file
Expand Up @@ -16,7 +16,8 @@
export CORES=2

mkdir build
cd build
pushd build

cmake -DWITH_CORE=OFF -DWITH_APIDOC=ON -DWITH_ASTYLE=ON -DENABLE_TESTS=ON ..
make -j${CORES}

popd
6 changes: 5 additions & 1 deletion .ci/travis/code_layout/script.sh
View file
Expand Up @@ -15,6 +15,10 @@
set -e

pushd build

export CTEST_BUILD_COMMAND="/usr/bin/make -j3 -i -k"
python ${TRAVIS_BUILD_DIR}/.ci/travis/scripts/ctest2travis.py xvfb-run ctest -V --output-on-failure -S ${TRAVIS_BUILD_DIR}/.ci/travis/travis.ctest
python3 ${TRAVIS_BUILD_DIR}/.ci/travis/scripts/ctest2travis.py xvfb-run ctest -V --output-on-failure -S ${TRAVIS_BUILD_DIR}/.ci/travis/travis.ctest

ctest -V -R PyQgsDocCoverage

popd

2 comments on commit 518ffe3

@nyalldawson
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You know... and alternative solution to getting this to pass both locally and on Travis would be just to add the dox for the members which are misbehaving...

@3nids
Copy link
Member Author

@3nids 3nids commented on 518ffe3 Aug 7, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documenting instead of hacking to fix the documentation test is indeed an interesting approach

Please sign in to comment.