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

go to bug id or search bugs for

Bug #72987 configure: sed problem with "clean_configure_args"
Submitted: 2016-08-31 08:47 UTC Modified: 2017-01-09 05:12 UTC
Votes:1
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:1 (100.0%)
Same OS:1 (100.0%)
From: lzsiga at freemail dot c3 dot hu Assigned: krakjoe (profile)
Status: Closed Package: Compile Warning
PHP Version: 7.0.10 OS: unix
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: lzsiga at freemail dot c3 dot hu
New email:
PHP Version: OS:

 

 [2016-08-31 08:47 UTC] lzsiga at freemail dot c3 dot hu 
Description:
------------
Hi,

For some reason, script 'configure' tries to clean its command-line with 'sed' (line 3451). Doing that, character /slash is used as sed-separator. Problem is variables CPPFLAGS, LDFLAGS and others might legally contain /slashes, so 'sed' will fail.
I suggest using sed once more: s;/;\\/;g (see the patch)

Test script:
---------------
CPPFLAGS='-I/usr/local/include'
./configure

Expected result:
----------------
lines like this should not be printed:

sed: Function s/'CPPFLAGS=-I/usr/local/include'// cannot be parsed.

Patches

configure.patch (last revision 2016-08-31 08:48 UTC by lzsiga at freemail dot c3 dot hu)

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-12-12 11:54 UTC] lzsiga at freemail dot c3 dot hu 
Oh, I see, it is (almost-)solved in 7.0.14
 [2017-01-09 05:12 UTC] krakjoe@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: krakjoe
 [2017-01-09 05:12 UTC] krakjoe@php.net 
The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.

I think this was resolved, in any case, the patch should not target configure script ;)
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Sat Dec 21 18:01:29 2024 UTC