guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#51364: [PATCH v2] gnu: emacs-flymake-shellcheck: Add shellcheck as a


From: Efraim Flashner
Subject: bug#51364: [PATCH v2] gnu: emacs-flymake-shellcheck: Add shellcheck as an input
Date: Thu, 4 Nov 2021 15:05:19 +0200

On Tue, Nov 02, 2021 at 07:13:10PM -0400, Morgan.J.Smith@outlook.com wrote:
> From: Morgan Smith <Morgan.J.Smith@outlook.com>
> 
> * gnu/packages/emacs-xyz.scm (emacs-flymake-shellcheck)
> [inputs]: Add shellcheck.
> [arguments]: Add substitute-shellcheck-path phase.
> ---
> 
> Sorry for the late reply, I didn't see this in my email.
> 
> People would still have been able to set the variable from their 
> configuration.  The value I'm editing just determines the default value.  
> However, I think I came up with a nice solution that should make everyone 
> happy.  It first checks if there is shellcheck in your path and only hard 
> codes the path if you don't have it.  Apparently Emacs even re-evaluates 
> defcustoms in certain situations so this *might* even work if someone 
> installs shellcheck after this file has been loaded.

Looks good to me. Patch pushed.

You also might've missed the email because my self-hosted email server
sometimes gets blocked. I've added you to X-Debbugs-CC so you should get
this email directly.

-- 
Efraim Flashner   <efraim@flashner.co.il>   רנשלפ םירפא
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]