Skip to content
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

Improve travis deployment #769

Merged
merged 2 commits into from
Jun 17, 2019
Merged

Conversation

JonasVautherin
Copy link
Collaborator

  • Deploy manylinux backend binary
  • Deploy macos backend binary
  • Fix rpm/deb deployment (that I broke when reorganizing the deps)

Notes:

  • The rpm/deb packages now include backend_bin (which I believe we should rename, e.g. to mavsdk_backend, but I will open another PR to discuss that).
  • I removed a few Debug builds, as it doubles the build time and that's important when building with -DBUILD_BACKEND=ON. If we really need to build in Debug mode, we could make a nightly build, IMO.
  • create_packages.sh is now called by travis-docker-build.sh, which IMO makes it cleaner since the later also calls fix_style.sh and generate_docs.sh.
  • create_packages.sh takes arguments now. The intent is to make it easier to use without relying on a convention like: "install into build/default/install/usr".
  • create_packages.sh copies the files into a temp dir, and adds /usr/ there. The benefit of this is that we don't have to build from scratch again just for packaging, improving the CI build time.

@JonasVautherin
Copy link
Collaborator Author

Trying it on a tag (v0.17.1) and it seems to work 🎉

Copy link
Collaborator

@julianoes julianoes left a comment

Choose a reason for hiding this comment

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

All good. And I'm assuming we're building for all Fedora and Ubuntu's now using manylinux?

@JonasVautherin JonasVautherin merged commit f6c689f into develop Jun 17, 2019
@JonasVautherin JonasVautherin deleted the improve-travis-deployment branch June 17, 2019 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants