[Github-comments] [geany/geany] add gcc 7 and 8 build to travis tests (#2067)

[Andy Alt]

> However, we most probably don't need to run `distcheck` on all compilers, as it's mostly checking the build system itself rather than the compilation. Only `all` (obviously) and `check` are really useful to run everywhere for a sensible test, and that might give a comfortable speed up by not building everything twice. This can be changed at some later point tho, and can probably be fixed by e.g. setting `TARGETS="all check distcheck"` in the "normal" case and removing `distcheck` in the "compiler builds", and replacing the `make -j2 && make -j2 check && make -j2 distcheck` with just `make -j2 $TARGETS`.

I pushed a commit that addresses that. But I made the change before I reviewed your suggested approach. If you'd like it changed, please let me know.

> 
> Anyway, the change seems a little verbose, but there might not be a shorter syntax for this (I'm no YAML expert tho).

That's another copy/paste from the travis docs. afaik the only way to reduce the lines is to use a script that manually imports the toolchain repo and keys, and installs the appropriate gcc package.

However, using the the "addons/source" travis directive as it is now may be easier in the long-run, to maintain. What do you think?



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2067#issuecomment-488541374
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190502/f77f0c62/attachment.html>