You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2149
-- Commit Summary --
* .travis.yml:add tests for gcc 7 and 8 * use build matrix #2139 * for gcc 8, specify xenial and the correct version * try globalizing the GTK3 env directives * try indenting the matrix to get global to work * revert a little * fix indentation * add clang 5 test (closes #2139) * only run `make distcheck` for a single job * specify dist and compiler explicity * remove unneeded hyphenated lines * add DISTCHECK_ONLY, don't hide it under gcc-5 * use xenial instead of trusty * reduce clang to 3.5 * try default clang on xenial * try default clang on trusty * clang:revert to using 5.0 on trusty * allow clang to fail * remove cache directive, try clang 3.5 again; also... * specifying 3.5 on trusty * install libc++ * clang:explicity enable C++11 support * make:temporarily add -n and VERBOSE=1 * use clang 3.8
-- File Changes --
M .travis.yml (77)
-- Patch Links --
https://github.com/geany/geany/pull/2149.patch https://github.com/geany/geany/pull/2149.diff
@b4n pushed 3 commits.
051701ea6a4fa85bae78054fbdb7dd53778a808c travis: Enable verbose build e7a0691759612328ce114a5fc38498053465d1f5 travis: Explicitly specify make targets to build 5b9529ce7d5902748bb9dab33e00769dddffb080 travis: Mark the clang build as using clang
@b4n pushed 1 commit.
363d6d9247035e3a4dab74037495b91ae4cd8e9d fixup! travis: Enable verbose build
Well, at least its consistently failing :grin: also only seems to build GTK3 or is that temporary and deliberate?
Well, at least its consistently failing
Problem is, I have no clue what the heck is failing…
also only seems to build GTK3
how so?
@b4n pushed 1 commit.
01b90bdb71461727d9443bf24183df6bad241cdc travis: Test running make targets separately
@b4n pushed 1 commit.
156f6895562d08007d10504b2d14252507a9d586 travis: Try and debug this weirdy
also only seems to build GTK3
how so?
Its ok, I just glanced at the details and it said GTK3 everywhere, and I missed the `=yes` `=no` after it :)
@b4n pushed 3 commits.
6d0086091767db491ec708aec7d4575531ebf0e5 travis: Run targets one after the other manually 82f0d006c947217292f16b3675668d8ad14885f2 fixup: set TARGETS on clang too b83e98be2b7fbbbc42c4a01574302fbaad39ae89 travis: Run clang tests simply by using different distros
@b4n pushed 2 commits.
ed51ed1fd31caadccdc2672f0d9ad9f5a8d96e41 travis: Remove some debugging code 97aaeea878241475877c66e72dbf9ec84beeb6bf scintilla: Don't make LineMarker() constructors noexcept to try and fix clang 3.x build
@b4n ok, got the answer.
There was a defect report and change on the C++ standard to allow the implicit exception spec to be different from the explicit exception spec of defaulted functions. So any compiler before that DR was implemented will fail because previously the spec required them to match.
I figured we're most likely just gonna have to drop clang 5.x support at least until Mitchell decides on something.
@b4n pushed 1 commit.
d2bad47736f77c8978e7fc2e2e57552da5f640a5 fixup! travis: Drop clang 3.4 and 5.0 checks, and add a clang 6 check
@b4n with regard to reducing verbosity, you can see an alternative I'm experimenting with on this other travis file I'm working on: https://github.com/gerddie/ginkgocadx/pull/49/files
On [L35](https://github.com/gerddie/ginkgocadx/blob/565d3ac2d25328073cdc2f2f9bbb09c4d...) the toolchain repo is added and on [L40](https://github.com/gerddie/ginkgocadx/blob/565d3ac2d25328073cdc2f2f9bbb09c4d...) the appropriate gcc package is installed.
btw, is it widely known yet that Travis does Windows builds now? https://docs.travis-ci.com/user/reference/windows/
@b4n with regard to reducing verbosity, you can see an alternative I'm experimenting with on this other travis file I'm working on: https://github.com/gerddie/ginkgocadx/pull/49/files
On [L35](https://github.com/gerddie/ginkgocadx/blob/565d3ac2d25328073cdc2f2f9bbb09c4d...) the toolchain repo is added and on [L40](https://github.com/gerddie/ginkgocadx/blob/565d3ac2d25328073cdc2f2f9bbb09c4d...) the appropriate gcc package is installed.
on @b4n's repo, I submitted https://github.com/b4n/geany/pull/4 to try something like this.
Ubuntu bionic is now available on Travis https://docs.travis-ci.com/user/reference/bionic/
It would mean not having to install the toolchain repo
You can see here that gcc versions up to 8 are included: https://packages.ubuntu.com/bionic/devel/
And now.. testing for Ubuntu 20.04 is available.. ;)
https://blog.travis-ci.com/2020-08-10-focal-build-environment
Since we recently switched to Github Actions, the changes no longer apply. I wonder if it is useful at all to test so many compiler variants. At first, it would require much more resources in the CI system for maybe only a little use, IMO.
The matrix configuration might be even easier with Github Actions if still wanted.
This PR can be closed due to https://github.com/geany/geany/pull/2948
github-comments@lists.geany.org