[PATCH] Remove shebang from fish completion
Dan Čermák
dcermak at suse.de
Tue Jun 15 06:17:07 UTC 2021
Johannes Altmanninger <aclopte at gmail.com> writes:
> On Mon, Jun 14, 2021 at 11:03:48PM +0200, Dan Čermák wrote:
>> Afaik fish shell completions don't need a shebang
>> (plus the script is not executable anyway)
>
> Yeah, that is the convention - I didn't make this change because it breaks
> syntax highlighting in my editor.
> This is because my editor recognizes fish filetypes based on
> 1. file extension (".fish")
> - but "pass.fish-completion" does not follow that convention
> 2. output of "file --mime"
> - without the shebang, file(1) no longer recognizes that this is a shell script.
Apologies, I was not aware that this would be a potential fallout. If
the modelines are not really viable alternatives, then my patch can be
reverted, as it does not really fix anything.
Cheers,
Dan
More information about the Password-Store
mailing list