summaryrefslogtreecommitdiff
path: root/meta/classes/sanity.bbclass
diff options
context:
space:
mode:
authorPaul Eggleton <paul.eggleton@linux.intel.com>2011-08-17 18:48:15 +0100
committerSaul Wold <sgw@linux.intel.com>2011-08-18 13:45:43 -0700
commit75e3875341ddc8940e9ee2ccbbb2ec18194a68e6 (patch)
treeabb49821123353a7490d408233042524c0bb6548 /meta/classes/sanity.bbclass
parent4316c5e9eec67d71f4699102a63a5f968bd8d1d9 (diff)
downloadopenembedded-core-75e3875341ddc8940e9ee2ccbbb2ec18194a68e6.tar.gz
openembedded-core-75e3875341ddc8940e9ee2ccbbb2ec18194a68e6.tar.bz2
openembedded-core-75e3875341ddc8940e9ee2ccbbb2ec18194a68e6.tar.xz
openembedded-core-75e3875341ddc8940e9ee2ccbbb2ec18194a68e6.zip
sanity.bbclass: add optional untested host distro warning
SANITY_TESTED_DISTROS, if specified, is expected to be a newline-delimited list of distro identifier strings, e.g. SANITY_TESTED_DISTROS = " \ Ubuntu 11.04 \ Fedora release 14 (Laughlin) \ " (spaces, tabs etc. are trimmed) If SANITY_TESTED_DISTROS is defined, we will attempt to detect the host distribution. If the distribution is not in SANITY_TESTED_DISTROS or we could not detect the distribution then we show a warning during sanity checking. Provides the mechanism for fixing [YOCTO #1096]. Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Diffstat (limited to 'meta/classes/sanity.bbclass')
-rw-r--r--meta/classes/sanity.bbclass48
1 files changed, 48 insertions, 0 deletions
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index d50c8434a..16af0293d 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -118,6 +118,52 @@ def check_connectivity(d):
return retval
+def check_supported_distro(e):
+ tested_distros = e.data.getVar('SANITY_TESTED_DISTROS', True)
+ if not tested_distros:
+ return
+
+ if os.path.exists("/etc/redhat-release"):
+ f = open("/etc/redhat-release", "r")
+ try:
+ distro = f.readline()
+ finally:
+ f.close()
+ elif os.path.exists("/etc/SuSE-release"):
+ f = open("/etc/SuSE-release", "r")
+ try:
+ distro = f.readline()
+ # Remove the architecture suffix e.g. (i586)
+ distro = re.sub(r' \([a-zA-Z0-9\-_]*\)$', '', distro).strip()
+ finally:
+ f.close()
+ else:
+ # Use LSB method
+ import subprocess as sub
+ try:
+ p = sub.Popen(['lsb_release','-d','-s'],stdout=sub.PIPE,stderr=sub.PIPE)
+ out, err = p.communicate()
+ distro = out.rstrip()
+ except Exception:
+ distro = None
+
+ if not distro:
+ if os.path.exists("/etc/lsb-release"):
+ f = open("/etc/lsb-release", "r")
+ try:
+ for line in f:
+ lns = line.split('=')
+ if lns[0] == "DISTRIB_DESCRIPTION":
+ distro = lns[1].strip('"\n')
+ break
+ finally:
+ f.close()
+ if distro:
+ if distro not in [x.strip() for x in tested_distros.split('\n')]:
+ bb.warn('Host distribution "%s" has not been validated with this version of the build system; you may possibly experience unexpected failures. It is recommended that you use a tested distribution.' % distro)
+ else:
+ bb.warn('Host distribution could not be determined; you may possibly experience unexpected failures. It is recommended that you use a tested distribution.')
+
def check_sanity(e):
from bb import note, error, data, __version__
@@ -249,6 +295,8 @@ def check_sanity(e):
if pseudo_msg != "":
messages = messages + pseudo_msg + '\n'
+ check_supported_distro(e)
+
# Check if DISPLAY is set if IMAGETEST is set
if not data.getVar( 'DISPLAY', e.data, True ) and data.getVar( 'IMAGETEST', e.data, True ) == 'qemu':
messages = messages + 'qemuimagetest needs a X desktop to start qemu, please set DISPLAY correctly (e.g. DISPLAY=:1.0)\n'