-
-
Notifications
You must be signed in to change notification settings - Fork 344
docs(readme): Update deb instructions to use keyring #332
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
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.
Thanks, looks mostly fine, added a two change requests, also could you also test this on Ubuntu 22.04 and Debian 11 as well? :)
trusted.gpg.d requires de-armored keys on some systems (Ubuntu 20.04) and ASCII-armored keys on others (Debian 12)
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
Signed-off-by: Sandro-Alessio Gierens <[email protected]>
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 changed to chmod 644 to make the command more general and added possible gpg installation instructions as some systems don't ship it by default (Debian 11/12) cloud images for example.
@cafkafk I've tested this on Ubuntu 20, 22 and Debian 11, 12 VMs. Works! Approved! Ready for merge!
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.
@cafkafk I've tested this on Ubuntu 20, 22 and Debian 11, 12 VMs. Works! Approved! Ready for merge!
Okie, merging
trusted.gpg.d requires de-armored keys on some systems (Ubuntu 20.04) and ASCII-armored keys on others (Debian 12)
this keyring method works on both systems. I tested in an ubuntu:20.04 container and a debian:12 container
it also uses
dd of=instead ofteeto avoid dumping binary to the terminalfixes #259