php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
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
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: Tue Dec 10 14:01:27 2024 UTC