[pkg-wine-party] [wine] 10/24: Simplify the foreign-arch test in the wine script.

Jens Reyer jreyer-guest at moszumanska.debian.org
Wed Jan 11 23:14:42 UTC 2017


This is an automated email from the git hooks/post-receive script.

jreyer-guest pushed a commit to branch stretch
in repository wine.

commit d4c1ce070b5d30a4cb71eebead18371d3648e739
Author: Jens Reyer <jre.winesim at gmail.com>
Date:   Fri Oct 14 14:58:16 2016 +0200

    Simplify the foreign-arch test in the wine script.
    
    Using "&&" shortcircuits the test (if the first expression is
    false the second won't be evaluated). This isn't the case with
    bash and dash using "-a".
    The new version of the second expression makes it more efficient
    and intuitive.
    
    # Conflicts:
    #	debian/changelog
---
 debian/scripts/wine.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/scripts/wine.in b/debian/scripts/wine.in
index cf3e7df..b47b39c 100644
--- a/debian/scripts/wine.in
+++ b/debian/scripts/wine.in
@@ -5,7 +5,7 @@ wine64=/BINDIR/wine64
 
 wine32_hint () {
     echo "it looks like wine32VERSION is missing, you should install it."
-    if [ "$(dpkg --print-architecture)" = "amd64" -a "$(dpkg --print-foreign-architectures | grep -cx "i386")" -ne 1 ]; then
+    if [ "$(dpkg --print-architecture)" = "amd64" ] && ! dpkg --print-foreign-architectures | grep -qx "i386"; then
         echo "multiarch needs to be enabled first.  as root, please"
         echo "execute \"dpkg --add-architecture i386 && apt-get update &&"
         echo "apt-get install wine32VERSION\""

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-wine/wine.git



More information about the pkg-wine-party mailing list