Skip Menu |

This queue is for tickets about the SHARYANTO-Module-Path CPAN distribution.

Report information
The Basics
Id: 98273
Status: rejected
Priority: 0/
Queue: SHARYANTO-Module-Path

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

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



Subject: please send patches to the original version
Why does this distribution exist? It looks like a fork of Module::Path. Wouldn't it be better to send patches back to the original author rather than creating something new?
On Fri Aug 22 20:21:30 2014, ETHER wrote: Show quoted text
> Why does this distribution exist? It looks like a fork of > Module::Path. Wouldn't it be better to send patches back to the > original author rather than creating something new?
I did open a couple of issues on GitHub, but so far Neil doesn't seem to be interested in them. ------------------------ commit 37f8241 Author: Steven Haryanto (on PC, Jakarta) <stevenharyanto@gmail.com> Date: Tue Aug 26 12:36:59 2014 +0700 Add description about this module's raison d'etre [RT#98273] diff --git a/lib/SHARYANTO/Module/Path.pm b/lib/SHARYANTO/Module/Path.pm index b8836f5..3a112ff 100644 --- a/lib/SHARYANTO/Module/Path.pm +++ b/lib/SHARYANTO/Module/Path.pm @@ -189,6 +189,27 @@ _ =head1 DESCRIPTION +This module is a fork of L<Module::Path>. It contains features that are not (or +have not been accepted) in the original module, namely: finding all matches +instead of the first found match, and finding .pmc/.pod in addition to .pm +files. There is also a difference of behavior: no abs_path() or symlink +resolving is being done by default because I think that's the sensible default +(doing abs_path() or resolving symlinks will sometimes fail or expose filesystem +quirks that we might not want to deal with at all). However, an C<abs> bool +option is provided if a user wants to do that. + +References: + +=over + +=item * L<https://github.com/neilbowers/Module-Path/issues/6> + +=item * L<https://github.com/neilbowers/Module-Path/issues/7> + +=item * L<https://github.com/neilbowers/Module-Path/issues/10> + +=back + =head1 SEE ALSO