Skip Menu |
 

This queue is for tickets about the CPAN CPAN distribution.

Report information
The Basics
Id: 17963
Status: open
Priority: 0/
Queue: CPAN

People
Owner: ANDK [...] cpan.org
Requestors: adamk [...] cpan.org
Cc:
AdminCc:

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



Subject: Bundle::CPAN should refer to readline, not Term::ReadLine::Perl
Download (untitled) / with headers
text/plain 366b
Sigh... The author of Term::ReadLine::Perl does not update the version of Term::ReadLine::Perl when that specific file does not change. He only updates readline.pm, which is installed as Term/ReadLine/readline.pm... It's a bit twisted up, but the only way I can see to make the Bundle work right is to put the dependency on readline.pm, not Term::ReadLine::Perl.
CC: undisclosed-recipients:;
Subject: Re: [rt.cpan.org #17963] Bundle::CPAN should refer to readline, not Term::ReadLine::Perl
Date: Mon, 06 Mar 2006 07:47:26 +0100
To: bug-CPAN [...] rt.cpan.org
From: andreas.koenig.gmwojprw [...] franz.ak.mind.de (Andreas J. Koenig)
Download (untitled) / with headers
text/plain 1.4k
Show quoted text
>>>>> On Fri, 3 Mar 2006 12:06:01 -0500 (EST), " via RT" <bug-CPAN@rt.cpan.org> said:
Show quoted text
> Fri Mar 03 12:06:00 2006: Request 17963 was acted upon. > Transaction: Ticket created by ADAMK > Queue: CPAN > Subject: Bundle::CPAN should refer to readline, not Term::ReadLine::Perl > Owner: Nobody > Requestors: ADAMK@cpan.org > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=17963 >
Show quoted text
> Sigh...
Show quoted text
> The author of Term::ReadLine::Perl does not update the version of > Term::ReadLine::Perl when that specific file does not change.
Show quoted text
> He only updates readline.pm, which is installed as > Term/ReadLine/readline.pm...
Show quoted text
> It's a bit twisted up, but the only way I can see to make the Bundle > work right is to put the dependency on readline.pm, not > Term::ReadLine::Perl.
Which does not work because readline.pm has an irregular package name: while it is installed as Term/ReadLine/readine.pm and can be required as C<require Term::ReadLine::readline>, it provides only the package C<readline>. So the indexer sees only the package C<readline> which cannot be loaded with C<require readline>. I have only the workaround to inject the distro ILYAZ/modules/Term-ReadLine-Perl-1.0207.tar.gz ino the bundle but this would have the disadvantage that the distro gets installed again and again whenever the user says C<install Bundle::CPAN>. Advice welcome. -- andreas
Fixed in 1.87_58 by special-casing readline.pm directly in the inst_file routine
Download (untitled) / with headers
text/plain 322b
Sorry, it was wrong to close this ticket. Indeed we were talking about two bugs: one was the 'package readline' Term/ReadLine/readline.pm. This is solved now. We'll have to wait a few months or years before we can make the requested change to Bundle::CPAN. So re-opened now and kept on hold until 1.88 has wide acceptance.
Keeping "on hold". I see no pressing needs to push this forward. Keeping this ticket oen though.


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.