-
Notifications
You must be signed in to change notification settings - Fork 13
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
Integrate Travis-CI #2
Comments
I gave it a try here https://github.com/khaledhosny/ScribusCTL/commits/travis-ci, but Travis CI uses Ubuntu 12.04 which is rather ancient and the build is now breaking on a too old zlib package and I couldn’t find a PPA providing a new one, and I doubt this will be the only breakage. |
@khaledhosny nice work!
This is rather depressing. |
Travis does not accept new projects for Mac OS X building anymore, it was a beta feature or some such. Only Linux is supported, but using that ancient Ubuntu. We can probably get this to work if we can make the Scribus PPA to support 12.04, but this will require backporting a few packages and I don’t have an Ubuntu installation right now. |
@khaledhosny can you provide a link to your failing build ? |
There is also Appveyor for windows. Thanks to @ebraminio for the heads up on that: khaledhosny/ots#67 |
Here is musescore .travis.yml file: https://github.com/musescore/MuseScore/blob/master/.travis.yml |
Interesting… |
I had to make a few changes: a) patch So I guess the last step is to find a PPA for a newer version of clang if you want to be building with it. |
It doesn't look like using newer clang works. It seems to be a bug in OpenSceneGraph, and I've yet to find a PPA for it. Since OpenSceneGraph is optional, I disabled it for clang only, and now the build passes. |
Epic @QuLogic ! |
@QuLogic, please make a PR to add that .travis.yml file to the repo |
Clang seems to get killed pretty often. I'm not sure if it's OOMing or is just buggy. I even tried reducing the parallelism to avoid it (it's only 3 in the mentioned build). |
hmm.. it doesn't happen each time you trigger a build though, right ? |
With NPROC=3, it's failed once and passed once. Let's see how the build in the PR goes. |
Actually, if you count the build in the fork of the main project, then it's passed twice. |
Annnd, it just failed on yours, so it's 50/50. 😞 |
What do you mean by: |
The Travis container-based infrastructure is faster (for booting, officially, but also for running). But you can't use And it looks like even building with 2 processors can fail. 😢 Building with clang in serial only takes 37 minutes and that leaves a bit of room under the 50 minute limit, but it's kind of annoying. |
I see. Thanks for the explanation. Well.. development will be slow with ScribusCTL for the time being. I say this because Andreas told me that he will have time this upcoming weekend to work on it. So we aren't in a hurry because he calls the shots. My opinion: it's ok to not parallelize the build and even give up the gcc build and stick to clang. |
No, you have it backwards; gcc works in parallel, and clang only works (consistently) in serial. |
@QuLogic @khaledhosny good news, TravisCI is rolling out an invite by request the possibility to build on Ubuntu 14.04. https://twitter.com/travisci/status/619027935857627138 |
It looks like that will negate the need for patching OpenSceneGraph and zlib stuff, and also probably won't need a PPA for gcc/g++ and cmake. |
From the Travis CI folks after enabling Ubuntu 14.04
|
Does that apply to forks as well, or will I have to work in the PR here? |
Guestimating, that if it's in your fork, then you need to request to join the beta testing 14.04 program so they can enable your account. |
@QuLogic the devs tell me that we don't need webkit anymore, in regards to the libqt5webkit5-dev in .travis.yml file |
Good news.. multi-os now supported https://twitter.com/travisci/status/629051392217477120 |
@andreas-vox you can go ahead and copy the SVN copy of .travis.yml in to this repo. It's been updated and also includes MacOSX test build |
@andreas-vox I know this is semi low priority, but we can now test PRs through travis again after the script was fixed. Feel free to cherry pick |
Closing this issue in favor of scribusproject/scribus#3 |
Changelog
Integrate different tests
Mirrored issue at scribusproject/scribus#3
Copied from original issue: scribusproject/scribus#3
The text was updated successfully, but these errors were encountered: