From: Martin Jansa Date: Fri, 1 Mar 2013 17:42:22 +0000 (+0100) Subject: sanity.bbclass: always use oe.lsb.distro_identifier method X-Git-Tag: rev_ivi_2015_02_04~13519 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ac8440e35ab482162d7242540611c9a6ae6e2663;p=scm%2Fbb%2Ftizen-distro.git sanity.bbclass: always use oe.lsb.distro_identifier method * even when /etc/redhat-release or /etc/SuSE-release exists * don't read /etc/lsb-release manually, NATIVELSBSTRING is not reading it too (From OE-Core rev: 3dba05d74821f08cd96a18351805c8812ddd626c) Signed-off-by: Martin Jansa Signed-off-by: Richard Purdie --- diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass index 94c6ce3..06e95e3 100644 --- a/meta/classes/sanity.bbclass +++ b/meta/classes/sanity.bbclass @@ -263,39 +263,11 @@ def check_supported_distro(sanity_data): if not tested_distros: return - if os.path.exists("/etc/redhat-release"): - f = open("/etc/redhat-release", "r") - try: - distro = f.readline().strip() - finally: - f.close() - elif os.path.exists("/etc/SuSE-release"): - import re - 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 - try: - distro = oe.lsb.distro_identifier() - except Exception: - distro = None + try: + distro = oe.lsb.distro_identifier() + 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)