diff mbox series

[1/1] php: fix build with mysqli

Message ID 20180827132127.20658-1-chrismcc@gmail.com
State Accepted
Commit 65f96452636b96c01ea3f00db6e4e7a240837101
Headers show
Series [1/1] php: fix build with mysqli | expand

Commit Message

Christopher McCrory Aug. 27, 2018, 1:21 p.m. UTC
This fixes autobuild failures like
http://autobuild.buildroot.net/results/3288b742cee650ee47a41c5b4d6aaef1fe67bff1

php compile breaks with:

ext/mysqlnd/mysqlnd_auth.o: In function `php_mysqlnd_scramble_sha2':
mysqlnd_auth.c:(.text+0x1054): undefined reference to `PHP_SHA256Init'
mysqlnd_auth.c:(.text+0x1064): undefined reference to `PHP_SHA256Update'
mysqlnd_auth.c:(.text+0x1070): undefined reference to `PHP_SHA256Final'
mysqlnd_auth.c:(.text+0x1078): undefined reference to `PHP_SHA256Init'
mysqlnd_auth.c:(.text+0x1088): undefined reference to `PHP_SHA256Update'
mysqlnd_auth.c:(.text+0x1094): undefined reference to `PHP_SHA256Final'
mysqlnd_auth.c:(.text+0x109c): undefined reference to `PHP_SHA256Init'
mysqlnd_auth.c:(.text+0x10ac): undefined reference to `PHP_SHA256Update'
mysqlnd_auth.c:(.text+0x10bc): undefined reference to `PHP_SHA256Update'
mysqlnd_auth.c:(.text+0x10c8): undefined reference to `PHP_SHA256Final'

It looks like the php mysqli extension needs the hash extension to work.  This
seems to be a php Make dependany bug.  This patch works around it until the
upstream maintainers can fix it.

--
I'm OK with modifing the above at commit time.

Signed-off-by: Christopher McCrory <chrismcc@gmail.com>
---
 package/php/Config.ext | 1 +
 1 file changed, 1 insertion(+)

Comments

Peter Korsgaard Aug. 27, 2018, 3:31 p.m. UTC | #1
>>>>> "Christopher" == Christopher McCrory <chrismcc@gmail.com> writes:

 > This fixes autobuild failures like
 > http://autobuild.buildroot.net/results/3288b742cee650ee47a41c5b4d6aaef1fe67bff1

 > php compile breaks with:

 > ext/mysqlnd/mysqlnd_auth.o: In function `php_mysqlnd_scramble_sha2':
 > mysqlnd_auth.c:(.text+0x1054): undefined reference to `PHP_SHA256Init'
 > mysqlnd_auth.c:(.text+0x1064): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x1070): undefined reference to `PHP_SHA256Final'
 > mysqlnd_auth.c:(.text+0x1078): undefined reference to `PHP_SHA256Init'
 > mysqlnd_auth.c:(.text+0x1088): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x1094): undefined reference to `PHP_SHA256Final'
 > mysqlnd_auth.c:(.text+0x109c): undefined reference to `PHP_SHA256Init'
 > mysqlnd_auth.c:(.text+0x10ac): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x10bc): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x10c8): undefined reference to `PHP_SHA256Final'

 > It looks like the php mysqli extension needs the hash extension to work.  This
 > seems to be a php Make dependany bug.  This patch works around it until the
 > upstream maintainers can fix it.

 > --
 > I'm OK with modifing the above at commit time.

 > Signed-off-by: Christopher McCrory <chrismcc@gmail.com>

The Signed-off-by line should go above the --  (like git commit -s
automatically does) so it isn't dropped by git when the patch is
applied.

Committed with that fixed, thanks.
Peter Korsgaard Aug. 28, 2018, 8:13 a.m. UTC | #2
>>>>> "Christopher" == Christopher McCrory <chrismcc@gmail.com> writes:

 > This fixes autobuild failures like
 > http://autobuild.buildroot.net/results/3288b742cee650ee47a41c5b4d6aaef1fe67bff1

 > php compile breaks with:

 > ext/mysqlnd/mysqlnd_auth.o: In function `php_mysqlnd_scramble_sha2':
 > mysqlnd_auth.c:(.text+0x1054): undefined reference to `PHP_SHA256Init'
 > mysqlnd_auth.c:(.text+0x1064): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x1070): undefined reference to `PHP_SHA256Final'
 > mysqlnd_auth.c:(.text+0x1078): undefined reference to `PHP_SHA256Init'
 > mysqlnd_auth.c:(.text+0x1088): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x1094): undefined reference to `PHP_SHA256Final'
 > mysqlnd_auth.c:(.text+0x109c): undefined reference to `PHP_SHA256Init'
 > mysqlnd_auth.c:(.text+0x10ac): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x10bc): undefined reference to `PHP_SHA256Update'
 > mysqlnd_auth.c:(.text+0x10c8): undefined reference to `PHP_SHA256Final'

 > It looks like the php mysqli extension needs the hash extension to work.  This
 > seems to be a php Make dependany bug.  This patch works around it until the
 > upstream maintainers can fix it.

Committed to 2018.02.x and 2018.05.x, thanks.
diff mbox series

Patch

diff --git a/package/php/Config.ext b/package/php/Config.ext
index 6b3d86925d..60ef1aee7c 100644
--- a/package/php/Config.ext
+++ b/package/php/Config.ext
@@ -113,6 +113,7 @@  endif
 
 config BR2_PACKAGE_PHP_EXT_MYSQLI
 	bool "Mysqli"
+	select BR2_PACKAGE_PHP_EXT_HASH
 	help
 	  MySQL Improved extension support