Skip Menu |

This queue is for tickets about the CPAN CPAN distribution.

Report information
The Basics
Id: 32923
Status: resolved
Priority: 0/
Queue: CPAN

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

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



Subject: Allow Bundle "CONTENTS" Header Case-Insensitivity
Date: Mon, 4 Feb 2008 13:55:02 -0800
To: bug-CPAN [...] rt.cpan.org
From: "David E. Wheeler" <dwheeler [...] cpan.org>
I tend to believe that any case should be allowed in `=head` POD headers, and POD formatters should convert them to uppercase as appropriate. CPAN.pm, however, currently depends on the `head1 CONTENTS` header in bundles being uppercase. This patch allows the word "contents" to be matched case-insensitively. Thanks, David --- lib/CPAN.pm.orig 2008-02-04 13:52:13.000000000 -0800 +++ lib/CPAN.pm 2008-02-04 13:52:30.000000000 -0800 @@ -9489,8 +9489,8 @@ my $in_cont = 0; $self->debug("inst_file[$inst_file]") if $CPAN::DEBUG; while (<$fh>) { - $in_cont = m/^=(?!head1\s+CONTENTS)/ ? 0 : - m/^=head1\s+CONTENTS/ ? 1 : $in_cont; + $in_cont = m/^=(?!head1\s+?(i-xsm:CONTENTS)/ ? 0 : + m/^=head1\s+?(i-xsm:CONTENTS)/ ? 1 : $in_cont; next unless $in_cont; next if /^=/; s/\#.*//;
Subject: Re: [rt.cpan.org #32923] Allow Bundle "CONTENTS" Header Case-Insensitivity
Date: Tue, 05 Feb 2008 03:57:16 +0100
To: bug-CPAN [...] rt.cpan.org
From: andreas.koenig.7os6VVqR [...] franz.ak.mind.de (Andreas J. Koenig)
Show quoted text
>>>>> On Mon, 04 Feb 2008 16:59:08 -0500, "David Wheeler via RT" <bug-CPAN@rt.cpan.org> said:
Show quoted text
> CPAN.pm, however, currently depends on the `head1 > CONTENTS` header in bundles being uppercase. This patch allows the > word "contents" to be matched case-insensitively.
Thanks applied to my repository (with tweaks). https://pause.perl.org:5460/svn/cpanpm/trunk -- andreas
Subject: Re: [rt.cpan.org #32923] Allow Bundle "CONTENTS" Header Case-Insensitivity
Date: Mon, 4 Feb 2008 20:03:19 -0800
To: bug-CPAN [...] rt.cpan.org
From: "David E. Wheeler" <dwheeler [...] cpan.org>
On Feb 4, 2008, at 18:57, (Andreas J. Koenig) via RT wrote: Show quoted text
>> CPAN.pm, however, currently depends on the `head1 >> CONTENTS` header in bundles being uppercase. This patch allows the >> word "contents" to be matched case-insensitively.
> > Thanks applied to my repository (with tweaks). > > https://pause.perl.org:5460/svn/cpanpm/trunk
Awesome, thanks Andreas. David