php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #75541 Compile error as php_smart_string.h incl's non-existent zend_smart_string.h
Submitted: 2017-11-20 09:37 UTC Modified: 2017-11-20 10:36 UTC
From: raivo dot hool at gmail dot com Assigned:
Status: Closed Package: Compile Failure
PHP Version: 7.2.0RC6 OS: FreeBSD 11.1
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
14 + 30 = ?
Subscribe to this entry?

 
 [2017-11-20 09:37 UTC] raivo dot hool at gmail dot com 
Description:
------------
I tried to build pecl-redis against PHP 7.2RC6, and tried both the latest stable release (3.1.4) and the git master branch. Both failed with the same result:

In file included from ./common.h:419:
/usr/local/include/php/ext/standard/php_smart_string.h:21:10: fatal error: 'zend_smart_string.h' file not found
#include "zend_smart_string.h"
         ^~~~~~~~~~~~~~~~~~~~~
1 error generated.
*** Error code 1

It appears that ext/standard/php_smart_string.h includes zend_smart_string.h, which does not exist. There exists, however, zend_smart_str.h. Including that file allows pecl-redis to build (even though it doesn't actually work yet for unrelated reasons).

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2017-11-20 10:06 UTC] spam2 at rhsoft dot net 
that bug should have been filed against pecl-redis because as always 7.0/7.1 and now 7.2 includes API changes and extensions needs to be updated to compile against newer PHP versions
 [2017-11-20 10:10 UTC] raivo dot hool at gmail dot com 
I will do that. It just caught my eye that a non-existent file is #included. For a moment I thought it *might* be misspelled on purpose, to make the builds fail and the authors to take notice, but then dismissed the idea as I thought it was perhaps too cryptic.
 [2017-11-20 10:27 UTC] nikic@php.net 
This is supposed to work. The file is in Zend/zend_smart_string.h and there should be an -IZend/ argument on the compiler invocation.
 [2017-11-20 10:36 UTC] raivo dot hool at gmail dot com
-Status: Open +Status: Closed
 [2017-11-20 10:36 UTC] raivo dot hool at gmail dot com 
My mistake. The FreeBSD port simply fails to install zend_smart_string.h. I'll take my complaints elsewhere.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Mon May 06 12:01:31 2024 UTC