lmi-commits
[Top][All Lists]
Advanced

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

[lmi-commits] [lmi] master 0d8f53e 02/18: Resolve shellcheck "SC1117" wa


From: Greg Chicares
Subject: [lmi-commits] [lmi] master 0d8f53e 02/18: Resolve shellcheck "SC1117" warning
Date: Wed, 29 May 2019 18:49:06 -0400 (EDT)

branch: master
commit 0d8f53ef1d66cbedf3935e19b8af954b2e37a80a
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Resolve shellcheck "SC1117" warning
    
    Replaced double quotes with single quotes. Tested thus:
    
    zsh: unrecognized modifier
    1583
    
    This manpages.debian.org example:
      dpkg-query -W -f='${binary:Package} ${Version}\t${Maintainer}\n' dpkg
    uses single quotes.
---
 install_msw.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install_msw.sh b/install_msw.sh
index 4f4777f..0c15c89 100755
--- a/install_msw.sh
+++ b/install_msw.sh
@@ -146,7 +146,7 @@ then
     # Disable shellcheck warning about the need to double quote $packages_list:
     # it can't be done here and we really want word splitting to happen here.
     # shellcheck disable=SC2086
-    missing_packages_count=$(dpkg-query -W -f="\${Status}\n" $packages_list 
2>&1 | \
+    missing_packages_count=$(dpkg-query -W -f='\${Status}\n' $packages_list 
2>&1 | \
       grep -v -c 'install ok installed')
 
     if [ "$missing_packages_count" -gt 0 ]



reply via email to

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