Re: libvirt-php --with-php-config=/path/to/php is broken.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/10/2017 07:46 PM, Shaun Reitan wrote:
> When running the following command I ended up getting errors that phpize
> was missing. Trying to dig into this further i found that configure.ac
> was just trying to run php-config without providing a path.  The
> following patch corrects the issue by using the $PHPCONFIG variable
> instead.
> 
> ./configure --prefix=/opt/libvirt-php
> --with-php-config=/path/to/php-config`
> 
> diff --git a/configure.ac b/configure.ac
> index f232756..d93e946 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -159,7 +159,7 @@ if test "x$PHPCONFIG" = "xno"; then
>     AC_MSG_ERROR([php-config not found; please install the PHP SDK])
> fi
> 
> -AC_PATH_PROG([PHPIZE], [phpize], [no], [`php-config
> --prefix`/bin$PATH_SEPARATOR$PATH])
> +AC_PATH_PROG([PHPIZE], [phpize], [no], [`$PHPCONFIG
> --prefix`/bin$PATH_SEPARATOR$PATH])
> if test "x$PHPIZE" = "xno"; then
>     AC_MSG_ERROR([phpize not found; please install the PHP SDK])
> fi
> 

Hey, sorry for the late reply. It's only now that I've noticed this
e-mail. This is obviously a bug and your fix is correct. I've turned it
into a commit and pushed it. Thank you!

Michal

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux