[gnome-continuous-yocto/gnomeostree-3.28-rocko: 5117/8267] sanity.bbclass: modify check for shell



commit ab2caed31ad7d20561643f4de9792f97d3a6bd5b
Author: Juro Bystricky <juro bystricky intel com>
Date:   Wed Mar 15 09:37:39 2017 -0700

    sanity.bbclass: modify check for shell
    
    Due to the recently implemented update-alternatives for bash binary,
    sanity checker may end up with a (false-positive) error such as:
    
    Error, /bin/sh links to /bin/bash.bash, must be dash or bash
    
    This patch modifies the test: presence of "/bash" or "/dash" in shell binary
    name results in pass.
    
    [YOCTO#11108]
    
    (From OE-Core rev: ef51746cdd12e6b08109e9bd90a0a465c3f9f93c)
    
    Signed-off-by: Juro Bystricky <juro bystricky intel com>
    Signed-off-by: Richard Purdie <richard purdie linuxfoundation org>

 meta/classes/sanity.bbclass |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
---
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index b5f4756..e3be40b 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -935,7 +935,9 @@ def check_sanity_everybuild(status, d):
     # If /bin/sh is a symlink, check that it points to dash or bash
     if os.path.islink('/bin/sh'):
         real_sh = os.path.realpath('/bin/sh')
-        if not real_sh.endswith('/dash') and not real_sh.endswith('/bash'):
+        # Due to update-alternatives, the shell name may take various
+        # forms, such as /bin/dash, bin/bash, /bin/bash.bash ...
+        if '/dash' not in real_sh and '/bash' not in real_sh:
             status.addresult("Error, /bin/sh links to %s, must be dash or bash\n" % real_sh)
 
 def check_sanity(sanity_data):


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