Skip Menu |

This queue is for tickets about the ExtUtils-Depends CPAN distribution.

Report information
The Basics
Id: 46128
Status: resolved
Priority: 0/
Queue: ExtUtils-Depends

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

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



Subject: better NETA.yml
META.yml will have more data in it, including modules used. And Test::More was added only in perl 5.6.2, so it is better to declare all deps that are on CPAN. Also better win32 compatibility when doing `make dist`. -- Alexandr Ciornii, http://chorny.net
Subject: Makefile.PL.patch
--- Makefile.PL.dist 2009-05-16 00:15:08.500000000 +0300 +++ Makefile.PL 2009-05-16 00:25:17.203125000 +0300 @@ -7,12 +7,31 @@ WriteMakefile( 'dist' => { - PREOP => 'pod2text lib/ExtUtils/Depends.pm | tee README >$(DISTVNAME)/README; chmod -R u=rwX,go=rX . ;', + ($^O eq 'MSWin32'?(): + (PREOP => 'pod2text lib/ExtUtils/Depends.pm | tee README >$(DISTVNAME)/README; chmod -R u=rwX,go=rX . ;',)), COMPRESS => 'gzip -9v', SUFFIX => '.gz', }, 'NAME' => 'ExtUtils::Depends', 'VERSION_FROM' => 'lib/ExtUtils/Depends.pm', + PREREQ_PM => { + 'Test::More' => 0, #build only + 'File::Spec' => 0, + 'Data::Dumper' => 0, + 'IO::File' => 0, + }, + ($ExtUtils::MakeMaker::VERSION ge '6.48'? + (MIN_PERL_VERSION => 5.006, #for META.yml + META_MERGE => { + resources=> { + repository => 'http://svn.gnome.org/svn/perl-ExtUtils-Depends/trunk/', + }, + keywords => ['XS','XS extensions','dependency'], + }, + 'LICENSE' => 'perl', + ) : ()), + + ); use Cwd;
On Fri May 15 17:32:30 2009, CHORNY wrote: Show quoted text
> META.yml will have more data in it, including modules used. And > Test::More was added only in perl 5.6.2, so it is better to declare all > deps that are on CPAN. > Also better win32 compatibility when doing `make dist`.
I committed a slightly altered patch: <http://git.gnome.org/cgit/perl-ExtUtils-Depends/commit/?id=de1e58>. In particular, I dropped the EU::MM version check -- using string comparison isn't correct, and using numeric comparison breaks on developer versions of EU::MM (because it doesn't do the eval-dance to remove the underscore). People with older versions of EU::MM will see harmless warnings now, but I think that's fine. Thanks for the patch!