Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Requires CPAN distribution.

Report information
The Basics
Id: 56152
Status: rejected
Priority: 0/
Queue: Test-Requires

People
Owner: Nobody in particular
Requestors: NKH [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: Better name suggestion for Module and API
See: http://search.cpan.org/~kitano/Test-Skip-UnlessExistsExecutable-0.041/lib/Test/Skip/UnlessExistsExecutable.pm It would be nice if the above module and this module could share code or be plugins to a higher level Test::Skip module. I have uploaded a place holder for Test::Skip to CPAN. The API doesn't support a user comment argument that is displayed in case of error. This may not be needed but I expected it to work as other test subs work. The Error message does not display the version required (which can be passed as argument)
Subject: Test::Skip plugins
Subject: Re: [rt.cpan.org #56152] Test::Skip plugins
Date: Thu, 1 Apr 2010 11:48:36 +0900
To: bug-Test-Requires [...] rt.cpan.org
From: Tokuhiro Matsuno <tokuhirom [...] gmail.com>
I don't think this is good idea. I don't want to depend any module. Test::Requires is simple and useful enough. Test::Requires never depend to Test::Skip. But so, the good solution is adding method to Test::Builder. regards, On Wed, Mar 31, 2010 at 8:51 PM, NKH via RT <bug-Test-Requires@rt.cpan.org> wrote: Show quoted text
>       Queue: Test-Requires >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=56152 > > > I have notified The other module author: > > https://rt.cpan.org/Ticket/Display.html?id=56153&results=6752828ea4ce0fade07a86995ecff120 > >
Subject: Re: [rt.cpan.org #56152] Test::Skip plugins
Date: Thu, 1 Apr 2010 12:04:42 +0900
To: bug-Test-Requires [...] rt.cpan.org
From: Tokuhiro Matsuno <tokuhirom [...] gmail.com>
And also, I don't like Test::Skip. 1. It is over-engineering. Simple interface makes happy. (Test::Requires already provides it.) 2. Test::* modules should not depend to any other modules. regards, On Thu, Apr 1, 2010 at 11:48 AM, tokuhirom@gmail.com via RT <bug-Test-Requires@rt.cpan.org> wrote: Show quoted text
>       Queue: Test-Requires >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=56152 > > > I don't think this is good idea. I don't want to depend any module. > Test::Requires is simple and useful enough. > Test::Requires never depend to Test::Skip. > > But so, the good solution is adding method to Test::Builder. > > regards, > > On Wed, Mar 31, 2010 at 8:51 PM, NKH via RT > <bug-Test-Requires@rt.cpan.org> wrote:
>>       Queue: Test-Requires >>  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=56152 > >> >> I have notified The other module author: >> >> https://rt.cpan.org/Ticket/Display.html?id=56153&results=6752828ea4ce0fade07a86995ecff120 >> >>
> >
I want to reject this report.
On Thu Sep 09 22:27:16 2010, TOKUHIROM wrote: Show quoted text
> I want to reject this report.
It is up to you if you want to listen to your users and have a more unified interface or not. You misunderstood the fact that your module would not be dependent on anything but my module would be dependent on yours. When I get to it, I'll just wrap your module and do whatever I feel is more logical without having any impact on your work. This may be the best compromise. Reject this ticket if you so want but my opinion has not changed.
You can write any wrapper module, I don't stop it. First, you should write some code. thanks.