[PATCH 3/6] fetch2: Drop ESTALE error handling

Richard Purdie <[email protected]>
Newsgroups org.openembedded.lists.bitbake-devel
Message-ID <[email protected]>
This is old code and nobody I've talked to using NFS has seen this. Translating
this into a checksum error probably isn't correct, even if we did want to silently
handle it.

I propose we remove this code and see if anyone des really need something like
this, if so, writing it correctly without the ChecksumError usage.

Signed-off-by: Richard Purdie <[email protected]>
---
 lib/bb/fetch2/__init__.py | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/lib/bb/fetch2/__init__.py b/lib/bb/fetch2/__init__.py
index 51db8304c65..2676e0962b9 100644
--- a/lib/bb/fetch2/__init__.py
+++ b/lib/bb/fetch2/__init__.py
@@ -1156,12 +1156,6 @@ def try_mirror_url(fetch, origud, ud, ld, check = False):
     except bb.fetch2.NetworkAccess:
         raise
 
-    except IOError as e:
-        if e.errno in [errno.ESTALE]:
-            logger.warning("Stale Error Observed %s." % ud.url)
-            return False
-        raise
-
     except bb.fetch2.BBFetchException as e:
         if isinstance(e, ChecksumError):
             logger.warning("Mirror checksum failure for url %s (original url: %s)\nCleaning and trying again." % (ud.url, origud.url))
@@ -1976,12 +1970,6 @@ class Fetch(object):
 
                 m.update_donestamp(ud, d)
 
-            except IOError as e:
-                if e.errno in [errno.ESTALE]:
-                    logger.error("Stale Error Observed %s." % u)
-                    raise ChecksumError("Stale Error Detected")
-                raise
-
             except BBFetchException as e:
                 if isinstance(e, NoChecksumError):
                     (message, _) = e.args
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.