From 51f827911b7202de3e855e683fdbd732d7a84e09 Mon Sep 17 00:00:00 2001 From: Khem Raj Date: Sat, 21 Mar 2020 16:28:03 -0700 Subject: Revert "fetch2: Allow ${AUTOREV} to be used when BB_SRCREV_POLICY is "cache"" As per mailing list discussion, the cache policy was behaving correctly before and wouldn't expect to update after the initial fetch even for AUTOREV. This reverts commit ba093a38539960e645e994a66ed7872a604c00a9. Signed-off-by: Khem Raj Signed-off-by: Richard Purdie --- lib/bb/fetch2/__init__.py | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) diff --git a/lib/bb/fetch2/__init__.py b/lib/bb/fetch2/__init__.py index 9734e2112..d8ad19b59 100644 --- a/lib/bb/fetch2/__init__.py +++ b/lib/bb/fetch2/__init__.py @@ -500,11 +500,6 @@ def fetcher_init(d): srcrev_policy = d.getVar('BB_SRCREV_POLICY') or "clear" if srcrev_policy == "cache": logger.debug(1, "Keeping SRCREV cache due to cache policy of: %s", srcrev_policy) - # We still need to remove keys that are marked with "dontcache". - for key in list(revs.keys()): - if key.startswith("dontcache-"): - logger.debug(1, "Removing SRCREV key: %s" % key) - revs.pop(key) elif srcrev_policy == "clear": logger.debug(1, "Clearing SRCREV cache due to cache policy of: %s", srcrev_policy) revs.clear() @@ -737,8 +732,9 @@ def subprocess_setup(): signal.signal(signal.SIGPIPE, signal.SIG_DFL) def get_autorev(d): - # Do not cache the srcrev in the autorev case - d.setVar('BB_DONT_CACHE', '1') + # only not cache src rev in autorev case + if d.getVar('BB_SRCREV_POLICY') != "cache": + d.setVar('BB_DONT_CACHE', '1') return "AUTOINC" def get_srcrev(d, method_name='sortable_revision'): @@ -1601,13 +1597,7 @@ class FetchMethod(object): return True, str(latest_rev) def generate_revision_key(self, ud, d, name): - key = self._revision_key(ud, d, name) - if d.getVar('BB_DONT_CACHE'): - # Mark the key so it can be removed on the next bitbake run even if - # BB_SRCREV_POLICY is "cache". - return "dontcache-%s" % key - else: - return key + return self._revision_key(ud, d, name) def latest_versionstring(self, ud, d): """ -- cgit 1.2.3-korg