Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 32769
Status: resolved
Priority: 0/
Queue: CGI

People
Owner: MARKSTOS [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

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



Subject: Garbage in SEE ALSO section in CGI::Carp Pod
There's some garbage in the SEE ALSO section of the CGI::Carp Pod, presumably an copy'n'paste accident. See: http://search.cpan.org/~lds/CGI.pm-3.33/CGI/Carp.pm#SEE_ALSO Regards, Slaven
On Tue Jan 29 08:31:50 2008, SREZIC wrote: Show quoted text
> There's some garbage in the SEE ALSO section of the CGI::Carp Pod, > presumably an copy'n'paste accident. See: > http://search.cpan.org/~lds/CGI.pm-3.33/CGI/Carp.pm#SEE_ALSO >
The "garbage", i.e. the lines if (defined($CGI::Carp::PROGNAME)) { $file = $CGI::Carp::PROGNAME; } are still in the Pod section (seen in CGI.pm-3.34). Regards, Slaven
Thanks, a patch for this has been applied in my git repo now.
01 Ağu. 2009 Cmt., 22:16:22 tarihinde, MARKSTOS yazdı: Show quoted text
> Thanks, a patch for this has been applied in my git repo now.
hmmm... I just checked and it appears to be on github :) can you update Makefile.PL to generate a proper meta file to point to github? SO that the cpan search will display a link to it? I've patched Mojo recently for this, so see this as an example: http://cpansearch.perl.org/src/SRI/Mojo-0.991246/Makefile.PL you'll need the META_MERGE part and a recent EU::MM to pack...
Subject: Re: [rt.cpan.org #32769] PATCH: Garbage in SEE ALSO section in CGI::Carp Pod
Date: Mon, 10 Aug 2009 11:13:46 -0400
To: bug-CGI.pm [...] rt.cpan.org
From: Mark Stosberg <mark [...] summersault.com>
On Sat, 01 Aug 2009 23:41:26 -0400 "Burak Gursoy via RT" <bug-CGI.pm@rt.cpan.org> wrote: Show quoted text
> Queue: CGI.pm > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=32769 > > > 01 Ağu. 2009 Cmt., 22:16:22 tarihinde, MARKSTOS yazdı:
> > Thanks, a patch for this has been applied in my git repo now.
> > hmmm... I just checked and it appears to be on github :) can you update > Makefile.PL to generate a proper meta file to point to github? SO that > the cpan search will display a link to it? I've patched Mojo recently > for this, so see this as an example: > > http://cpansearch.perl.org/src/SRI/Mojo-0.991246/Makefile.PL > > you'll need the META_MERGE part and a recent EU::MM to pack...
Thanks for the suggestion and example. We will do this once the github location is finalized. Lincoln Stein is planning to conver the full CVS history to git and github, and that version will become the official version when it is done, and may be possibly be hosted through a different URI on github. Mark -- . . . . . . . . . . . . . . . . . . . . . . . . . . . Mark Stosberg Principal Developer mark@summersault.com Summersault, LLC 765-939-9301 ext 202 database driven websites . . . . . http://www.summersault.com/ . . . . . . . .
Show quoted text
> > > > http://cpansearch.perl.org/src/SRI/Mojo-0.991246/Makefile.PL > > > > you'll need the META_MERGE part and a recent EU::MM to pack...
OK, I've now updated our Makefile.PL based on this example. The official repository URL, but we have something in there as an placeholder now at least. Mark
Show quoted text
> > > > http://cpansearch.perl.org/src/SRI/Mojo-0.991246/Makefile.PL > > > > you'll need the META_MERGE part and a recent EU::MM to pack...
OK, I've now updated our Makefile.PL based on this example. The official repository URL, but we have something in there as an placeholder now at least. Mark
Subject: released, thanks.
I believe this change was released today as part of CGI.pm 3.45. Thanks for the contribution.