Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
snap: Build snaps on Travis without remote-build & drop i386 + armhf
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.
- Loading branch information
1 parent
3ed1719
commit 65ab595
Showing
3 changed files
with
12 additions
and
19 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,15 +1,4 @@ | ||
#!/bin/sh -xe | ||
|
||
lp_data_dir="$HOME/.local/share/snapcraft/provider/launchpad" | ||
lp_credentials="$lp_data_dir/credentials" | ||
|
||
mkdir -p "$lp_data_dir" | ||
openssl aes-256-cbc -K $encrypted_c4bc81f026a2_key -iv $encrypted_c4bc81f026a2_iv \ | ||
-in .travis/launchpad-credentials.enc \ | ||
-out "$lp_credentials" -d | ||
chmod 600 "$lp_credentials" | ||
|
||
./pkg/snap/build.sh remote-build \ | ||
--launchpad-user solvespace \ | ||
--launchpad-accept-public-upload \ | ||
--build-on=amd64,arm64,armhf,i386 | ||
sudo apt-get update | ||
sudo ./pkg/snap/build.sh --destructive-mode |
Binary file not shown.
65ab595
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@whitequark Maybe we should remove the dropped architectures from the store to avoid confusion: https://forum.snapcraft.io/t/is-it-possible-to-drop-an-unsupported-architecture-from-the-store/10510/2
65ab595
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And deployment failed. We have to run snapcraft with sudo, now that we run the build in destructive mode...
65ab595
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closed the channel, the next push should fix this.
65ab595
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#548 should fix the last brain fart