Bug #54029 for Catalyst-Authentication-Credential-HTTP: Wrong dependency types

Wed Jan 27 04:43:12 2010 REHSACK [...] cpan.org - Ticket created

Subject:

Wrong dependency types

Hi Thomas, the dependencies to Test::Exception and Test::MockObject aren't runtime dependencies (requires) but test dependencies (build_requires). Maybe - when you fix RT #47098 - you can change this one, too? Jens

Sat May 08 16:59:10 2010 bobtfish [...] bobtfish.net - Correspondence added

I don't see why this is an issue - surely the entire point of build_requires and test_requires is to do that? I'm going to close this, but please feel free to re-open with an explanation of what I'm missing here.. Cheers t0m

Sat May 08 16:59:11 2010 The RT System itself - Status changed from 'new' to 'open'

Sat May 08 16:59:11 2010 bobtfish [...] bobtfish.net - Status changed from 'open' to 'resolved'

Sun May 09 06:42:21 2010 REHSACK [...] cpan.org - Correspondence added

On Sat May 08 16:59:10 2010, BOBTFISH wrote: Show quoted text

> I don't see why this is an issue - surely the entire point of > build_requires and test_requires is to do that?

Sure - and that's why both dependencies should be moved from requires => test_requires. The issue is, that both are listed as runtime requirements. Show quoted text

> I'm going to close this, but please feel free to re-open with an > explanation of what I'm missing here..

I thought I explained it - sorry when I'm impolite and re-open without a better explanation. Best regards, Jens

Sun May 09 06:42:23 2010 The RT System itself - Status changed from 'resolved' to 'open'

Tue Jun 15 21:03:22 2010 bobtfish [...] bobtfish.net - Correspondence added

Thanks for re-opening. I was obviously insane when I looked at this first time round and you were entirely clear what the issue was. Sorry about that. I've committed the change as r13355.

Tue Jun 15 21:03:24 2010 bobtfish [...] bobtfish.net - Status changed from 'open' to 'resolved'