Johannes Sixt wrote: > Brandon Casey schrieb: >> From: Brandon Casey <drafnel@xxxxxxxxx> >> >> It appears that ExtUtils::MakeMaker versions older than 6.11 do not >> implement the DESTDIR mechanism. So add a test to the generated perl.mak >> to detect when DESTDIR is used along with a too old ExtUtils::MakeMaker and >> abort with a message suggesting the use of NO_PERL_MAKEMAKER. >> >> Signed-off-by: Brandon Casey <casey@xxxxxxxxxxxxxxx> >> --- >> >> >> This just reverses the logic in the test on $(MM_VERSION) so that the test >> will also fail if MM_VERSION is unset. Who knows if ancient versions set >> it. Sorry for the quick v2. >> >> -brandon >> >> >> perl/Makefile.PL | 8 ++++++++ >> 1 files changed, 8 insertions(+), 0 deletions(-) >> >> diff --git a/perl/Makefile.PL b/perl/Makefile.PL >> index 320253e..0b9deca 100644 >> --- a/perl/Makefile.PL >> +++ b/perl/Makefile.PL >> @@ -5,6 +5,14 @@ sub MY::postamble { >> instlibdir: >> @echo '$(INSTALLSITELIB)' >> >> +ifneq (,$(DESTDIR)) >> +ifeq (0,$(shell expr '$(MM_VERSION)' '>' 6.10)) > > I don't think the test works as intended, because 6.2 *is* greater than > 6.10 (aka 6.1). Hmm... I think you're right. -brandon -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html