From 12fc1c5637bca79a7a5735d9a8e819aefd0dd19b Mon Sep 17 00:00:00 2001 From: Khem Raj Date: Sat, 21 Mar 2020 16:28:03 -0700 Subject: bitbake: 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. (Bitbake rev: 51f827911b7202de3e855e683fdbd732d7a84e09) Signed-off-by: Khem Raj Signed-off-by: Richard Purdie --- bitbake/lib/bb/fetch2/__init__.py | 18 ++++-------------- 1 file changed, 4 insertions(+), 14 deletions(-) (limited to 'bitbake') diff --git a/bitbake/lib/bb/fetch2/__init__.py b/bitbake/lib/bb/fetch2/__init__.py index 9734e21126..d8ad19b59b 100644 --- a/bitbake/lib/bb/fetch2/__init__.py +++ b/bitbake/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 v1.2.3-54-g00ecf