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]. (From OE-Core rev: 75e3875341ddc8940e9ee2ccbbb2ec18194a68e6) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
333d571b36
commit
e043c9d1ce
|
@ -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'
|
||||
|
|
Loading…
Reference in New Issue