Re: [PATCH] fixes for ActiveState Perl

On Mon, Feb 14, 2011 at 8:39 PM, Rafael Kitover [off-list ref] wrote:

On 2/14/2011 7:49 AM, Erik Faye-Lund wrote:
...

quoted

quoted

diff --git a/perl/Makefile b/perl/Makefile
index a2ffb64..1fa99cd 100644
--- a/perl/Makefile
+++ b/perl/Makefile

@@ -3,18 +3,28 @@

 #
 makfile:=perl.mak

+# support PERL_PATH=C:\\Perl\\bin\\perl
+PERL_PATH := $(subst \,\\,$(PERL_PATH))
+
 PERL_PATH_SQ = $(subst ','\'',$(PERL_PATH))
+PERL_MAKE := MAKEFLAGS="" $(subst \,\\,$(shell $(subst \,\\,$(PERL_PATH)) -MConfig -le "print ((\%Config)->{make})"))
+
+ifneq (,$(findstring nmake,$(PERL_MAKE)))
+       PERL_MAKE := $(PERL_MAKE) -nologo
+endif
+
 prefix_SQ = $(subst ','\'',$(prefix))

 ifndef V
       QUIET = @
 endif

+

Why?

Windows perl (activestate, strawberry) uses nmake or dmake. The
makefiles for these makes are incompatible with GNU make. This code
retrieves the make Perl would use from its Config, and uses that on
invoking perl.mak .

This should also allow for using EU::MM for the MSVC+ActiveState build.

I'm sorry for not being clear; I was talking about that single
new-line, not the PERL_MAKE-stuff.