Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
README: add snap as an installation option.
- Loading branch information
1 parent
2dcd81d
commit 72bc71c
Showing
1 changed file
with
14 additions
and
0 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
72bc71c
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.
@ppd1990 OK, failing the CI for ~50% of builds is unacceptable. I restarted the build manually here and it failed the second time, building snaps of course. Unless a solution is found I'll have to revert this.
72bc71c
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.
Hm. Very unfortunate. We need a way to ignore previously started remote-builds. I'll investigate the options.
72bc71c
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.
Reading through snapcraft's code, it seems like it wants to either recover or clean a build on launchpad. So no starting parallel builds without killing currently running ones.
Also no cancelling previous builds without user input (or hacky yes n | ...) ; there is no command option to do so. 🤦♂️
72bc71c
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.
Ouch.
72bc71c
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.
We can do builds with docker as mentioned in the issue. At least for amd64 it should be quite simple
72bc71c
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.
Maybe open a feature request?
72bc71c
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.
I will. But I think the "one build at a time" constraint is here to stay and that means we'll always have problems with quick sequential commits or builds for PRs.