Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Devel-CheckOS CPAN distribution.

Report information
The Basics
Id: 98155
Status: rejected
Priority: 0/
Queue: Devel-CheckOS

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: use Module::Runtime::module_notional_filename
Devel::CheckOS repeats the logic from module_notional_filename. DRY. /Users/ether/.perlbrew/libs/21.2@std/lib/perl5/Devel/CheckOS.pm:188: $file_part =~ s/\.pm$//; /Users/ether/.perlbrew/libs/21.2@std/lib/perl5/Devel/CheckOS.pm:200: } File::Find::Rule->file()->name('*.pm')->in(
On Mon Aug 18 16:20:12 2014, ETHER wrote: Show quoted text
> Devel::CheckOS repeats the logic from module_notional_filename. DRY. > > /Users/ether/.perlbrew/libs/21.2@std/lib/perl5/Devel/CheckOS.pm:188: > $file_part =~ s/\.pm$//; > /Users/ether/.perlbrew/libs/21.2@std/lib/perl5/Devel/CheckOS.pm:200: > } File::Find::Rule->file()->name('*.pm')->in(
Adding a dependency for such a trivial bit of functionality would be really silly in a module which is intended to be bundled with other peoples' code. If you think the code is *wrong* then I'd welcome a bug report, of course.