-
Notifications
You must be signed in to change notification settings - Fork 224
Add deb/rpm package building and publishing #80
Conversation
Basically copied segmentio/chamber#129
4a8bb04
to
abdec99
Compare
abdec99
to
5198001
Compare
Ubuntu install test
|
Fedora install test
|
- persist_to_workspace: | ||
root: . | ||
paths: ['dist/*'] | ||
|
||
release: | ||
publish-linux: |
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.
is there a reason we're not also publish darwin releases?
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.
Yeah, I should add a comment. Basically because the darwin build depends on Keychain libs that are not freely distributable, so we need a real macOS box to build this on, which is a whole other thing :)
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.
#81 for later
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.
Actually, I'm not sure this is correct. Chamber uses Keychain as well, and yet it builds fine... Hmmm
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.
Nvm, Chamber doesn't use keychain.
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.
similarly to chamber pr, lgtm!
After merging this, I cut a new release v0.19.4 and it appears to have built and published everything correctly https://circleci.com/workflow-run/124d882e-cb15-4425-b639-ce564f663694 |
Basically copied segmentio/chamber#129
Fixes #79