Bug #76309 for Dist-Zilla-Plugin-Test-Compile: don't skip if Test::Script not present

Wed Apr 04 14:10:16 2012 xenoterracide [...] gmail.com - Ticket created

Subject:	don't skip if Test::Script not present
Date:	Wed, 4 Apr 2012 13:10:06 -0500
To:	bugs-dist-zilla-plugin-test-compile [...] rt.cpan.org
From:	Caleb Cushing <xenoterracide [...] gmail.com>

my personal opinion is that I'd rather like this test to run on all systems. I use it as a basic sanity check and I'd like to know if it's going to fail in any environment. (also I find it weird that this is named Test::Compile but uses Test::Script instead of Test::Compile ) -- Caleb Cushing http://xenoterracide.com

Tue Apr 17 11:49:37 2012 dolmen [...] cpan.org - Requestor DOLMEN added

Tue Apr 17 12:24:56 2012 dolmen [...] cpan.org - Correspondence added

Le 2012-04-04 20:10:16, XENO a écrit : Show quoted text

> my personal opinion is that I'd rather like this test to run on all > systems. I use it as a basic sanity check and I'd like to know if it's > going to fail in any environment.

I agree with Caleb. Note that Test::Script presence should be enforced only if the distribution provides scripts. A distribution that provides only modules should not require Test::Script. Enforcing Test::Script presence requires: - to load Test::Script unconditionnally (fail instead of skip if it is missing) - to list Test::Script in the distribution dependancies (META.yml/json) -- Olivier Mengué - http://perlresume.org/DOLMEN

Tue Apr 17 12:24:56 2012 dolmen [...] cpan.org - Status changed from 'new' to 'open'

Tue Apr 17 16:02:50 2012 xenoterracide [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #76309] don't skip if Test::Script not present
Date:	Tue, 17 Apr 2012 15:02:35 -0500
To:	bug-Dist-Zilla-Plugin-Test-Compile [...] rt.cpan.org
From:	Caleb Cushing <xenoterracide [...] gmail.com>

2012/4/17 Olivier Mengué via RT <bug-Dist-Zilla-Plugin-Test-Compile@rt.cpan.org>: Show quoted text

> Note that Test::Script presence should be enforced only if the > distribution provides scripts. A distribution that provides only modules > should not require Test::Script.

I'd like it even for modules... but just by making it not conditionally loaded (e.g use Test::Script; ) will do that in dzil -- Caleb Cushing http://xenoterracide.com

Wed Jul 03 19:21:48 2013 ether [...] cpan.org - Taken

Fri Aug 02 11:41:15 2013 ether [...] cpan.org - Correspondence added

Test::Script is now required if scripts are present, and not referenced at all if not.

Fri Aug 02 11:41:16 2013 ether [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Aug 02 11:41:16 2013 ether [...] cpan.org - Fixed in 2.007-TRIAL added