Opened 16 years ago

Closed 15 years ago

#17785 closed defect (fixed)

p5-pod-readme should not depend on p5-podlators

Reported by: vinc17@… Owned by: narf_tm@…
Priority: Normal Milestone:
Component: ports Version: 1.7.0
Keywords: Cc:
Port: p5-pod-readme

Description

p5-pod-readme cannot be installed since it depends on p5-podlators, which is itself not installable, because the files are already provided by the perl5.8 port. As a consequence this dependency is also unnecessary, and indeed, p5-pod-readme can be built even though p5-podlators is not installed.

Change History (4)

comment:1 Changed 16 years ago by narf_tm@…

Resolution: fixed
Status: newclosed

Pod::Readme 0.09 requires Pod::Text 3.0. Perl 5.8 has a much older version of Pod::Text. I changed the p5-podlators port so it will overwrite the Perl modules. Fixed in changeset [44413].

comment:2 Changed 16 years ago by vinc17@…

Resolution: fixed
Status: closedreopened

No, Pod::Readme 0.09 does not require Pod::Text. No errors at build time, and no references in the module:

$ grep Pod:: /opt/local/lib/perl5/vendor_perl/5.8.8/Pod/Readme.pm
Pod::Readme - Convert POD to README file
  Pod::PlainText
  use Pod::Readme;
  my $parser = Pod::Readme->new();
package Pod::Readme;
use Pod::PlainText;
@ISA = qw( Pod::PlainText );
# This code is based on code from Pod::PlainText 2.02
    # enhancement over the original Pod::Text.
This module is a subclass of L<Pod::PlainText> which provides additional
  $parser = Pod::Readme->new( readme_type => "copying" );
by other POD processors (such as "testing" or "html").  L<Pod::Readme> will
  $parser = Pod::Readme->new( debug => 1 );
Some portions are based on L<Pod::PlainText> 2.02.

and no error with pod2readme.

Also ports that overwrite the Perl modules are broken.

comment:3 Changed 16 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

comment:4 in reply to:  2 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: reopenedclosed

Replying to vinc17@…:

No, Pod::Readme 0.09 does not require Pod::Text.

I removed the dependency in r55831.

Note: See TracTickets for help on using tickets.