[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/4] qemu-iotests: remove bash shebang from library files
From: |
Cleber Rosa |
Subject: |
Re: [PATCH v2 1/4] qemu-iotests: remove bash shebang from library files |
Date: |
Wed, 9 Oct 2019 16:54:00 -0400 (EDT) |
----- Original Message -----
> From: "Eric Blake" <address@hidden>
> To: "Cleber Rosa" <address@hidden>, address@hidden
> Cc: address@hidden, "Max Reitz" <address@hidden>, "Kevin Wolf"
> <address@hidden>, address@hidden,
> "Michael Tokarev" <address@hidden>, "Laurent Vivier" <address@hidden>
> Sent: Wednesday, October 9, 2019 3:51:56 PM
> Subject: Re: [PATCH v2 1/4] qemu-iotests: remove bash shebang from library
> files
>
> On 10/9/19 2:47 PM, Cleber Rosa wrote:
> > Due to not being able to find a reason to have shebangs on files that
> > are not executable.
> >
> > While at it, add a mode hint to emacs, which would be clueless or
> > plain wrong about these containing shell code.
> >
> > Suggested-by: Eric Blake <address@hidden>
> > Signed-off-by: Cleber Rosa <address@hidden>
> > ---
> > tests/qemu-iotests/common.config | 2 +-
> > tests/qemu-iotests/common.filter | 2 +-
> > tests/qemu-iotests/common.nbd | 3 +--
> > tests/qemu-iotests/common.pattern | 2 +-
> > tests/qemu-iotests/common.qemu | 2 +-
> > tests/qemu-iotests/common.rc | 2 +-
> > tests/qemu-iotests/common.tls | 2 +-
> > 7 files changed, 7 insertions(+), 8 deletions(-)
> >
> > diff --git a/tests/qemu-iotests/common.config
> > b/tests/qemu-iotests/common.config
> > index 9bd1a5a6fc..b85a6a6f96 100644
> > --- a/tests/qemu-iotests/common.config
> > +++ b/tests/qemu-iotests/common.config
> > @@ -1,4 +1,4 @@
> > -#!/usr/bin/env bash
> > +# -*- emacs mode: sh -*-
>
> I thought my version:
> # hey emacs, this file will be sourced by bash -*- mode: sh -*-
> was cuter, but that's not a requirement, and yours works ;)
>
I will not make any disputes on that! :)
Thanks for the reviews!
- Cleber.
> Reviewed-by: Eric Blake <address@hidden>
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3226
> Virtualization: qemu.org | libvirt.org
>
[PATCH v2 3/4] qemu-iotests: 044: pass is actually a noop, so remove it, Cleber Rosa, 2019/10/09
[PATCH v2 4/4] qemu-iotests: 044: remove inaccurate docstring class description, Cleber Rosa, 2019/10/09