Closed ribasushi closed 9 years ago
It's test/recommends, not runtime. The module is still Tiny.
Why Inline::C? To test that Capture::Tiny can capture output to C filehandles.
What would you suggest as an alternative approach?
It's test/recommends, not runtime.
I specifically addressed this in https://github.com/dagolden/Capture-Tiny/issues/32#issue-57219248:
... (and does it after the testing took place, so test_recommends are essentially moot)
This is also why I included the full log. I am not sure what to recommend, given that the test does not run in either case:
[18:24:44] t/19-inline-c.t ............ skipped: Inline::C module required
Maybe an excerpt of the full log would have made it clearer instead of looking like spam.
Putting test recommends after the module tests looks like a CPAN.pm bug to me. Might still be my fault, but wrong repo to complain in.
David,
What I gave you is an excerpt. The full log is in an expire-able pastebin and is 4 times larger than what I attached (factoring in that it stopped half way due to a timeout, as the deps were taking > 10 minutes to install). Hence why the ...
's at the start and end of the log.
I assumed github would auto-truncate long entries on display, apparently it is not RT-level smart.
You are not being trolled, this is a bugreport describing an actual issue.
Would it not be possible to lower the test-recommends prereq to test-suggests, or leave it out entirely? You can still run that test if Inline::C is installed, but if the user doesn't need that feature, I don't see a point in testing it.
Perhaps I can work up a way to only require it for CPAN Testers. Seeing cross-platform test results of this is quite helpful. Note -- this isn't about Inline::C per, se, it's just a convenient way to test output from XS.
Another option could be to write a simple Acme::OutputFromXS module and test/recommend that, which would give XS output for testing without the Inline dependency tree hell.
Perhaps I can work up a way to only require it for CPAN Testers
[DynamicPrereqs]
-raw = $WriteMakefileArgs{TEST_REQUIRES}{'Inline::C'} = $FallbackPrereqs{'Inline::C'} = '0' if $ENV{AUTOMATED_TESTING};
A combination of events causes the following list of deps (non-core or upgrades) is pulled in on a standard perl 5.20 configuration:
The avalanche is triggered by https://metacpan.org/changes/distribution/CPAN#L45 which enables recommends installation by default (and does it after the testing took place, so test_recommends are essentially moot).
I am not sure what the proper way forward is, as it is not entirely clear to me why C::T recommends Inline::C in the first place.
Attached is an excerpt of the actual CPAN worklog: