The problem has existed since the PHP 7 RC 2 version, there was this once a report and after that it went again, but be the PHP7 in RC status is, it is no longer

Hey bob.

This breaks start from this commit: 
https://github.com/php/php-src/commit/82b73fd1e3c53228bb31b75a446114b5fd0cd240

After the diff PHP_EXT_DIR will not able to locate other extension's path, there should no one that I think, but it is a break.


This patch (the first one of the gist) should fix the problem: https://gist.github.com/reeze/89fd045f1d0960d2add3

The second is working in m4 1.4.16 but not 1.4.6, so I have to take the dirty one.

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?

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

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

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?

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.

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

thanks

and about the webhelper one, maybe you can introduce a new NEW_EXTENSION_EX, which accpet arguamnets as build_dir, source_dir, it will be more safe than the current one

Do you have a link to the patch or I can simply load the https://github.com/php/php-src/tree/php-7.0.0RC7 again?


Currently I have reloaded the Branch.
same error
Forcing buildconf
Removing configure caches
buildconf: checking installation...
buildconf: autoconf version 2.69 (ok)
rebuilding aclocal.m4
rebuilding configure
/usr/bin/m4:ext/imagick/config.m4:14: cannot open `$config_m4_dir/imagemagick.m4': No such file or directory
autom4te: /usr/bin/m4 failed with exit status: 1
rebuilding main/php_config.h.in
/usr/bin/m4:ext/imagick/config.m4:14: cannot open `$config_m4_dir/imagemagick.m4': No such file or directory
autom4te: /usr/bin/m4 failed with exit status: 1
autoheader: '/usr/bin/autom4te' failed with exit status: 1

or something i do wrong ?

So i have test this patch https://gist.github.com/reeze/89fd045f1d0960d2add3#file-more-compatible-patch-diff

so i have test it 
wget https://github.com/mkoppanen/imagick/archive/phpseven.zip
wget https://downloads.php.net/~ab/php-7.0.0RC7.tar.gz

tar xvf php-7.0.0RC7.tar.gz
unzip phpseven.zip

mv imagick-phpseven imagick 
cp imagick php-7.0.0RC7/ext/

Edit patch the 2 files 
rm php-7.0.0RC7/configure 

./buildconf --force 


./configure '--prefix=/usr/share/php7' '--datadir=/usr/share/php7' '--mandir=/usr/share/man' '--bindir=/usr/bin/php7' '--with-libdir=lib/x86_64-linux-gnu' '--includedir=/usr/include/php7' '--sysconfdir=/etc/php7/apache2' '--with-config-file-path=/etc/php7/apache2' '--with-config-file-scan-dir=/etc/php7/conf.d' '--enable-libxml' '--enable-session' '--with-pcre-regex=/usr' '--enable-xml' '--enable-simplexml' '--enable-filter' '--disable-debug' '--enable-inline-optimization' '--disable-rpath' '--disable-static' '--enable-shared' '--with-pic' '--with-gnu-ld' '--with-gd' '--with-jpeg-dir' '--with-png-dir' '--with-xpm-dir' '--enable-exif' '--with-zlib' '--with-bz2' '--with-curl' '--with-ldap' '--with-freetype-dir' '--enable-soap' '--enable-sockets' '--enable-calendar' '--enable-ftp' '--enable-mbstring' '--enable-gd-native-ttf' '--enable-bcmath' '--enable-zip' '--with-pear' '--with-openssl' '--with-imap' '--with-imap-ssl' '--with-kerberos' '--enable-phar' '--enable-pdo' '--with-mysqli' '--with-mysql-sock' '--with-mcrypt' '--with-pdo-mysql' '--enable-mysqlnd' '--with-imagick' 


now it works

@amind instead of waitting for final fix of php7, you can also fix this in imagick side by move whole contents of imagemagick.m4 into config.m4 as it only define some AF_FUNCS, I don't think it's necessary to keep it in a separted file.

thanks

do you forgot something in the rc8 ?

*the some failure in rc8

Anatol told me it were not critical enough; feel free to complain…

Hi admin at dunkleallianz,

I can't reproduce this issue for the RC version of Imagick that supports PHP 7 that has just been tagged.

Please could you let me know if it's still an issue for you with the released version of PHP 7. If so I guess I'll could make a change to support it before 7.0.1 is released.

Either here, or it's easier for me to get notifications at https://github.com/mkoppanen/imagick/issues

https://github.com/mkoppanen/imagick/issues/128