Skip Menu |

This queue is for tickets about the GnuPG-Interface CPAN distribution.

Report information
The Basics
Id: 57406
Status: resolved
Priority: 0/
Queue: GnuPG-Interface

People
Owner: Nobody in particular
Requestors: dkg [...] fifthhorseman.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in:
  • 0.42_02
  • 0.43
  • 0.44
  • 0.45
  • 0.46



Subject: GnuPG::Key->hex_data non-functional -- proposing GnuPG::Key->pubkey_data instead
Date: Wed, 12 May 2010 00:03:30 -0400
To: bug-GnuPG-Interface [...] rt.cpan.org
From: Daniel Kahn Gillmor <dkg [...] fifthhorseman.net>
Looking at the source code for GnuPG::Interface, i see nowhere that GnuPG::Key->hex_data ever gets set. I'm not sure how that was ever supposed to have been used. Rather than try to implement what appears to be a poorly specified member, I propose to add a new member of GnuPG::Key named pubkey_data, which contains a reference to a list of Math::BigInt objects. (the objects differ based on the algorithm of the key in question). I've added this to the head of my git repository here: git://lair.fifthhorseman.net/~dkg/gnupg-interface I welcome feedback on it. --dkg
Download signature.asc
application/pgp-signature 892b

Message body not shown because it is not plain text.

On 2010-05-12T00:03:53-04:00, dkg@fifthhorseman.net wrote: Show quoted text
> Looking at the source code for GnuPG::Interface, i see nowhere that > GnuPG::Key->hex_data ever gets set. I'm not sure how that was ever > supposed to have been used.
I agree. Unfortunately, history diving doesn't prove terribly useful in this regard either. Show quoted text
> Rather than try to implement what appears to be a poorly specified > member, I propose to add a new member of GnuPG::Key named pubkey_data, > which contains a reference to a list of Math::BigInt objects. (the > objects differ based on the algorithm of the key in question). > > I've added this to the head of my git repository here: > > git://lair.fifthhorseman.net/~dkg/gnupg-interface
I only see two commits on the 'master' branch in that repo: 618a63bc87977341890e9a0a9487bbb0d6a058eb 9e29a91c18e46d3afb0f55820eb0215d95660129 ..both of which look to be typo fixes. Did the commits in question not get pushed? - Alex
On Sat Oct 27 16:27:29 2012, ALEXMV wrote: Show quoted text
> On 2010-05-12T00:03:53-04:00, dkg@fifthhorseman.net wrote:
> > I've added this to the head of my git repository here: > > > > git://lair.fifthhorseman.net/~dkg/gnupg-interface
> > I only see two commits on the 'master' branch in that repo: > 618a63bc87977341890e9a0a9487bbb0d6a058eb > 9e29a91c18e46d3afb0f55820eb0215d95660129 > ..both of which look to be typo fixes. Did the commits in question not > get pushed?
wow, this is far enough in the past that i can't find these commits any more. I'm not sure where they are (or if i indeed ever created them, though the proposal still seems reasonable to me). sorry to not have them handy.
On Thu Sep 05 14:46:58 2013, DKG wrote: Show quoted text
> wow, this is far enough in the past that i can't find these commits > any more. I'm not sure where they are (or if i indeed ever created > them, though the proposal still seems reasonable to me). sorry to not > have them handy.
aha. they're already included upsteam, as of e33e2d54b , which was folded into 0.42_02