[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 3/7] iotests: Add default common.env

From: Max Reitz
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).


reply via email to

[Prev in Thread] Current Thread [Next in Thread]