Skip to content
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

Delmendo remove apt key #1039

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

delmendo
Copy link

What does this implement/fix? Explain your changes.

The Debian instructions use apt-key which has been deprecated. Using the existing directions causes a deprecation warning on every package update. I re-wrote them to use the modern path - this removes the error and installs the pacakge.

Does this close any currently open issues?

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Any other comments?

Update install.sh - remove use of apt-key
(apt-key is deprecated) [https://manpages.debian.org/testing/apt/apt-key.8.en.html#:~:text=DESCRIPTION,keys%20from%20the%20main%20keyring.]

The preferred solution now is to download the key and attach it to the line in the apt sources file.  

This eliminates the following message when you update packages: 

W: https://gitsecret.jfrog.io/artifactory/git-secret-deb/dists/git-secret/InRelease: Key is stored in legacy trusted.gpg keyring (/etc/apt/trusted.gpg), see the DEPRECATION section in apt-key(8) for details.

This follows the new guidelines. (Digital Ocean)[https://www.digitalocean.com/community/tutorials/how-to-handle-apt-key-and-add-apt-repository-deprecation-using-gpg-to-add-external-repositories-on-ubuntu-22-04]

This is based on example from (Helm)[https://helm.sh/docs/intro/install/]
Removing line that was duplicated.
Added in a note to the changelong
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant