-
Notifications
You must be signed in to change notification settings - Fork 79
chore: add back support for generating signable binaries #339
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
Conversation
Use `boxednode` to create binaries that are compiled in a way that should enable us to sign them. `pkg` is retained as a helper for local development, since it requires no native compilation.
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'd just remove the non signable one if we don't use it
d1a690c
to
9ad21ab
Compare
display_name: "RHEL 8.0" | ||
run_on: rhel80-large | ||
display_name: "RHEL 8.0 (in Ubuntu docker)" | ||
run_on: ubuntu1804-test |
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.
ubuntu1804-test
has docker installed?
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.
Apparently 🤷♀️
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.
Excited for this to land!
Use
boxednode
to create binaries that are compiled in a waythat should enable us to sign them.
pkg
is retained as ahelper for local development, since it requires no native compilation.
Fixes MONGOSH-226