Skip Menu |
 

This queue is for tickets about the Crypt-X509 CPAN distribution.

Report information
The Basics
Id: 66538
Status: open
Priority: 0/
Queue: Crypt-X509

People
Owner: alexander.w.jung [...] gmail.com
Requestors: dkg [...] fifthhorseman.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.51



Subject: Crypt::X509 should not export error(), new(), not_before(), not_after(), or serial()
Date: Fri, 11 Mar 2011 01:18:31 -0500
To: bug-Crypt-X509 [...] rt.cpan.org
From: Daniel Kahn Gillmor <dkg [...] fifthhorseman.net>
Download (untitled) / with headers
text/plain 801b
The attached simple demonstration program produces a spurious warning because of subroutines needlessly exported by Crypt::X509: Show quoted text
> 0 dkg@pip:~$ ./demo-new.pl > Subroutine new redefined at ./demo-new.pl line 8. > 0 dkg@pip:~$
The documentation makes no claims that it will export any subroutines by default, but X509.pm contains: our @EXPORT = qw(error new not_before not_after serial); I believe these should all be removed from @EXPORT. Is there reason to believe these symbols are expected to be exported? It would be good to be able to do things like: use Crypt::X509; in other modules that also define subs named new() or error() without seeing these warnings. Does this make sense? Or is there some reason i'm missing for these items are exported and the others are not? --dkg
Download demo-new.pl
text/x-perl 93b

Message body is not shown because sender requested not to inline it.

Download signature.asc
application/pgp-signature 1k

Message body not shown because it is not plain text.

Download (untitled) / with headers
text/plain 202b
You're right. The export of new() and error() was a missunderstanding of perls mechanisms on my side. I'll put up a new version with the exports fixed. yours, Alexander (back from parental leave....)
put Crypt-X509 0.51 on CPAN Yours, Alexander
Subject: Re: [rt.cpan.org #66538] Crypt::X509 should not export error(), new(), not_before(), not_after(), or serial()
Date: Wed, 06 Jul 2011 10:29:56 -0400
To: bug-Crypt-X509 [...] rt.cpan.org
From: Daniel Kahn Gillmor <dkg [...] fifthhorseman.net>
Download (untitled) / with headers
text/plain 377b
On 07/06/2011 04:03 AM, Alexander Jung via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=66538 > > > You're right. The export of new() and error() was a missunderstanding of > perls mechanisms on my side. > > I'll put up a new version with the exports fixed.
Thank you very much, Alexander. Show quoted text
> (back from parental leave....)
Welcome back! Regards, --dkg
Download signature.asc
application/pgp-signature 1k

Message body not shown because it is not plain text.

this is fixed in 0.51


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.