summaryrefslogtreecommitdiffstats
path: root/meta/classes/sanity.bbclass
Commit message (Collapse)AuthorAgeFilesLines
* bitbake.conf/sanity: Separate versions and PN stamp components into separate ↵Richard Purdie2012-11-201-0/+2
| | | | | | | | | | | | | | | directories for WORKDIR and STAMP This means some of the hacks we have to tell where the package name ends and the version starts in the directory layout becomes obsolete, simplifying the work of some of the cleanup scripts. It also makes the layout slightly more intuitive to the user. It does force a rebuild onto the user but it will reuse sstate successfully. (From OE-Core rev: 05075cf3138d1c61f5cf4fe0e1a4587acc00c692) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: trigger network tests explicitlyBogdan Marinescu2012-10-181-0/+3
| | | | | | | | | | | | The network tests in sanity.bbclass can now be trigerred explicitly by firing the NetworkTest event. This is part of the fix for bug #3026. [YOCTO #3026] (From OE-Core rev: f1f43d55dbb020a0145c58731d4259fd906d9d1e) Signed-off-by: Bogdan Marinescu <bogdan.a.marinescu@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Fix invalid test for network errorBogdan Marinescu2012-10-021-1/+1
| | | | | | | | | The test for network error in sanity.bbclass was negated. (From OE-Core rev: 9fcd0866f0e30a50182434f6bcae13bf9575807f) Signed-off-by: Bogdan Marinescu <bogdan.a.marinescu@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity: Added explicit network error status in SanityCheckFailed eventBogdan Marinescu2012-09-281-7/+13
| | | | | | | | | | | | | | If we fail a network test, a special flag is set in the SanityChekFailed event. This helps Hob identify the network error properly and display a special message to the user. [YOCTO #3025] (From OE-Core rev: 7877c4344db89237bba5f9a03342bfd9a03aebbf) Signed-off-by: Bogdan Marinescu <bogdan.a.marinescu@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: bblayers.conf should be updated automaticallyConstantin Musca2012-09-281-1/+60
| | | | | | | | | | | | | | | | - add check_bblayers_conf bitbake function which does the bblayers.conf v4 -> v5 update if necessary (every layer should make its specific bblayers.conf upgrades appending to the check_bblayers_conf function) - we ask the user to re-run bitbake because we can't trigger reparsing without being invasive [YOCTO #3082] (From OE-Core rev: 03ad4edace5db9c6e15ca776d06d20b7d4e42afc) Signed-off-by: Constantin Musca <constantinx.musca@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: remove obsolete codePaul Eggleton2012-09-121-7/+2
| | | | | | | | | | | | We can now rely upon the minimum BitBake version having the SanityCheckFailed event, so remove the code to handle if this is not there. (From OE-Core rev: ba17572c9c11efb45a92ba97914ce1f6d84002c8) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: skip tune checks if machine is invalidPaul Eggleton2012-09-121-3/+7
| | | | | | | | | | | If there is no valid machine configuration it's almost guaranteed that the tune checks will fail, so just suppress them in that case. (From OE-Core rev: 629c585e687cda9290efcffd18dd92fdf16009ab) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Move back to running at ConfigParsed timeRichard Purdie2012-09-051-5/+5
| | | | | | | | | | | | If we don't do this, users can get extremely confused errors since the sanity tests happen too late (after parsing) and don't see the warnings. Also cleanup messages about merging configuration file changes to give better hints at where the updated files may be. (From OE-Core rev: 7b8a68cda7ef8186e834b39e73ee12a55b33f85b) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* meta/classes: Various python whitespace fixesRichard Purdie2012-08-211-2/+2
| | | | | | | | | | | It was pointed out we have a number of weird indentations in the python functions. This patch cleans up 3, 7 and other weird indentations for the core bbclass files. It also fixes some wierd (odd) shell function indentation which my searches picked up. (From OE-Core rev: 8385d6d74624000d68814f4e3266d47bc8885942) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: remove texi2html from required host utilitiesPaul Eggleton2012-08-201-1/+1
| | | | | | | | | | | | | | | texi2html is not actually required to build world of OE-Core anymore, so we don't need to mandate it. The only difference without it (verified with buildhistory) is that groff doesn't produce HTML documentation; the rest of its docs are still produced and packaged and no other packages are affected. Part of the work towards [YOCTO #2423]. (From OE-Core rev: aa1c4519f8e549b43a7d8bc51c5342d3409b464b) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity: display the parsed BBPATH when complaining about CWD elementsRoss Burton2012-07-201-0/+1
| | | | | | | | | | Many people don't understand the nuances of PATH/BBPATH, so help them by clarifying the warning and displaying the parsed list element-by-element. (From OE-Core rev: 6f7be2498134402b5e0f766df5c57e1f496d0d06) Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* Remove a number of unneeded import os/bb callsRichard Purdie2012-07-191-5/+2
| | | | | | | | | | The bb and os modules are always imported so having these extra import calls are a waste of space/execution time. They also set a bad example for people copy and pasting code so clean them up. (From OE-Core rev: 7d674820958be3a7051ea619effe1a6061d9cbe2) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity: display the parsed PATH when complaining about CWD elementsRoss Burton2012-07-181-1/+2
| | | | | | | | | | Many people don't understand the nuances of PATH, so help them by clarifying the warning and displaying the parsed PATH element-by-element. (From OE-Core rev: 47e6ab41f0f84e6bed7d337ebf96c954ec6909e8) Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* bitbake.conf: Add git-native to ASSUME_PROVIDEDRichard Purdie2012-07-181-1/+1
| | | | | | | | | | | | | | | Originally, git was something new, not installed everywhere and had commandline stability problems. This has changed and git it no longer makes sense to continually build this when the system installed version is likely sufficient. This speeds up build since recipes no longer have to wait for git-native to build if they're fetched from a git:// SRC_URI. Also add git to the sanity checks and drop the no unneeded svn reference. (From OE-Core rev: 79e24186481770181565a18d177584d0d72399fe) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: the tmpdir can't be longer than 410Robert Yang2012-07-091-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There will be errors when the length of the tmpdir is longer than 410: 1) Longer than 420: Can't exec "/bin/sh": Argument list too long at /usr/lib/perl/5.10/IO/File.pm line 66. This error happens on both Ubuntu 10.04 and 10.10 when the pkg needs run "autoreconf", this is because it passes many files with absolute path to aclocal, aclocal passes them to perl, this is a limitation of the perl on Ubuntu 10.04 and 10.10, and the perl-native is not ready at this very early stage. 2) Longer than 490: bitbake/lib/bb/persist_data.py", line 197, in connect(database=...) > return sqlite3.connect(database, timeout=5, isolation_level=None) OperationalError: unable to open database file This error happens on Ubuntu 10.04, 10.10 and Fedora 17. This is because the length of the database in sqlite3 module (host's) can't be longer than 490 (or little smaller). The python-native is not ready at this very early stage. The 2 errors are host related, I think that limit the length of the TMPDIR to 410 is OK for most of the build, rarely build sets TMPDIR's longer than 410. [YOCTO #2434] (From OE-Core rev: ebcf949853ff667478a1ea1d3f1f8f41d643e708) Signed-off-by: Robert Yang <liezhi.yang@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: warn the user if BBPATH contains wrong entriesLaurentiu Palcu2012-06-131-0/+11
| | | | | | | | | | | | If BBPATH references the working directory, the user is warned and asked to fix the problem. [Yocto #1465] (From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184) Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: fix a couple of grammatical errors in messagesPaul Eggleton2012-05-301-2/+2
| | | | | | | (From OE-Core rev: 004ba67298e3e6e618df29597e9166c971a1941c) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: send sanity check failure as a separate event for HobPaul Eggleton2012-05-301-3/+13
| | | | | | | | | | | | | | | | | | In order to show a friendlier error message within Hob that does not bury the actual sanity error in our typical preamble about disabling sanity checks, use a separate event to indicate that sanity checks failed. This change is intended to work together with the related change to BitBake, however it has a check to ensure that it does not fail with older versions that do not include that change. Fixes [YOCTO #2336]. (From OE-Core rev: 49d8b7b8c3b18da64583637db207f1f064a5bdb2) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: remove broken TARGET_ARCH checkPaul Eggleton2012-05-301-4/+0
| | | | | | | | | | | | | The variable name has been typo'd as TARGE_ARCH since it was introduced some time ago, so the check has never worked. Fixing the typo shows that the test is not quite functional, so let's just remove it: http://lists.linuxtogo.org/pipermail/openembedded-core/2011-October/010613.html (From OE-Core rev: 897a9b1dddef385253b16ea7c193483e0ea679b6) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* meta: replace os.system with subprocess.callRobert Yang2012-05-301-6/+7
| | | | | | | | | | | | | | | Replace os.system with subprocess.call since the older function would fail (more or less) silently if the executed program cannot be found More info: http://docs.python.org/library/subprocess.html#subprocess-replacements [YOCTO #2454] (From OE-Core rev: a07d03cc6f67c88feb9813ae7deb6e4a93552dfe) Signed-off-by: Robert Yang <liezhi.yang@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: check sanity at BuildStarted rather than ConfigParsedJoshua Lock2012-05-241-1/+1
| | | | | | | | | | This enables a user to use bitbake -e even when the sanity checks are failing. (From OE-Core rev: be317df78535120137ecaadf797e3f4dfe04119e) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: catch an extra exception in check_create_long_filenameJoshua Lock2012-05-241-0/+2
| | | | | | | | | | The call to bb.mkdirhier() in check_create_long_filename() can fail with an OSError, explicitly catch this and report something useful to the user. (From OE-Core rev: b066906477eb0496a2babb3d8e87682a1b7df0de) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: add extra information when SSTATE_CACHE unusableJoshua Lock2012-05-241-0/+7
| | | | | | | | | | | | If the user does not have write permissions to SSTATE_CACHE, detected by the check_create_long_filename() test failing with a "Permission denied" value in strerror, then suggest they might want to use the location as an entry in SSTATE_MIRRORS. (From OE-Core rev: 719d44305508c75a9f2decacdff6558ca14277bb) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: add newline to check_create_long_filename failure messageJoshua Lock2012-05-241-1/+1
| | | | | | | | | Each failure in the sanity message should be reported on a new line. (From OE-Core rev: 9362702aa9dd82fed7e6d6e3cb1289521b802075) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: data.getVar(VAR, obj, exp) -> obj.getVar(VAR, exp)Joshua Lock2012-05-241-30/+30
| | | | | | | | | | Replace calls to data.getVar(VARIABLE, data_object, expand) to direct calls to the getVar method the the data_object. (From OE-Core rev: 41ee978b62a9c40f36f8ad0acef147e36edffa17) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: copy the data store and finalise before running checksJoshua Lock2012-05-241-60/+69
| | | | | | | | | | | | | At the ConfigParsed event the datastore has yet to be finalised and thus appends and overrides have not been set. To ensure the sanity check is being run against the configuration values the user has set call finalize() on a copy of the datastore and pass that for all sanity checks. (From OE-Core rev: 71142172efc0f44a50216550c2b6cc3094fdc21d) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* Revert "sanity.bbclass: check user can read and write to SSTATE_DIR"Joshua Lock2012-05-241-14/+3
| | | | | | | | | | This has caused problems for several users, including the Yocto Project autobuilder. Since the message was added in order to be more user friendly revert the change. This reverts commit 0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c. Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: check user can read and write to SSTATE_DIRJoshua Lock2012-05-171-3/+14
| | | | | | | | | | | The user needs read and write permissions to SSTATE_DIR, check whether they have sufficient permissions and if not recommend use of SSTATE_MIRRORS. (From OE-Core rev: 0c0c4efbf92bcf0f8942f17c18525a4b4ed1798c) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Detect empty $PATH components tooPeter Seebach2012-05-111-2/+3
| | | | | | | | | | | Empty components in $PATH have the same effect as a . in $PATH, and are a common side-effect of inserting a misspelled or unset shell variable in $PATH. (From OE-Core rev: ac4201b714c83c614113bfa735d0a2fc6f64db99) Signed-off-by: Peter Seebach <peter.seebach@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Implement initial toolchain sanity checksPeter Seebach2012-05-091-0/+73
| | | | | | | | | | | | | | | | | | | | This introduces a sanity check for the toolchain, which verifies each tuning (including any multilibs), producing meaningful diagnostics for problems, and also provides some higher-level tuning features. The TUNEVALID and TUNECONFLICT/TUNECONFLICTS settings were not implemented. Listed one or two missing features in TUNEVALID, also (in a previous patch) fixed the references to features which didn't exist. This patch also provides a whitelisting mechanism (which is completely unused) to allow vendors providing prebuilt toolchain components to restrict tunings to those based on or compatible with a particular ABI. (From OE-Core rev: 2a91ff0ba0d587c516a5a972553280364853faa4) Signed-off-by: Peter Seebach <peter.seebach@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Add a new case to issue sanity_check()Dongxiao Xu2012-04-241-0/+3
| | | | | | | | | | Judge if "SanityCheck" event is received, it will issue the sanity_check() and send "SanityCheckPassed" back if succeeded. (From OE-Core rev: 19704f9e69ecf09531687385b478b47f49fe372d) Signed-off-by: Dongxiao Xu <dongxiao.xu@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: add variable to disable the sanity checksJoshua Lock2012-04-231-1/+1
| | | | | | | | | | | | | It's useful for Hob to be able to disable the sanity checks completely without marking them as passed so that the user can get into the GUI to configure their settings, etc. Add a variable, DISABLE_SANITY_CHECKS, to do so. (From OE-Core rev: b022641f939bcfcdaddddc4db3af4d2dc70de832) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: import regular expression module in SuSE distro checkPaul Eggleton2012-04-181-0/+1
| | | | | | | | | | We call re.sub here, so we need to "import re" or an error occurs (tested on OpenSuSE 12.1). (From OE-Core rev: cb1f7cffc171e6b182f33a69ff688d76b7f1baed) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: trim trailing newline when reading /etc/redhat-releasePaul Eggleton2012-04-181-1/+1
| | | | | | | | | | Any trailing whitespace needs to be stripped before comparing it to the SANITY_TESTED_DISTROS list. (From OE-Core rev: bea0311a6478febe8ab4884fb1c479f610856534) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: If the sanity tests fail, don't mark them as completeRichard Purdie2012-04-171-1/+1
| | | | | | | | | | | If the sanity tests fail, we still were writing out the stamp which means they'd get skipped the next time we run bitbake. This is clearly wrong and we should only write out the stamp file if the sanity tests complete successfully. (From OE-Core rev: b99acaa8aa06dcbca46c3f7048124d6f8d643a8b) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* meta: Replace bb.data.expand(xxx, d) -> d.expand(xxx)Richard Purdie2012-03-051-9/+9
| | | | | | | | sed \ -e 's:bb.data.\(expand([^,()]*\), *\([^) ]*\) *):\2.\1):g' \ -i `grep -ril bb.data.expand *` Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Add ABI version 8, used for transition from the OEBasic to ↵Richard Purdie2012-02-221-0/+2
| | | | | | | | OEBasicHash stamp layout (From OE-Core rev: bf440fd971dd549ae4e92c54046ea062ef51cd85) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes: replace 'Poky' with 'OE-core'Koen Kooi2012-02-011-1/+1
| | | | | | | (From OE-Core rev: 1fdcd32520a05465b0d54c062f28bac9cdf74a20) Signed-off-by: Koen Kooi <koen@dominion.thruhere.net> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: No need to check for cvs any moreRichard Purdie2011-11-161-1/+1
| | | | | | (From OE-Core rev: dbe154965aa19ba30504895f0abc0575dd67d890) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* Convert to use direct access to the data store (instead of bb.data.*Var*())Richard Purdie2011-11-101-5/+5
| | | | | | | | | | | | | | | | | This is the result of running the following over the metadata: sed \ -e 's:bb.data.\(setVar([^,()]*,[^,()]*\), *\([^ )]*\) *):\2.\1):g' \ -e 's:bb.data.\(setVarFlag([^,()]*,[^,()]*,[^,()]*\), *\([^) ]*\) *):\2.\1):g' \ -e 's:bb.data.\(getVar([^,()]*\), *\([^(), ]*\) *,\([^)]*\)):\2.\1,\3):g' \ -e 's:bb.data.\(getVarFlag([^,()]*,[^,()]*\), *\([^(), ]*\) *,\([^)]*\)):\2.\1,\3):g' \ -e 's:bb.data.\(getVarFlag([^,()]*,[^,()]*\), *\([^() ]*\) *):\2.\1):g' \ -e 's:bb.data.\(getVar([^,()]*\), *\([^) ]*\) *):\2.\1):g' \ -i `grep -ril bb.data *` (From OE-Core rev: b22831fd63164c4db9c0b72934d7d734a6585251) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* Allow use of dash as /bin/shRichard Purdie2011-11-101-3/+0
| | | | | | | | | | | | | | | | | | | | | | | We've had the check for dash as /bin/sh for a long time. Dash has been around long enough now that most major issues have been identified and fixed from build perspective. This patch fixes a bashism in the openjade-native recipe. It also adjusts libtool so that the header at the script is used and not the value of $SHELL. This is because many Makefiles change $SHELL so dash can get used to execute what is otherwise configured as a bash shell script. Since we don't need to execute scripts this way on any system I'm aware of us building upon, the simplest fix is just to remove $SHELL. With these two changes the dash check can be removed and we can allow builds with dash as /bin/sh (From OE-Core rev: 07ded02ffd37b4fe60a6210dbf56490ea306f0b6) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> [Note: I know I need to add the description of the libtool change above into the prefix.patch]
* bitbake.conf: Drop remaining TERMCMD pieces and document OE_TERMINAL usage ↵Richard Purdie2011-11-041-13/+0
| | | | | | | | | | | instead (From OE-Core rev: ddb825c87dd3e59d50841a993080a00d1459b1e7) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> [Yes, this one is against meta-yocto, I'll fix it to apply the same change to OE-Core's local.conf.sample in the final version]
* sanity.bbclass: add a sanity check for KDE 4.x konsole in TERMCMDPaul Eggleton2011-09-191-0/+6
| | | | | | | | | | | | | | If the user has specified konsole in TERMCMD and it is version 2.x from KDE 4.x, raise an error as this version will not work for patch resolution purposes (it forks into the background and returns immediately). Addresses [YOCTO #1294] (From OE-Core rev: 843f9af47ffccb7ffdf3039c560bc3e4c88175a2) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity: disable mirrors for connectivity checkJoshua Lock2011-09-161-2/+6
| | | | | | | | | | | It's difficult to test a range of fetchers when using mirrors, therefore create a throwaway copy of the datastore and delete the MIRROR and PREMIRROR variables to ensure they aren't used in the connectivity check. (From OE-Core rev: 31cb3974eeaf2a808e8da3864e70986578c58cb9) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: re-enable DISTRO checkPaul Eggleton2011-09-071-3/+5
| | | | | | | | | | If DISTRO has been specified, ensure it is valid. (Unset or empty string is valid for DISTRO in OE-core by the use of defaultsetup.conf.) (From OE-Core rev: 895aa6b8ee9636a9f9d716c62c0adf951e1ff138) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: fix splitting on newlines of SANITY_TESTED_DISTROSPaul Eggleton2011-09-021-1/+1
| | | | | | | | | | | | | | | | | This variable should be split with \n sequences and these need to be specified literally in the string. A corrected version of the example given in the original commit (OE-core rev 75e3875341ddc8940e9ee2ccbbb2ec18194a68e6): SANITY_TESTED_DISTROS = " \ Ubuntu 11.04 \n \ Fedora release 14 (Laughlin) \n \ " (From OE-Core rev: cfc72d5796b6f83a01e06f3a1f044869db2d5d18) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Clean up various MACHINE checksRichard Purdie2011-08-311-37/+50
| | | | | | | | | | | | | | | | | | | | [YOCTO #1398] Firstly we should start checking if MACHINE is set. It if isn't lets make this a sanity warning since its very hard to successfully build without setting one (and anyone wanting to do that can disable the sanity checks easily enough anyway). Some of the checks depend on a MACHINE being specified. This change moves those checks to a separate function so they only run if MACHINE has been set correctly. Both these issues combine to ensure the user sees a sane message and avoids the nasty tracebacks in the bug report referenced above. (From OE-Core rev: 02aceca132f9e259cdc5283c4bfe84e6a55df54d) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Fix broken whitespaceRichard Purdie2011-08-311-6/+6
| | | | | | (From OE-Core rev: ab44ec9618109b0852d9441b6dd065c72c86acb2) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* sanity.bbclass: Fix broken indentation leading to code being skipped ↵Richard Purdie2011-08-311-19/+19
| | | | | | | | unintentionally (From OE-Core rev: 5200d38222c3188e695080b45b975daa20c8a9fa) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
* classes/sanity: enhance the network connectivity testJoshua Lock2011-08-251-12/+2
| | | | | | | | | | | | | | Switch to use the checkstatus fetcher API for the network connectivity test, this has several advantages: * we no longer print any messages to the console whilst the check is being run * we don't actually download anything, removing the need for tidy up and making the code more concise (From OE-Core rev: 81aa4bfbde871af1d569b64499c34a315bf6218e) Signed-off-by: Joshua Lock <josh@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>