[pacman-dev] [PATCH] Add compatibility with gpgme-1.5.0
Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/signing.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index caacd24..b14945b 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -342,6 +342,7 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH: + case GPGME_PK_ECC: pgpkey->pubkey_algo = 'E'; break; } -- 2.0.0
On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae <allan@archlinux.org> wrote:
Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/signing.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index caacd24..b14945b 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -342,6 +342,7 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH:
#ifdef GPGME_PK_ECC
+ case GPGME_PK_ECC:
#endif Worth it for backward compat?
pgpkey->pubkey_algo = 'E'; break; } -- 2.0.0
On 02/06/14 09:56, Dan McGee wrote:
On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae <allan@archlinux.org> wrote:
Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/signing.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index caacd24..b14945b 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -342,6 +342,7 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH:
#ifdef GPGME_PK_ECC
+ case GPGME_PK_ECC:
#endif
Worth it for backward compat?
Of course! I don't think about people suffering without the latest versions of software...
pgpkey->pubkey_algo = 'E'; break; } -- 2.0.0
On Sun, Jun 01, 2014 at 06:56:01PM -0500, Dan McGee wrote:
On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae <allan@archlinux.org> wrote:
Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/signing.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index caacd24..b14945b 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -342,6 +342,7 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH:
#ifdef GPGME_PK_ECC
+ case GPGME_PK_ECC:
#endif
Worth it for backward compat?
This doesn't work, since GPGME_PK_ECC is part of an enum, and not a preprocessor definition.
pgpkey->pubkey_algo = 'E'; break; } -- 2.0.0
On 02/06/14 10:13, Dave Reisner wrote:
On Sun, Jun 01, 2014 at 06:56:01PM -0500, Dan McGee wrote:
On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae <allan@archlinux.org> wrote:
Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/signing.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index caacd24..b14945b 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -342,6 +342,7 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH:
#ifdef GPGME_PK_ECC
+ case GPGME_PK_ECC:
#endif
Worth it for backward compat?
This doesn't work, since GPGME_PK_ECC is part of an enum, and not a preprocessor definition.
Yeah - it requires autoconf magic using gpgme_check_version. I will post a working patch later. A
On 02/06/14 10:27, Allan McRae wrote:
On 02/06/14 10:13, Dave Reisner wrote:
On Sun, Jun 01, 2014 at 06:56:01PM -0500, Dan McGee wrote:
On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae <allan@archlinux.org> wrote:
Signed-off-by: Allan McRae <allan@archlinux.org> --- lib/libalpm/signing.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/lib/libalpm/signing.c b/lib/libalpm/signing.c index caacd24..b14945b 100644 --- a/lib/libalpm/signing.c +++ b/lib/libalpm/signing.c @@ -342,6 +342,7 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH:
#ifdef GPGME_PK_ECC
+ case GPGME_PK_ECC:
#endif
Worth it for backward compat?
This doesn't work, since GPGME_PK_ECC is part of an enum, and not a preprocessor definition.
Yeah - it requires autoconf magic using gpgme_check_version. I will post a working patch later.
Or I could read the header... Here is the updated patch: @@ -342,6 +342,10 @@ static int key_search(alpm_handle_t *handle, const char *fpr, case GPGME_PK_ELG: case GPGME_PK_ECDSA: case GPGME_PK_ECDH: +/* value added in gpgme 1.5.0 */ +#if GPGME_VERSION_NUMBER >= 0x010500 + case GPGME_PK_ECC: +#endif pgpkey->pubkey_algo = 'E'; break; }
participants (3)
-
Allan McRae
-
Dan McGee
-
Dave Reisner