Patchwork [bitbake-devel] Paul Eggleton : bitbake/fetch2: allow resuming interrupted fetch

mail settings
Date Jan. 20, 2012, 4:50 p.m.
Message ID <20120120165058.B0C6B10339@opal>
Download mbox | patch
Permalink /patch/19849/
State Not Applicable
Headers show

Comments - Jan. 20, 2012, 4:50 p.m.
Module: bitbake.git
Branch: master
Commit: 721e986624529aedef96dd06c9fe6243f2897b10

Author: Paul Eggleton <>
Date:   Mon Jan 16 15:01:08 2012 +0000

bitbake/fetch2: allow resuming interrupted fetch

A lot of our checks in the fetcher code assume that if the downloaded
file exists, the download finished successfully; this meant that if
BitBake was interrupted in the middle of a fetch then it would not
resume the fetch the next time, but instead attempt to use the
half-fetched file and usually fail as a result.

Since we're already writing a ".done" stamp file when a fetch completes,
just check for its existence and assume the download didn't complete
successfully if it isn't present.

Signed-off-by: Paul Eggleton <>
Signed-off-by: Richard Purdie <>


 lib/bb/fetch2/ |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)


diff --git a/lib/bb/fetch2/ b/lib/bb/fetch2/
index fe4f4b1..07aac4c 100644
--- a/lib/bb/fetch2/
+++ b/lib/bb/fetch2/
@@ -479,7 +479,7 @@  def try_mirrors(d, origud, mirrors, check = False):
                     return found
-            if ud.method.need_update(newuri, ud, ld):
+            if not os.path.exists(ud.donestamp) or ud.method.need_update(newuri, ud, ld):
       , ud, ld)
                 if hasattr(ud.method,"build_mirror_data"):
                     ud.method.build_mirror_data(newuri, ud, ld)
@@ -955,7 +955,7 @@  class Fetch(object):
                 self.d.setVar("BB_NO_NETWORK", network)
-                if not m.need_update(u, ud, self.d):
+                if os.path.exists(ud.donestamp) and not m.need_update(u, ud, self.d):
                     localpath = ud.localpath
                 elif m.try_premirror(u, ud, self.d):
                     logger.debug(1, "Trying PREMIRRORS")
@@ -966,7 +966,7 @@  class Fetch(object):
                     self.d.setVar("BB_NO_NETWORK", "1")
                 firsterr = None
-                if not localpath and m.need_update(u, ud, self.d):
+                if not localpath and ((not os.path.exists(ud.donestamp)) or m.need_update(u, ud, self.d)):
                         logger.debug(1, "Trying Upstream")
               , ud, self.d)