Bug #55634 [Com]: ./configure does not fail if invalid option is used

From: Date: Thu, 22 Sep 2011 19:07:27 +0000
Subject: Bug #55634 [Com]: ./configure does not fail if invalid option is used
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-163794@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=55634&edit=1

 ID:                 55634
 Comment by:         ashnazg@php.net
 Reported by:        cweiske@php.net
 Summary:            ./configure does not fail if invalid option is used
 Status:             Open
 Type:               Bug
 Package:            *Configuration Issues
 PHP Version:        5.3.8
 Block user comment: N
 Private report:     N

 New Comment:

Perhaps a BC compromise here would be to at least *also* echo to stderr, while leaving the existing
echo to stdout in place?


Previous Comments:
------------------------------------------------------------------------
[2011-09-07 22:28:17] fa@php.net

I'm not saying this behaviour is wrong or right, I've been bitten by this already myself,
but I do think that could be considered a BC break, as I remember it always being like that,
although I'm only 100% sure on 5.2+

------------------------------------------------------------------------
[2011-09-07 13:17:55] cweiske@php.net

Description:
------------
When using ./configure --with-foo, configure tells me at the end:
> Notice: Following unknown configure options were used:
> --with-foo

There are two problems:
- This problem is echoed to stdout, not stderr where capturing it would be possible
- The exit code is still 0, although I clearly issued a wrong option.

In the end I cannot figure out if the configure run was *fully* successful.

Expected result:
----------------
1. config option errors echoed to stderr
2. exit code of configure script != 0



------------------------------------------------------------------------



-- 
Edit this bug report at https://bugs.php.net/bug.php?id=55634&edit=1


Thread (8 messages)

« previous php.bugs (#163794) next »