<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 16, 2016 at 1:58 AM, Pascal Bach <span dir="ltr">&lt;<a href="mailto:pascal.bach@siemens.com" target="_blank">pascal.bach@siemens.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1n1" class="a3s aXjCH m15732581c0425444">When an update of the local copy is required always try to fetch the update<br>
from a premirror first.<br>
The premirror should always have precedence over the upstream repository<br>
as updating from there might not work if BB_FETCH_PREMIRRORONLY,<br>
BB_NO_NBB_ALLOWED_NETWORKS or BB_NO_NETWORK is set.<br>
<br>
This also gets rid of the special case for BB_FETCH_PREMIRRORONLY.<br>
<br>
Signed-off-by: Pascal Bach &lt;<a href="mailto:pascal.bach@siemens.com">pascal.bach@siemens.com</a>&gt;<br>
---<br>
 lib/<span class="il">bb</span>/fetch2/git.py | 9 ++-------<br>
 1 file changed, 2 insertions(+), 7 deletions(-)<br>
<br>
diff --git a/lib/<span class="il">bb</span>/fetch2/git.py b/lib/<span class="il">bb</span>/fetch2/git.py<br>
index 1bec60a..cac2a87 100644<br>
--- a/lib/<span class="il">bb</span>/fetch2/git.py<br>
+++ b/lib/<span class="il">bb</span>/fetch2/git.py<br>
@@ -227,13 +227,8 @@ <span class="il">class</span> Git(FetchMethod):<br>
         return False<br>
<br>
     def try_premirror(self, ud, d):<br>
-        # If we don&#39;t do this, updating an existing checkout with only premirrors<br>
-        # is not possible<br>
-        if d.getVar(&quot;BB_FETCH_<wbr>PREMIRRORONLY&quot;, True) is not None:<br>
-            return True<br>
-        if os.path.exists(ud.clonedir):<br>
-            return False<br>
-        return True<br>
+        # Always try premirror if an update is required<br>
+        return self.need_update(ud, d)</div></blockquote></div><br>Will this result in fetching a mirror tarball even if you already have a clone in ud.clonedir? The previous logic always returned False if the clonedir existed..<br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics</div>
</div></div>