Bug #70899 [Asn]: php 7 RC 7 buildconf failure

From: Date: Fri, 13 Nov 2015 05:42:51 +0000
Subject: Bug #70899 [Asn]: php 7 RC 7 buildconf failure
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-197218@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=70899&edit=1

 ID:                 70899
 Updated by:         laruence@php.net
 Reported by:        admin at dunkleallianz dot net
 Summary:            php 7 RC 7 buildconf failure
 Status:             Assigned
 Type:               Bug
 Package:            Compile Failure
 Operating System:   Debian 8.3
 PHP Version:        7.0.0RC6
-Assigned To:        reeze
+Assigned To:        bwoebi
 Block user comment: N
 Private report:     N

 New Comment:

@bwoebi since the origin change break things, I suggest we revert it, it's not a critical
change anyway.

thanks


Previous Comments:
------------------------------------------------------------------------
[2015-11-13 02:41:54] reeze@php.net

Hi bob. the second patch's PHP_CURRENT_CONFIG_M4_DIR is a macro to retrive current
extension's dir which is initialized before evaluate every extension, this could be though as
the variable $config_m4_dir but is "autoconf time",  imagick requires preprocess time
path to include another m4 file.

------------------------------------------------------------------------
[2015-11-13 02:37:12] reeze@php.net

Hi bob,

The first patch is incorrect, I just updated it. I moved the macros: PHP_EXT_DIR from
configure.in to  build/config-stubs so every time we include a m4 file from extension
the PHP_EXT_DIR is redefined to the proper one (the same as previous hehavior, but it
done in "autoconf-time", or we can only get the proper path at "configure time"
which triggered this bug report).

Yes the second patch looks better  but I tested it with m4 1.4.6, it didn't work. (Mac shipped
1.4.6 with Xcode, so we might need to compatible with it).

@dunkleallianz could you please try the first patch?

------------------------------------------------------------------------
[2015-11-12 17:52:56] admin at dunkleallianz dot net

ah ok i find the version but this is a really old one (Version 1.4.6) 
m4-1.4.17          2013-09-22 09:11
m4-1.4.9           2007-03-23 21:39

------------------------------------------------------------------------
[2015-11-12 17:43:12] admin at dunkleallianz dot net

reeze the newest version of m4 ist 1.4.17  i can't see something about 1.4.6

if you have Patch it ;) i can test it again

------------------------------------------------------------------------
[2015-11-12 17:19:38] bwoebi@php.net

I'm not an expert here… but where is PHP_EXT_CONFIG_M4_DIR and PHP_CURRENT_CONFIG_M4_DIR
defined? (the first diff)
Not sure what you did there…

Second patch looks more coherent, but what's PHP_CURRENT_CONFIG_M4_SRC doing?

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=70899


--
Edit this bug report at https://bugs.php.net/bug.php?id=70899&edit=1


Thread (19 messages)

« previous php.bugs (#197218) next »