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
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: raivo dot hool at gmail dot com
New email:
PHP Version: OS:

 

 [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: Fri Apr 26 12:01:30 2024 UTC