Bug 1091285 - Should include dependency on perl(Test::Pod::Content)
Summary: Should include dependency on perl(Test::Pod::Content)
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-Test-ConsistentVersion
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Emmanuel Seyman
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-25 09:50 UTC by Paul Howarth
Modified: 2014-04-27 15:03 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-04-27 15:03:41 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Paul Howarth 2014-04-25 09:50:50 UTC
It's optional but we generally include optional dependencies, at least until such time as soft dependencies can be used.

Without it, I'm seeing this in MouseX::SimpleConfig's test suite:

$ make test RELEASE_TESTING=1
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
t/00-compile.t .................. skipped: no tests to run
t/01use.t ....................... ok
t/10simple.t .................... ok
t/11default.t ................... ok
t/12config_any_args.t ........... ok
t/author-critic.t ............... skipped: these tests are for testing by the author
t/multiple.t .................... ok
t/release-check-changes.t ....... ok
# Test::Pod::Content required to test POD version consistency
# Distribution version: 0.11
t/release-consistent-version.t .. ok
# Unable to parse MANIFEST.SKIP file:
# No such file or directory
# Using default skip data from ExtUtils::Manifest 1.63
t/release-dist-manifest.t ....... ok
t/release-distmeta.t ............ ok
t/release-eol.t ................. ok
t/release-has-version.t ......... ok
t/release-kwalitee.t ............ ok
t/release-minimum-version.t ..... ok
t/release-no-tabs.t ............. ok
t/release-pod-coverage.t ........ ok
t/release-pod-syntax.t .......... ok
t/release-portability.t ......... ok
t/release-unused-vars.t ......... ok
All tests successful.

Comment 1 Emmanuel Seyman 2014-04-25 10:23:30 UTC
(In reply to Paul Howarth from comment #0)
> It's optional but we generally include optional dependencies, at least until
> such time as soft dependencies can be used.

Agreed. I'll do this over the weekend.

Comment 2 Paul Howarth 2014-04-25 10:29:49 UTC
Would you be interested in building this and its deps for EPEL-7? If not, I'll be happy to take it.

Comment 3 Emmanuel Seyman 2014-04-25 11:15:24 UTC
I don't use RHEL so I'ld rather someone else handle it.

Comment 4 Paul Howarth 2014-04-25 14:14:50 UTC
OK, I've requested EPEL7 branches on the wiki. When they're done, you can transfer ownership to me:

$ pkgdb-cli unorphan --owner pghmcfc perl-Test-Pod-Content epel7
$ pkgdb-cli unorphan --owner pghmcfc perl-Test-ConsistentVersion epel7

(pkgdb-cli is from the packagedb-cli package)

Comment 5 Paul Howarth 2014-04-25 18:33:57 UTC
(In reply to Emmanuel Seyman from comment #1)
> (In reply to Paul Howarth from comment #0)
> > It's optional but we generally include optional dependencies, at least until
> > such time as soft dependencies can be used.
> 
> Agreed. I'll do this over the weekend.

I've done the fix in the epel7 branch; you can fast-forward merge that into master for a quick fix if you like.

Comment 6 Emmanuel Seyman 2014-04-27 15:03:41 UTC
Done. This will show up in the next rawhide push.


Note You need to log in before you can comment on or make changes to this bug.