On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae wrote:

Signed-off-by: Allan McRae --- 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 Sun, Jun 01, 2014 at 06:56:01PM -0500, Dan McGee wrote:

On Sun, Jun 1, 2014 at 6:44 PM, Allan McRae wrote:

...

Signed-off-by: Allan McRae --- 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: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 wrote:

...

Signed-off-by: Allan McRae --- 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; }