Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 497348 - dev-perl/Config-Tiny-2.160.0 lacks dependency on >=virtual/perl-Module-Build-0.380.0
Summary: dev-perl/Config-Tiny-2.160.0 lacks dependency on >=virtual/perl-Module-Build-...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Perl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-07 03:31 UTC by Andrew Savchenko
Modified: 2014-12-22 19:02 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info (emerge.info,9.00 KB, text/plain)
2014-01-07 03:31 UTC, Andrew Savchenko
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Savchenko gentoo-dev 2014-01-07 03:31:11 UTC
Hi,

QA: configure
QA Notice: The ebuild uses Module::Build but doesn't depend on it.
           Add virtual/perl-Module-Build to DEPEND
Comment 1 Andrew Savchenko gentoo-dev 2014-01-07 03:31:56 UTC
Created attachment 367250 [details]
emerge --info
Comment 2 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2014-01-07 13:10:36 UTC
The upstream module in question has dual-build, so its not a show-stopper, however, our eclass situation is such that gentoo will:

1. In the presence of a Build.PL file, always used Module::Build style invocation, regardless of truth. ( Potentially exploding if Module::Build is not installed )

2. In the scenario where Build.PL is _Not_ Module::Build, erroneously complain about a dependency on Module::Build, which is not neccessary

3. Also entirely fail to QA check on the appropriate dependency when Build.PL requires some other things.

Essentially, the QA check we have in place is a bit wonky, and the mechanism that triggers it is wonky and unreliable too.

I've a few ideas on how we should solve this, but its lengthy and I've thrown them here: https://gist.github.com/kentfredric/8299018
Comment 3 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2014-10-11 11:04:27 UTC
A short term redux for @others, because I keep hitting the QA warning:

1. perl-module.eclass currently uses Build.PL if one exists, regardless of Module::Build being installed.

2. Thus, if there is a Build.PL and Module::Build is not installed, breakage will happen.

3. Therefore this module must depend on virtual/perl-Module-Build

4. Additionally, upstream declares a mandatory minimum of 0.380.0 

https://metacpan.org/source/RSAVAGE/Config-Tiny-2.16/META.json#L24 

5. Therefore this must >=virtual/perl-Module-Build-0.380.0


This is especially relevant now that Build.PL can sometimes imply Module::Build::Tiny instead of Module::Build, or because older Module::Builds could theoretically be installed still. 

The rest of my comments are more metadiscussion about longer term strategies.

But solving this bug directly is much more straight forward.
Comment 4 Andreas K. Hüttel archtester gentoo-dev 2014-10-11 11:17:25 UTC
Fixed.