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)

History Show all quoted text
  Wed Dec 12 18:34:19 2012 NEILB [...] cpan.org - Ticket created
Subject: distname is CGI.pm rather than CGI
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'.
  Thu Dec 13 10:38:36 2012 mark [...] summersault.com - Correspondence added
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>
What does the current name break? Changing the dist name after 10 years is like to break " some things", too. Mark
  Thu Dec 13 10:38:37 2012 The RT System itself - Status changed from 'new' to 'open'
  Thu Dec 13 17:28:02 2012 NEILB [...] cpan.org - Correspondence added
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.
  Thu Dec 13 17:29:32 2012 NEILB [...] cpan.org - Correspondence added
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.
  Thu Dec 13 18:13:23 2012 NEILB [...] cpan.org - Correspondence added
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.
  Thu Dec 13 22:12:15 2012 tsibley [...] cpan.org - Cc TSIBLEY added
  Fri Dec 14 09:28:11 2012 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added
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.
  Fri Dec 14 09:34:56 2012 NEILB [...] cpan.org - Correspondence added
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.
  Fri Dec 14 12:52:05 2012 tsibley [...] cpan.org - Correspondence added
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>
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
  Fri Dec 14 12:58:16 2012 LDS [...] cpan.org - Correspondence added
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.
  Fri Dec 14 15:19:37 2012 mark [...] summersault.com - Correspondence added
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>
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
  Fri Dec 14 15:30:48 2012 neil [...] bowers.com - Correspondence added
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!
  Thu May 22 08:16:41 2014 LEEJO [...] cpan.org - Correspondence added
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.
  Fri May 23 14:28:11 2014 The RT System itself - Queue changed from CGI.pm to CGI
  Wed Aug 13 07:41:54 2014 LEEJO [...] cpan.org - Correspondence added
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.
  Wed Aug 13 07:41:55 2014 LEEJO [...] cpan.org - Status changed from 'open' to 'resolved'
  Wed Aug 13 08:14:48 2014 mark [...] stosberg.com - Correspondence added
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
  Wed Aug 13 08:16:18 2014 NEILB [...] cpan.org - Correspondence added
Show quoted text
> Will go out within the next week or two. Thanks.
Thank you!