Fixed bug #62460 (php binaries installed as binary.dSYM) #135

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@reeze
Contributor
reeze commented Jul 18, 2012

Hi,
See this fix for autoconf for Mac OS X 10.5 & 10.6: http://marc.info/?l=php-cvs&m=125961714419896
but Mac OS X 10.7&10.8 affected too, It affect PHP-5.3 only(5.3 requires autoconf 2.13, 5.4 didn't).

see also:
https://bugs.php.net/bug.php?id=62460 (10.8)
http://news.php.net/php.internals/61261 (10.7)

Thanks

@smalyshev
Contributor

Interestingly enough, my system returns host i386-apple-darwin10.8.0 on Darwin 10.8. What's host on your system?

@reeze
Contributor
reeze commented Jul 19, 2012

Hi, stas:

In my system (10.7.3), it output:

$host_alias = i386-apple-darwin11.4.0
➜  php-src-5.3 git:(fix-configure-exeext) ✗ uname -a
Darwin MacBookPro.local 11.4.0 Darwin Kernel Version 11.4.0: Mon Apr  9 19:32:15 PDT 2012; 
root:xnu-1699.26.8~1/RELEASE_X86_64 x86_64
@laruence
Member

does set that exeext to null harm?
if not, we can simply set it for all darwin*..

thanks

@smalyshev
Contributor

No, it doesn't harm just wanted to understand why I can't reproduce it. The patch itself looks fine.

@reeze
Contributor
reeze commented Jul 19, 2012

did you try clear config.cache file in my box, after clear it could be reproduced.

@laruence
Member

the interesting this is, why 5.4 doesn't need this? in the same box, you should have the same autoconf with the same version right?

I will assign the bug to johannes, could you please attach this patch to that bug entry ?

@reeze
Contributor
reeze commented Jul 19, 2012

Hi,
It's a bug of autoconf 2.1.3 in mac os,
PHP5.4 require 2.59, I'v installed two versions: 2.13 and 2.59 to build 5.3 and 5.4.
so it's PHP5.3 only bug.

I will attach it to the bug.

@johannes @reeze
thanks

reeze | reeze.cn

在 2012年7月19日星期四,下午2:18,Xinchen Hui 写道:

the interesting this is, why 5.4 doesn't need this? in the same box, you should have the same autoconf with the same version right?

I will assign the bug to johannes, could you please attach this patch to that bug entry ?


Reply to this email directly or view it on GitHub:
#135 (comment)

@php-pulls

Comment on behalf of stas at php.net:

merged

@php-pulls php-pulls closed this Aug 6, 2012
@mattheath mattheath referenced this pull request in mattheath/puppet-php Mar 22, 2013
Closed

Unable to build PHP 5.3.2 or 5.3.3 #16

@php-pulls php-pulls pushed a commit that referenced this pull request Sep 25, 2013
@dstogov dstogov Merge branch 'PHP-5.5'
* PHP-5.5:
  Fixed issue #135 (segfault in interned strings if initial memory is too low)

Conflicts:
	NEWS
4e249c8
@php-pulls php-pulls pushed a commit that referenced this pull request Sep 27, 2013
Christopher Jones Merge branch 'master' of https://git.php.net/repository/php-src
* 'master' of https://git.php.net/repository/php-src:
  Added function opcache_compile_file() to load PHP scripts into cache without execution.
  Fixed issue #135 (segfault in interned strings if initial memory is too low)
c02ff01
@php-pulls php-pulls pushed a commit that referenced this pull request Sep 28, 2013
Christopher Jones Merge branch 'master' of https://git.php.net/repository/php-src
# By Adam Harvey (2) and others
# Via Adam Harvey (2) and others
* 'master' of https://git.php.net/repository/php-src:
  Implement variadic function syntax
  Added function opcache_compile_file() to load PHP scripts into cache without execution.
  Fixed issue #135 (segfault in interned strings if initial memory is too low)
  Fix typo: HTTP_ROW_POST_DATA → HTTP_RAW_POST_DATA.
  Make message and format arguments const char * to avoid build warning about invalid cast.
  Copy dba_*() keys before converting to string.
a3b0fa0
@php-pulls php-pulls pushed a commit that referenced this pull request Sep 28, 2013
Christopher Jones Merge branch 'PHP-5.5' of https://git.php.net/repository/php-src into…
… PHP-5.5

# By Dmitry Stogov (2) and others
# Via Adam Harvey (1) and Dmitry Stogov (1)
* 'PHP-5.5' of https://git.php.net/repository/php-src:
  Added function opcache_compile_file() to load PHP scripts into cache without execution.
  Fixed issue #135 (segfault in interned strings if initial memory is too low)
  Make message and format arguments const char * to avoid build warning about invalid cast.
  Copy dba_*() keys before converting to string.
9a655fc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment