CVS commit: pkgsrc/lang/php

["Jonathan Perkin" <jperkin%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   jperkin
Date:           Wed Feb  7 16:14:36 UTC 2018

Modified Files:
        pkgsrc/lang/php: ext.mk

Log Message:
php: Remove LIBTOOL_OVERRIDE=YES.

This was added 8 years ago intending to ensure pkgsrc libtool was used, except
it actually did the opposite.  By default we will try to replace any script
named "libtool" up to the override depth.  Setting LIBTOOL_OVERRIDE changes
the behaviour to instead override the list of filenames specified by the
LIBTOOL_OVERRIDE variable.  Thus we were trying to replace any file named
"YES" instead of "libtool".

Noticed by SmartOS/clang builds of php-intl which were using the supplied
broken libtool script instead of the correct pkgsrc one.


To generate a diff of this commit:
cvs rdiff -u -r1.44 -r1.45 pkgsrc/lang/php/ext.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/lang/php/ext.mk
diff -u pkgsrc/lang/php/ext.mk:1.44 pkgsrc/lang/php/ext.mk:1.45
--- pkgsrc/lang/php/ext.mk:1.44 Sat Dec 10 07:08:39 2016
+++ pkgsrc/lang/php/ext.mk      Wed Feb  7 16:14:36 2018
@@ -1,4 +1,4 @@
-# $NetBSD: ext.mk,v 1.44 2016/12/10 07:08:39 taca Exp $
+# $NetBSD: ext.mk,v 1.45 2018/02/07 16:14:36 jperkin Exp $
 #
 # PHP extension package framework, for both PECL and bundled PHP extensions.
 #
@@ -72,7 +72,6 @@ CONFIGURE_ARGS+=      --with-php-config=${PHP
 
 USE_CONFIG_WRAPPER=    YES
 USE_LIBTOOL=           YES
-LIBTOOL_OVERRIDE=      YES
 USE_TOOLS+=            automake
 
 # Ensure we export symbols in the linked shared object.