[prev in list] [next in list] [prev in thread] [next in thread]
List: qemu-commits
Subject: [Qemu-commits] [qemu/qemu] fb59da: configure: Don't claim 'unsupported host OS' when ...
From: GitHub <noreply () github ! com>
Date: 2017-03-30 13:00:11
Message-ID: 58dd015b5158b_29e53ff0e1883c24266e8 () hookshot-fe5-cp1-prd ! iad ! github ! net ! mail
[Download RAW message or body]
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: fb59dabd4fa7e6586824ac3012073b943fc8dc79
https://github.com/qemu/qemu/commit/fb59dabd4fa7e6586824ac3012073b943fc8dc79
Author: Peter Maydell <peter.maydell@linaro.org>
Date: 2017-03-30 (Thu, 30 Mar 2017)
Changed paths:
M configure
Log Message:
-----------
configure: Don't claim 'unsupported host OS' when better message available
The change in commit 898be3e0415c6d which made completely
unrecognized OSes cause an error_exit "Unsupported host OS"
has some unfortunate unintended effects:
* if you run 'configure --help' on an unsupported host OS
(eg if intending to use it as a build machine for a
cross compile to a supported host) then the message
is printed instead of --help
* if the C compiler doesn't work or is missing (eg if
you passed an incorrect --cross-prefix by mistake)
the message is printed instead of the more useful
'compiler does not exist or does not work' message
Fix this by postponing the error_exit in this situation
until later, when we have already identified the more
useful cases for this.
The long term fix for this would be to move handling
of --help much further up in the configure script,
and make its output not dependent on checks that configure
runs. However for 2.9 this would be too invasive.
Reported-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Stefan Weil <sw@weilnetz.de>
Tested-by: Stefan Weil <sw@weilnetz.de>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic