From c4a0b5463d1fe7cbc605ae95e00d53cbc14903c2 Mon Sep 17 00:00:00 2001 From: Joshua Lock Date: Mon, 7 Jun 2010 16:10:48 +0100 Subject: sanity.bbclass: version each of the files in the build/conf/ directory We already have versioning of the local.conf file but it's entirely plausible for the bblayers.conf and site.conf to change incompatibly so we should version those too. Signed-off-by: Joshua Lock --- meta/classes/sanity.bbclass | 21 +++++++++++++++++---- meta/conf/sanity.conf | 4 +++- 2 files changed, 20 insertions(+), 5 deletions(-) (limited to 'meta') diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass index ab929ad61d..8ed8975edd 100644 --- a/meta/classes/sanity.bbclass +++ b/meta/classes/sanity.bbclass @@ -54,12 +54,25 @@ def check_sanity(e): if data.getVar('TARGET_OS', e.data, True) == 'INVALID': messages = messages + 'Please set TARGET_OS directly, or choose a MACHINE or DISTRO that does so.\n' - # Check we are using a valid conf setup - current_conf = data.getVar('CONF_VERSION', e.data, True) - conf_version = data.getVar('POKY_CONF_VERSION', e.data, True) + # Check we are using a valid lacal.conf + current_conf = data.getVar('CONF_VERSION', e.data, True) + conf_version = data.getVar('POKY_CONF_VERSION', e.data, True) if current_conf != conf_version: - messages = messages + "Poky has noticed your version of local.conf was generated from an older version of local.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/local.conf conf/local.conf.sample\" is a good way to visualise the changes" + messages = messages + "Poky has noticed your version of local.conf was generated from an older version of local.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/local.conf conf/local.conf.sample\" is a good way to visualise the changes.\n" + + # Check bblayers.conf is valid + current_lconf = data.getVar('LCONF_VERSION', e.data, True) + lconf_version = data.getVar('LAYER_CONF_VERSION', e.data, True) + if current_lconf != lconf_version: + messages = messages + "Poky has noticed your version of bblayers.conf was generated from an older version of bblayers.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/bblayers.conf conf/bblayers.conf.sample\" is a good way to visualise the changes.\n" + + # If we have a site.conf, check it's valid + if check_conf_exists("conf/site.conf", e.data): + current_sconf = data.getVar('SCONF_VERSION', e.data, True) + sconf_version = data.getVar('SITE_CONF_VERSION', e.data, True) + if current_sconf != sconf_version: + messages = messages + "Poky has noticed your version of site.conf was generated from an older version of site.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.\nMatching the version numbers will remove this message.\n\"meld conf/site.conf conf/site.conf.sample\" is a good way to visualise the changes.\n" assume_provided = data.getVar('ASSUME_PROVIDED', e.data , True).split() # Check user doesn't have ASSUME_PROVIDED = instead of += in local.conf diff --git a/meta/conf/sanity.conf b/meta/conf/sanity.conf index c9d77794cf..abe5eee01e 100644 --- a/meta/conf/sanity.conf +++ b/meta/conf/sanity.conf @@ -7,7 +7,9 @@ BB_MIN_VERSION = "1.8.10" SANITY_ABIFILE = "${TMPDIR}/abi_version" -POKY_CONF_VERSION = "1" +POKY_CONF_VERSION = "1" +LAYER_CONF_VERSION = "1" +SITE_CONF_VERSION = "1" INHERIT += "sanity" -- cgit v1.2.3-54-g00ecf