[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH 03/10] qemu-iotests: make 218 exe
|
From: |
Cleber Rosa |
|
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH 03/10] qemu-iotests: make 218 executable |
|
Date: |
Fri, 5 Oct 2018 13:49:57 -0400 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.0 |
On 10/5/18 9:36 AM, Philippe Mathieu-Daudé wrote:
> On 04/10/2018 18:18, Cleber Rosa wrote:
>> Commit 990dc39c made all tests executable at the time, but 218 came in
>> later, and missing those permissions.
>>
>> Signed-off-by: Cleber Rosa <address@hidden>
>> ---
>> tests/qemu-iotests/218 | 0
>> 1 file changed, 0 insertions(+), 0 deletions(-)
>> mode change 100644 => 100755 tests/qemu-iotests/218
>>
>> diff --git a/tests/qemu-iotests/218 b/tests/qemu-iotests/218
>> old mode 100644
>> new mode 100755
>
> We could add a git pre-commit hook grep'ing for "^#!\
> *(/usr/bin/env|/bin/(b?a)?sh)" in the 1st line and then checking the
> file perms.
>
Good idea. Probably better to send that as a different patch, right?
[Qemu-trivial] [PATCH 06/10] thunk.c: clean up commented out definition, Cleber Rosa, 2018/10/04
[Qemu-trivial] [PATCH 05/10] docs/devel/testing.rst: add missing newlines after code block, Cleber Rosa, 2018/10/04