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

Fixes + cleanups to rpmkeys manual #3433

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

pmatilai
Copy link
Member

@pmatilai pmatilai commented Nov 7, 2024

Drop all references to pre-historical Red Hat keys, short key-ids and gpg-pubkey packages. Use the term fingerprint consistently everywhere. Add a brief explanation of the --list format and usage example for --export too.

@pmatilai pmatilai requested a review from a team as a code owner November 7, 2024 11:59
@pmatilai pmatilai requested review from dmnks and removed request for a team November 7, 2024 11:59
Copy link
Contributor

@nwalfield nwalfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only a few small nits. Feel free to ignore.

docs/man/rpmkeys.8.md Outdated Show resolved Hide resolved
docs/man/rpmkeys.8.md Outdated Show resolved Hide resolved
docs/man/rpmkeys.8.md Outdated Show resolved Hide resolved
docs/man/rpmkeys.8.md Outdated Show resolved Hide resolved
@pmatilai
Copy link
Member Author

pmatilai commented Nov 7, 2024

Thanks for the suggestions @nwalfield , should be sorted now. I also found yet more fubar about the rpm database and headers in there, removed.


Finally, public keys can be erased after importing just like packages.
Here\'s how to remove the Red Hat GPG/DSA key:
Write the key(s) in an ASCII-armor encoding.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps: s/in/using/

Copy link
Member Author

@pmatilai pmatilai Nov 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, "write" seemed lonely now that the context changed, so I replaced it with 'Output'. And, there was another use of 'This blahblah', fixed now 😄

Copy link
Contributor

@dmnks dmnks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks Neal for looking, too! I've just fixed up a lonely opening square bracket before the --delete argument (which is actually mandatory).

@dmnks
Copy link
Contributor

dmnks commented Nov 8, 2024

One more typo fixup 😄

Drop all references to pre-historical Red Hat keys, short key-ids
and gpg-pubkey packages. Use the term fingerprint consistently everywhere.
Add a brief explanation of the --list format and usage example for
--export too.
@dmnks
Copy link
Contributor

dmnks commented Nov 8, 2024

I've autosquashed the fixup commit on top and will merge this once the CI finishes. This is the fixup patch I've applied, just for full transparency:

diff --git a/docs/man/rpmkeys.8.md b/docs/man/rpmkeys.8.md
index e7993e52c..093a91606 100644
--- a/docs/man/rpmkeys.8.md
+++ b/docs/man/rpmkeys.8.md
@@ -25,7 +25,7 @@ The general forms of rpm digital signature commands are
 
 **rpmkeys** **\--import** *PUBKEY \...*
 
-**rpmkeys** **\--delete** \[*FINGERPRINT \...*
+**rpmkeys** **\--delete** *FINGERPRINT \...*
 
 **rpmkeys** {**-K\|\--checksig**} *PACKAGE\_FILE \...*
 
@@ -53,7 +53,7 @@ The fingerprint is the handle used for all operations on the keys.
 
 Output the key(s) using an ASCII-armor encoding.
 
-Exporting allows inspectecting the data with specialized tools, such
+Exporting allows for inspecting the data with specialized tools, such
 as Sequoia or GnuPG. For example:
 
 **rpmkeys --export 771b18d3d7baa28734333c424344591e1964c5fc | sq inspect **

@dmnks dmnks merged commit bd907b0 into rpm-software-management:master Nov 8, 2024
1 check passed
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.

3 participants