| Subject: | bad mod_perl dep |
| Date: | Wed, 5 Jun 2013 15:17:04 +0100 |
| To: | bug-OAuth-Lite [...] rt.cpan.org |
| From: | Zefram <zefram [...] fysh.org> |
OAuth::Lite::Server::mod_perl2 requires a bunch of mod_perl modules, but
the dependency isn't declared in OAuth-Lite's META.yml or Makefile.PL.
This doesn't show up in the main part of the test suite, because
it doesn't test OAuth::Lite::Server::mod_perl2, but does cause test
failure in the POD coverage check (which attempts to load all modules)
if Test::Pod::Coverage has been installed but mod_perl has not.
I suspect that there's an attempt here to make mod_perl an optional
dependency, so that the rest of OAuth-Lite can be used without needing
to install mod_perl. If that is the intention, it would be best to
split this distro into two parts. OAuth-Lite itself would have nothing
specific to mod_perl. The module OAuth::Lite::Server::mod_perl2 would
go in a new distro OAuth-Lite-Server-mod_perl2, which would depend
(and declare dependency) on OAuth-Lite and mod_perl.
-zefram