kaare/Role-REST-Client — pull request #14 by markstos (2012-11-06 22:13:21+00:00)
Title: remove MooseX::Declare as dependency, don't require HTTP::Tiny to be loaded if it is not used

Hello,

Here are two patches to consider.

First, I found I couldn't install this distribution on two machines that I tried because of the MooseX::Declare dependency, which I found was only used in one place in the test suite.

One of patches updates that test to avoid MooseX::Declare as a dependency, which saves installing it's own dependency chain as well as that module itself.

Second, I noticed that HTTP::Tiny is required and loaded into memory even if it never used. I made a small change to make it loaded on demand, so that if you are using another user agent, HTTP::Tiny does not need to be loaded.

I'm not familiar with the dist.ini syntax, but I also wanted to more formally express that HTTP::Tiny may be required for testing and is recommend, but it is not actually a required dependency.

In a large project that is already using LWP::UserAgent everywhere else, it would be nice to not have yet-another user agent install and in use, even if HTTP::Tiny has it's own merits.

Thanks for the consideration!

Mark