[Bug 1119158] Review Request: perl-URL-Encode - Encoding and decoding of application/x-www-form-urlencoded encoding

bugzilla at redhat.com bugzilla at redhat.com
Wed Aug 13 07:56:50 UTC 2014


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



--- Comment #3 from Petr Šabata <psabata at redhat.com> ---
(In reply to Christopher Meng from comment #2)
> (In reply to Petr Šabata from comment #1)
> > FIX: Missing lots of builddeps for the bundled Module::Install -- base,
> > Config, Cwd, ExtUtils::MM_Unix, Fcntl, File::Find, File::Path, Pod::Text,
> > vars
> 
> Question, will this be absorbed by guideline? IMO these are not important or
> should the building environment auto detect these? Since Module::Install is
> often bundled.

The reason for listing those is exactly the same as listing any other.  The
module is used at build time and it has these dependencies.  Bundled modules
(if used for build only) are no special case.

If you don't feel like going through the inc/ directory, one way to deal with
this is removing it, dropping it from MANIFEST and BR'ing inc::Module::Install.
 However, given this may change how your package gets built, you may want to
stick with just checking inc for dependencies, too.  It's not that much work,
really.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component


More information about the package-review mailing list