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: 81935
Status: resolved
Priority: 0/
Queue: CGI

People
Owner: Nobody in particular
Requestors: NEILB [...] cpan.org
Cc: tsibley [...] cpan.org
AdminCc:

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



Subject: distname is CGI.pm rather than CGI
Download (untitled) / with headers
text/plain 176b
The distname being CGI.pm breaks a number of things, and in doing so goes against a convention used by nearly every other dist on CPAN. Fix: change the distname to be 'CGI'.
Subject: Re: [rt.cpan.org #81935] distname is CGI.pm rather than CGI
Date: Thu, 13 Dec 2012 10:38:19 -0500
To: bug-cgi.pm [...] rt.cpan.org
From: Mark Stosberg <mark [...] summersault.com>
Download (untitled) / with headers
text/plain 121b
What does the current name break? Changing the dist name after 10 years is like to break " some things", too. Mark
Download (untitled) / with headers
text/plain 304b
Show quoted text
> What does the current name break?
Various scripts and modules, but in particular, CPAN::DistnameInfo, which is used by a number of smoke testers, and has 42 other CPAN dists dependent on it. There are plenty of dists with distnames that break this, but I'm not sure any are as widely known as CGI.
Download (untitled) / with headers
text/plain 194b
Plus there's the issue with two queues on rt.cpan.org. I reckon the RT team would be happy to do some manual jiggery-pokery with the queues, to sort things out if you did change the distname.
Download (untitled) / with headers
text/plain 517b
CPAN::DistnameInfo has decided that the distname should be 'CGI', even thought the metadata says it's 'CGI.pm'. This means that a number of tools expect the distname to be CGI, and if you honour the metadata, you don't get any joy. For example, MetaCPAN seems to this the distname is CGI rather than 'CGI.pm', which is why a script of mine was pointing to the wrong place. See: https://metacpan.org/release/CGI which gives the 'expected' contents, but https://metacpan.org/release/CGI.pm gives nothing.
Download (untitled) / with headers
text/plain 156b
On Thu Dec 13 18:13:23 2012, NEILB wrote: Show quoted text
> CPAN::DistnameInfo has decided that the distname should be 'CGI'.
That sounds like a bug in CPAN::DistnameInfo.
Download (untitled) / with headers
text/plain 612b
Show quoted text
> That sounds like a bug in CPAN::DistnameInfo.
I've raised it with Graham Barr, but in doing so I noticed that there are other places which assume the distname is 'CGI', for example MetaCPAN. There are 42 dists usings CPAN::DistnameInfo (and maybe MetaCPAN is also using it), so I suspect that changing CGI.pm's distname to 'CGI' (in line with most dists) would probably have less knock-on effect than fixing CPAN::DistnameInfo. And RT seems to have a special case for CGI.pm: other modules that have set their distname to include ".pm" have the .pm dropped from the RT queue name. For example Net::ITE.
Subject: Re: [rt.cpan.org #81935] distname is CGI.pm rather than CGI
Date: Fri, 14 Dec 2012 09:51:26 -0800
To: bug-CGI.pm [...] rt.cpan.org
From: Thomas Sibley <tsibley [...] cpan.org>
Download (untitled) / with headers
text/plain 1.1k
On 12/14/2012 06:34 AM, Neil_Bowers via RT wrote: Show quoted text
> And RT seems to have a special case for CGI.pm: other modules that have set their distname to > include ".pm" have the .pm dropped from the RT queue name. For example Net::ITE.
This is less a special case and more a historical accident. Previous versions of CPAN::DistnameInfo parsed the distname as CGI.pm, so that's what rt.cpan.org used. With version 0.09, however, CPAN::DistnameInfo was patched to return CGI instead [1] and the automatic CPAN → RT importer didn't know it any different from a new dist. I didn't realize the CGI queue existed until recently, although clearly the maintainers have (they added a note to the CGI queue). In the near future when I'm supposed to get some more time to hack on rt.cpan.org, the queues for CGI and CGI.pm will be merged into each other. We'll use what CPAN::DistnameInfo returns as the dist, since that's simplest, although I'll make sure bug-CGI.pm@rt.cpan.org will still work. Are there objections with that plan from the maintainers? Would you rather the CGI.pm queue stays canonical? (assuming CPAN::DistnameInfo isn't changed) [1] https://metacpan.org/source/GBARR/CPAN-DistnameInfo-0.12/Changes
Download (untitled) / with headers
text/plain 176b
The first version of CGI.pm predates CPAN, so this inconsistency is probably to be expected. If no major mayhem is likely to ensue from changing the distname, I'm all for it.
Subject: Re: [rt.cpan.org #81935] distname is CGI.pm rather than CGI
Date: Fri, 14 Dec 2012 15:19:23 -0500
To: bug-cgi.pm [...] rt.cpan.org
From: Mark Stosberg <mark [...] summersault.com>
Download (untitled) / with headers
text/plain 185b
I'm fine with changing the "distname" and merging the bug queues as well. I just wanted to understand what specifically was having problems with the current state of things. Mark
CC: NEILB [...] cpan.org, tsibley [...] cpan.org
Subject: Re: [rt.cpan.org #81935] distname is CGI.pm rather than CGI
Date: Fri, 14 Dec 2012 20:30:36 +0000
To: bug-CGI.pm [...] rt.cpan.org
From: Neil Bowers <neil [...] bowers.com>
Show quoted text
> I'm fine with changing the "distname" and merging the bug queues as well.
Awesome!
Download (untitled) / with headers
text/plain 241b
This issue has been copied to: https://github.com/leejo/CGI.pm/issues/109 please take all future correspondence there. This ticket will remain open but please do not reply here. This ticket will be closed when the github issue is dealt with.
Download (untitled) / with headers
text/plain 490b
Bug queues have already been merged (and migrated to github). And now: commit 52132ae71f44ab4eff9d4e3bdab421fdf70faf94 Author: Lee Johnson <lee@givengain.ch> Date: Wed Aug 13 13:35:52 2014 +0200 resolve #109 [rt.cpan.org #81935] - change distname from CGI.pm to CGI - this will allow various CPAN tools to query the dist without having exceptions for CGI. Makefile.PL | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Will go out within the next week or two. Thanks.
Subject: Re: [rt.cpan.org #81935] distname is CGI.pm rather than CGI
Date: Wed, 13 Aug 2014 05:14:38 -0700
To: Lee Johnson via RT <bug-CGI [...] rt.cpan.org>
From: Mark Stosberg <mark [...] stosberg.com>
Thanks, Lee, for this and all your other CGI.pm maintenance. Mark
Show quoted text
> Will go out within the next week or two. Thanks.
Thank you!


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.