Skip to content
Snippets Groups Projects

Add .travis.yml for Travis CI

Closed Niklas Haas requested to merge ePirat:master into master

Created by: ePirat

This adds continuous integration through Travis CI

An example can be found here: https://travis-ci.org/ePirat/libplacebo

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Maintainer

    A few comments:

    1. ninja test does nothing unless you append -Dtests=true to the meson step.
    2. wgetting a fixed release version of the ninja command like that seems suspect. Likely nobody will ever think to update this URL. How do we avoid it becoming stale?
  • Author Maintainer

    Seems the version of ninja in the trusty repos is way too old. We might still be able to add a PPA though, which would have a better chance of staying up-to-date than some random hard-coded URL.

  • Author Maintainer

    Created by: ePirat

    I don't think hardcoding a URL is a big problem. If it ever is too old to work with changes in your ninja build file, it will fail and we will notice and update it, no? I agree it is far from perfect. If you find a PPA that we can use, let me know and I'll give it a try.

  • Author Maintainer

    Yeah, fair enough. After all, this is just for testing - as long as the test environment builds, the version is fine. It's not like we're distributing a binary or something; and performance+bug fixes most likely don't affect us.

Please register or sign in to reply
Loading