diff options
| author | Joshua Lock <josh@linux.intel.com> | 2011-08-25 08:29:30 -0700 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2011-08-25 08:51:30 -0700 |
| commit | 6c2b7beac3cd23ed44bd3e195c6360a0932876bf (patch) | |
| tree | 95d5774445d635c27a61d88d95d5109d6db45497 | |
| parent | c03195482276d7c96d52f21562f8fa6a636f68a7 (diff) | |
| download | poky-6c2b7beac3cd23ed44bd3e195c6360a0932876bf.tar.gz | |
classes/sanity: enhance the network connectivity test
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>
| -rw-r--r-- | meta/classes/sanity.bbclass | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass index 16af0293d9..cefe6a483e 100644 --- a/meta/classes/sanity.bbclass +++ b/meta/classes/sanity.bbclass | |||
| @@ -95,15 +95,9 @@ def check_connectivity(d): | |||
| 95 | network_enabled = not bb.data.getVar('BB_NO_NETWORK', d, True) | 95 | network_enabled = not bb.data.getVar('BB_NO_NETWORK', d, True) |
| 96 | check_enabled = len(test_uris) | 96 | check_enabled = len(test_uris) |
| 97 | if check_enabled and network_enabled: | 97 | if check_enabled and network_enabled: |
| 98 | data = bb.data.createCopy(d) | ||
| 99 | bookmark = os.getcwd() | ||
| 100 | dldir = bb.data.expand('${TMPDIR}/sanity', data) | ||
| 101 | bb.data.setVar('DL_DIR', dldir, data) | ||
| 102 | |||
| 103 | try: | 98 | try: |
| 104 | fetcher = bb.fetch2.Fetch(test_uris, data) | 99 | fetcher = bb.fetch2.Fetch(test_uris, d) |
| 105 | fetcher.download() | 100 | fetcher.checkstatus() |
| 106 | fetcher.clean(test_uris) | ||
| 107 | except Exception: | 101 | except Exception: |
| 108 | # Allow the message to be configured so that users can be | 102 | # Allow the message to be configured so that users can be |
| 109 | # pointed to a support mechanism. | 103 | # pointed to a support mechanism. |
| @@ -111,10 +105,6 @@ def check_connectivity(d): | |||
| 111 | if len(msg) == 0: | 105 | if len(msg) == 0: |
| 112 | msg = "Failed to fetch test data from the network. Please ensure your network is configured correctly.\n" | 106 | msg = "Failed to fetch test data from the network. Please ensure your network is configured correctly.\n" |
| 113 | retval = msg | 107 | retval = msg |
| 114 | finally: | ||
| 115 | # Make sure we tidy up the cruft | ||
| 116 | oe.path.remove(dldir) | ||
| 117 | os.chdir(bookmark) | ||
| 118 | 108 | ||
| 119 | return retval | 109 | return retval |
| 120 | 110 | ||
