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

snap: Add missing sudo in Travis deploy step #548

Merged
merged 1 commit into from Feb 8, 2020

Conversation

ppd
Copy link
Member

@ppd ppd commented Feb 8, 2020

Running the build step with sudo necessitates running the deploy step with sudo too.
Otherwise, we face permission problems.

Running the build step with sudo necessitates running the deploy step with sudo too.
Otherwise, we face permission problems.
ppd referenced this pull request Feb 8, 2020
Snapcraft's remote-build does not fit the requirements of CI,
so replace it with builds running directly on Travis:

1. Builds on Travis can run independently, whereas remote-build
competes for the same resource (Launchpad) and will potentially
block if multiple commits require building in succession.
2. Snapcrafts CLI for remote-build is not designed to be easily
scriptable.
3. Travis recently introduced building on arm64, so builds for
arm64 are now very fast and don't require emulation.

We do not build for armhf and i386 any more because they are
of little relevance on the desktop.
@whitequark whitequark merged commit dcc80de into solvespace:master Feb 8, 2020
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.

None yet

2 participants