clean up module hiding in tests

[p5sagit/JSON-MaybeXS.git] / Makefile.PL

diff --git a/Makefile.PL b/Makefile.PL
index 2982b81..0b78345 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -1,7 +1,7 @@
 use strict;
 use warnings FATAL => 'all';
 use ExtUtils::MakeMaker;
-(do 'maint/Makefile.PL.include' or die $@) unless -f 'META.yml';
+(do './maint/Makefile.PL.include' or die $@) unless -f 'META.yml';
 
 my %WriteMakefileArgs = (
   NAME => 'JSON::MaybeXS',
@@ -11,9 +11,14 @@ my %WriteMakefileArgs = (
     'meta-spec' => { version => 2 },
     dynamic_config => 1,
     resources => {
+      # GitHub mirrors from Shadowcat. We list it so we can get pull requests.
+      # The canonical repo is:
+      # r/o: git://git.shadowcat.co.uk/p5sagit/JSON-MaybeXS.git
+      # r/w: p5sagit@git.shadowcat.co.uk:JSON-MaybeXS.git
+      # web: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=p5sagit/JSON-MaybeXS.git
       repository => {
-        url => 'https://github.com/karenetheridge/JSON-MaybeXS.git',
-        web => 'https://github.com/karenetheridge/JSON-MaybeXS',
+        url => 'https://github.com/p5sagit/JSON-MaybeXS.git',
+        web => 'https://github.com/p5sagit/JSON-MaybeXS',
         type => 'git',
       },
       bugtracker => {
@@ -21,9 +26,19 @@ my %WriteMakefileArgs = (
         web => 'https://rt.cpan.org/Public/Dist/Display.html?Name=JSON-MaybeXS',
       },
     },
+    x_contributors => [ # manually added, from git shortlog -e -s -n
+      'Clinton Gormley <develop@traveljury.com>',
+      'Graham Knop <haarg@haarg.org>',
+      'John SJ Anderson <genehack@genehack.org>',
+      'Karen Etheridge <ether@cpan.org>',
+      'Kieren Diment <diment@gmail.com>',
+      'Matt S Trout <mst@shadowcat.co.uk>',
+    ],
+    keywords => [ qw(json serializer serialiser data) ],
   },
 
   META_ADD => {
+    'meta-spec' => { version => 2 },
     prereqs => {
       configure => {
         requires => {
@@ -35,7 +50,9 @@ my %WriteMakefileArgs = (
       },
       runtime => {
         requires => {
-          'JSON::PP' => '2.27202',
+          'Scalar::Util' => '0',
+          'Carp' => '0',
+          'JSON::PP' => '2.27300',
           # we may also add a runtime prereq for Cpanel::JSON::XS, on the
           # installer's machine
           perl => '5.006',
@@ -44,7 +61,6 @@ my %WriteMakefileArgs = (
       },
       test => {
         requires => {
-          'Test::Without::Module' => '0.17',
           'Test::More' => '0.88',
         },
       },
@@ -67,11 +83,20 @@ for (qw(configure build test runtime)) {
 
 # dynamic prereqs get added here.
 
-# we require Cpanel::JSON::XS, except if JSON::XS is already installed.
-# (we also always recommend Cpanel::JSON::XS, just to make sure.)
-$WriteMakefileArgs{PREREQ_PM}{'Cpanel::JSON::XS'} = '2.3310'
-    if not parse_args()->{PUREPERL_ONLY}
-        && not eval { require JSON::XS; 1; } && can_xs();
+my $args = parse_args();
+
+if (not $args->{PUREPERL_ONLY}) {
+    # we require Cpanel::JSON::XS, except if JSON::XS is already installed.
+    # (we also always recommend Cpanel::JSON::XS, just to make sure.)
+    $WriteMakefileArgs{PREREQ_PM}{'Cpanel::JSON::XS'} = '2.3310'
+        if eval { require Cpanel::JSON::XS; 1 }
+            or (not eval { require JSON::XS; 1; } and can_xs());
+
+    # JSON::XS 3 changed its boolean handling - update it
+    # if JSON::XS is installed and < 3.0
+    $WriteMakefileArgs{PREREQ_PM}{'JSON::XS'} = '3.00'
+        if eval { require JSON::XS; 1 } and not eval { JSON::XS->VERSION('3.0'); 1 };
+}
 
 $WriteMakefileArgs{MIN_PERL_VERSION} = delete $WriteMakefileArgs{PREREQ_PM}{perl} || 0;