Home GnuPG

Add helper returning the version number of the Gpg4win release
Concern Raised3a04f6a635c9

Description

Add helper returning the version number of the Gpg4win release

The version may contain a product-specific prefix which the new helper
strips off.

Details

Auditors
bernhard
Provenance
ikloeckerAuthored on Oct 20 2021, 1:24 PM
Parents
rLIBKLEO6fca33aa1aa7: Bump library version
Branches
Unknown
Tags
Unknown
Tasks
T5663: Kleopatra's "Check for updates" does not work

Event Timeline

bernhard added a subscriber: bernhard.

@ikloecker I've added the following inline comment above (but I am not sure if it was visible, it still says "unsubmitted", whatever that means)
I've also experimentally pressed "raise concern" hoping it would by inline comment visible. Anyway I've meant to only make a suggestion:

"""
If a version number like "3.1.17-dev1" is used which would be a semver.org (9.) compatible version number, does the regex work fine or is it greedy?

Suggestion
"^[^-]*-" to match first hyphen charakter.
"""

/src/utils/gnupg.cpp
221

If a version number like "3.1.17-dev1" is used which would be a semver.org (9.) compatible version number, does the regex work fine or is it greedy?

Suggestion
"^[^-]*-" to match first hyphen charakter.

This commit now has outstanding concerns.Oct 23 2021, 5:04 PM

(Ah seems I needed to do any comment, before the inline comment was published at all.)

Good point. I have added support for semantic versioning to Kleo::gpg4winVersionNumber().