-
-
Notifications
You must be signed in to change notification settings - Fork 153
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
Updated manpage and help file #481
Conversation
@@ -786,7 +786,7 @@ usage() { | |||
echo | |||
[[ $QRENCODE == 1 ]] && { | |||
_print " // Backup on paper:" | |||
_print " engrave makes a QR code of a KEY to be saved on paper" | |||
_print " engrave makes a QR code of a KEY to be saved on paper" |
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.
Are you sure that this is an intended change?
Or generally asked: Why is the spacing needed?
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.
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.
Ah, I see. Missed that, because I don't have that dep installed and didn't notice it was the help text.
Just saw the diff and not the surroundings :D
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.
Or generally asked: Why is the spacing needed?
Ok, is not NEEDED, it's just aesthetics.
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.
PR reminds me that at some point we should generate the PDF file via github actions rather than committing a binary into this repo. But OK for now.
(2) Created a README file to clear up some confused issues about versioning; (3) Updated 'es', 'fr', 'it' and 'ru.po' to line up help message (PR dyne#481 and Jaromil request in PR dyne#483); (4) Added Brazilian Portuguese translation (for Tomb 2.3 and 2.10); (5) Created new tomb.pot template based upon Tomb 2.10 (current);
I just run:
In the other commit i have updated help messages:
From this:
To this: