Foxboron / sbctl

:computer: :lock: :key: Secure Boot key manager
MIT License
1.35k stars 71 forks source link

Add Option ROM warning to Usage section of the manpage #300

Closed 00-kat closed 2 months ago

00-kat commented 2 months ago

The text in the usage section did not mention that some firmware might need Microsoft certificates as well, so I copied pasted it from line 48.

I noticed on a Reddit comment though that there's a warning if option ROM is detected, so this warning might be unnecessary. However, I also found issue #212, so I'm not sure if option ROM will always be reliably detected. Feel free to reject this PR if there is already a reliable warning when you have option ROM (I don't so I can't test).

I also did not include the -m flag by default in the sbctl enroll-keys, since I was not sure how widespread OpROMs are.

Some $s were also changed to #s to indicate that they should be run as root (didn't want to make a separate PR for changing 7 characters). I'm not familiar with asciidoc, so hopefully #s don't have special meaning.