diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-05-13 12:00:45 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-05-20 09:24:23 +0100 |
commit | 2f009e5b1e4e0a4148ab7d5c60bf0e7a193faa2e (patch) | |
tree | dbf344deff3ef34e6e3df97f2e4e79a8596504b2 | |
parent | d993f172e806b095fa8bf799f11f975f6a5c238b (diff) | |
download | ast2050-yocto-poky-2f009e5b1e4e0a4148ab7d5c60bf0e7a193faa2e.zip ast2050-yocto-poky-2f009e5b1e4e0a4148ab7d5c60bf0e7a193faa2e.tar.gz |
bitbake/fetch2: Improve visibility of checksum warnings when fetching from mirrors
When fetching from mirrors, checksum errors would get buried in the
logs. This raises their profile so a warning is logged on the console
when fetcher checksum issues are encountered, even if other attempts
are made to get the file (which may or may not have the same issue).
(Bitbake rev: d43fafd7f01b5534499b45213197d8ccececdbc4)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | bitbake/lib/bb/fetch2/__init__.py | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/bitbake/lib/bb/fetch2/__init__.py b/bitbake/lib/bb/fetch2/__init__.py index ba6849e..9b5ba5a 100644 --- a/bitbake/lib/bb/fetch2/__init__.py +++ b/bitbake/lib/bb/fetch2/__init__.py @@ -63,6 +63,9 @@ class FetchError(BBFetchException): BBFetchException.__init__(self, msg) self.args = (message, url) +class ChecksumError(FetchError): + """Exception when mismatched checksum encountered""" + class UnpackError(BBFetchException): """General fetcher exception when something happens incorrectly when unpacking""" def __init__(self, message, url): @@ -312,7 +315,7 @@ def verify_checksum(u, ud, d): msg = msg + "\nFile: '%s' has %s checksum %s when %s was expected" % (ud.localpath, 'sha256', sha256data, ud.sha256_expected) if len(msg): - raise FetchError('Checksum mismatch!%s' % msg, u) + raise ChecksumError('Checksum mismatch!%s' % msg, u) def update_stamp(u, ud, d): @@ -504,8 +507,12 @@ def try_mirrors(d, origud, mirrors, check = False): raise except bb.fetch2.BBFetchException as e: - logger.debug(1, "Mirror fetch failure for url %s (original url: %s)" % (newuri, origud.url)) - logger.debug(1, str(e)) + if isinstance(e, ChecksumError): + logger.warn("Mirror checksum failure for url %s (original url: %s)\nCleaning and trying again." % (newuri, origud.url)) + logger.warn(str(e)) + else: + logger.debug(1, "Mirror fetch failure for url %s (original url: %s)" % (newuri, origud.url)) + logger.debug(1, str(e)) try: ud.method.clean(ud, ld) except UnboundLocalError: @@ -979,8 +986,11 @@ class Fetch(object): raise except BBFetchException as e: - logger.warn('Failed to fetch URL %s' % u) - logger.debug(1, str(e)) + if isinstance(e, ChecksumError): + logger.warn("Checksum error encountered with download (will attempt other sources): %s" % str(e)) + else: + logger.warn('Failed to fetch URL %s, attempting MIRRORS if available' % u) + logger.debug(1, str(e)) firsterr = e # Remove any incomplete fetch m.clean(ud, self.d) |