[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: |
Kevin Wolf |
Subject: |
Re: [PATCH v2 1/4] qemu-iotests: remove bash shebang from library files |
Date: |
Fri, 11 Oct 2019 22:30:46 +0200 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
Am 11.10.2019 um 22:05 hat Cleber Rosa geschrieben:
> On Fri, Oct 11, 2019 at 02:27:25PM +0300, Nir Soffer wrote:
> > On Fri, Oct 11, 2019, 12:36 Kevin Wolf <address@hidden> wrote:
> >
> > > Am 09.10.2019 um 21:47 hat Cleber Rosa geschrieben:
> > > > 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.
> > >
> > > vim still doesn't like the change.
> > >
> > > Of course, we could also add another line for vim and for every other
> > > editor in use, but actually, I think I'd prefer just dropping this
> > > patch. It even makes each file a few bytes larger instead of saving
> > > something. Shebang lines are a shorter and more portable format
> > > indicator than the alternatives.
> > >
> > > So I think in the end we have found a good reason to keep them. :-)
> >
> > What about .sh suffix? Should be most portable way.
>
> That's the approach I tend to follow for my sh code. Explicit is
> better than implicit if you ask me.
I would certainly agree for new files.
> Kevin,
>
> Do you have any strong feelings here? I'd be fine with either this
> or dropping the patch.
No strong feelings. The result of renaming the files would be a bit
nicer than what we have today, but renaming always comes with a cost
when working with the version history later. Hard to tell if it's a net
gain or loss in the end.
Myself, I would probably pick the lazy way and stick with "if it ain't
broke, don't fix it", but I'm not objecting to a change either.
Kevin
[PATCH v2 2/4] qemu-iotests: remove forceful execution success from library files, Cleber Rosa, 2019/10/09
[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