diff mbox

tests: Ignore test-blockjob-txn

Message ID 1447353228-30550-1-git-send-email-eblake@redhat.com
State New
Headers show

Commit Message

Eric Blake Nov. 12, 2015, 6:33 p.m. UTC
Commit 6c6f312d added a new test, but did not mark it for
exclusion in .gitignore.

Signed-off-by: Eric Blake <eblake@redhat.com>
---

Yeah, I know this affects only non-VPATH builds, which we don't
recommend, but we might as well be consistent.

 tests/.gitignore | 1 +
 1 file changed, 1 insertion(+)

Comments

Stefan Hajnoczi Nov. 13, 2015, 2:27 a.m. UTC | #1
On Thu, Nov 12, 2015 at 11:33:48AM -0700, Eric Blake wrote:
> Commit 6c6f312d added a new test, but did not mark it for
> exclusion in .gitignore.
> 
> Signed-off-by: Eric Blake <eblake@redhat.com>
> ---
> 
> Yeah, I know this affects only non-VPATH builds, which we don't
> recommend, but we might as well be consistent.
> 
>  tests/.gitignore | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Eric Blake Nov. 13, 2015, 3:43 a.m. UTC | #2
On 11/12/2015 07:27 PM, Stefan Hajnoczi wrote:
> On Thu, Nov 12, 2015 at 11:33:48AM -0700, Eric Blake wrote:
>> Commit 6c6f312d added a new test, but did not mark it for
>> exclusion in .gitignore.
>>
>> Signed-off-by: Eric Blake <eblake@redhat.com>
>> ---
>>
>> Yeah, I know this affects only non-VPATH builds, which we don't
>> recommend, but we might as well be consistent.
>>
>>  tests/.gitignore | 1 +
>>  1 file changed, 1 insertion(+)
> 
> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

Another one popped up: bd797fc1 introduced test-timed-average. I'll just
respin a v2 of this, although the subject line will change.
Stefan Hajnoczi Nov. 13, 2015, 10:26 a.m. UTC | #3
On Thu, Nov 12, 2015 at 11:33:48AM -0700, Eric Blake wrote:
> Commit 6c6f312d added a new test, but did not mark it for
> exclusion in .gitignore.
> 
> Signed-off-by: Eric Blake <eblake@redhat.com>
> ---
> 
> Yeah, I know this affects only non-VPATH builds, which we don't
> recommend, but we might as well be consistent.

I'm a non-VPATH diehard too :)

>  tests/.gitignore | 1 +
>  1 file changed, 1 insertion(+)

Thanks, applied to my block tree:
https://github.com/stefanha/qemu/commits/block

Stefan
Eric Blake Nov. 13, 2015, 2:55 p.m. UTC | #4
On 11/13/2015 03:26 AM, Stefan Hajnoczi wrote:
> On Thu, Nov 12, 2015 at 11:33:48AM -0700, Eric Blake wrote:
>> Commit 6c6f312d added a new test, but did not mark it for
>> exclusion in .gitignore.
>>
>> Signed-off-by: Eric Blake <eblake@redhat.com>
>> ---
>>
>> Yeah, I know this affects only non-VPATH builds, which we don't
>> recommend, but we might as well be consistent.
> 
> I'm a non-VPATH diehard too :)
> 
>>  tests/.gitignore | 1 +
>>  1 file changed, 1 insertion(+)
> 
> Thanks, applied to my block tree:
> https://github.com/stefanha/qemu/commits/block

Did you want to apply v2 instead?
https://lists.gnu.org/archive/html/qemu-devel/2015-11/msg03214.html
diff mbox

Patch

diff --git a/tests/.gitignore b/tests/.gitignore
index e96f569..de5e793 100644
--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -9,6 +9,7 @@  check-qom-proplist
 rcutorture
 test-aio
 test-bitops
+test-blockjob-txn
 test-coroutine
 test-crypto-cipher
 test-crypto-hash