[Bug 168227] Review request: perl-HTTP-Server-Simple-Mason - HTTP::Server::Simple::Mason Perl module

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review request: perl-HTTP-Server-Simple-Mason - HTTP::Server::Simple::Mason Perl module


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=168227





------- Additional Comments From jpo at di.uminho.pt  2005-09-29 15:08 EST -------
(In reply to comment #2)
> (In reply to comment #1)
> > * missing requirement: perl(HTTP::Server::Simple::CGI)
> 
> # rpm -q --whatprovides 'perl(HTTP::Server::Simple::CGI)'
> perl-HTTP-Server-Simple-0.13-1.fc4

Hmm... I am not following. I am talking about a missing requirement not
about the BR.

$ rpm -qpR ../RPMS/noarch/perl-HTTP-Server-Simple-Mason-0.09-1.noarch.rpm | grep
^perl
perl(:MODULE_COMPAT_5.8.6)
perl(Encode)
perl(HTML::Mason::CGIHandler)
perl(HTML::Mason::FakeApache)
perl(Hook::LexWrap)
perl(base)
perl(bytes)
perl(strict)

The rpmbuild perl.req script fails to require perl(HTTP::Server::Simple::CGI).
As you mentioned, this module is provided by perl-HTTP-Server-Simple, which
should be explicitly required.


> > * directory ownership should start at
> > 
> >     %{perl_vendorlib}/HTTP/Server/Simple/
> > 
> >   as it requires HTTP::Server::Simple
> 
> Well, I'm hestant to apply this proposal, because
> 1. RPM doesn't guarantee any particular order when removing multiple packages.
> Therefore, this convention causes empty directories to stay around when removing
> several packages at once.
> 
> 2. perl(HTTP::Server::Simple) doesn't correspond to
> %{perl_vendorlib}/HTTP/Server/Simple,
> it corresponds to
> %{perl_vendorlib}/HTTP/Server/Simple.pm
> 
> I.e. to ensure the directory you'd have to 
> Requires: %{perl_vendorlib}/HTTP/Server/Simple
> 
> (Or even Requires(pre) rsp. Requires(preun), as this seems to help rpm upon
> multiple package removal requests)

You have a point about the directory ownership.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.