|
From: | Eric Blake |
Subject: | Re: [PATCH 1/4] qemu-iotests: remove bash shebang from library files |
Date: | Fri, 27 Sep 2019 11:37:52 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 |
On 9/27/19 9:17 AM, Cleber Rosa wrote:
Due to not being able to find a reason to have shebangs on files that are not executable. Signed-off-by: Cleber Rosa <address@hidden> --- tests/qemu-iotests/common.config | 2 -- tests/qemu-iotests/common.filter | 2 -- tests/qemu-iotests/common.nbd | 1 - 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, 13 deletions(-)
Loss of the shebang changes the mode in which emacs opens the files (from Shell-script[bash] to Conf[space] in my case). I agree that a #! comment is not appropriate for a file that is not executable as a standalone file, but it becomes harder to edit the file correctly unless we replace it with some other way of letting editors realize that the contents of each file is still meant to be consumed by bash.
Something like this would work: # hey emacs, this file will be sourced by bash: -*- sh -*- -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
[Prev in Thread] | Current Thread | [Next in Thread] |