|Subject:||Re: [Qemu-devel] [PATCH v3 3/7] iotests: Add default common.env|
|Date:||Sat, 24 May 2014 21:40:10 +0200|
|User-agent:||Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0|
On 23.05.2014 15:33, Markus Armbruster wrote:
Eric Blake <address@hidden> writes:On 05/23/2014 12:10 AM, Fam Zheng wrote:On Thu, 05/22 23:30, Max Reitz wrote:Add a default common.env in case the one supposed to be emitted by configure cannot be found. Signed-off-by: Max Reitz <address@hidden>Now we can run both from in-tree build and out-of-tree build without this, which is already very good, but we need this patch to run from the source tree with an out-of-tree build. I'm not sure we really need that, though.If you have a source tree but no in-tree build, then there is no way to know how many out-of-tree builds there are, or which one you should pick. If you are going to do out-of-tree builds, you aren't going to try to run tests from the source tree.Let's not try to guess a build tree. If someone tries to run tests in the source tree, and the source tree has not been made a build tree by running configure there, simply fail.
Okay, as you and Fam (and probably also Eric) seem to agree on this, I'll drop this patch and try to make the error message for failing to include common.env more explanatory ("make sure the iotests are run from the build tree" or something similar).
|[Prev in Thread]||Current Thread||[Next in Thread]|