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.

Sign up for GitHub

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

Jump to bottom

fixed expire parameter #396

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed expire parameter #396

wants to merge 1 commit into from

Conversation

blurks
Copy link

@blurks blurks commented Jan 23, 2022

using gpg 2.2.32 I get the following error using this configuration:

$ gpg --batch --full-gen-key ./params.txt
gpg: ./params.txt:8: unknown keyword

The correct parameter to set the expiration date would be Expire-Date, not Expires. cf. Unattended key generation.

@blurks
fixed expire parameter
bfc5283 
using gpg 2.2.32 I get the following error using this configuration:

```
$ gpg --batch --full-gen-key ./params.txt
gpg: ./params.txt:8: unknown keyword
```

The correct parameter to set the expiration date would be `Expire-Date`, not `Expires`.  cf. [Unattended key generation](https://www.gnupg.org/documentation/manuals/gnupg/Unattended-GPG-key-generation.html).
sebix
sebix approved these changes Jan 23, 2022
View reviewed changes
Copy link
Member

@sebix sebix left a comment

Choose a reason for hiding this comment

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

Although the repo's activity has stalled and the project is more or less dead, that change looks OK :)

@blurks
Copy link
Author

blurks commented Jan 23, 2022

Thanks :)
I've been using the website for quite some time and only now noticed that the project isn't really active anymore. Maybe one should also put a disclaimer similar to the one in the README on the website.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebix sebix sebix approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@blurks @sebix