Re: [PATCH 1/1] gitsm: Add need_update method to determine when we are going to a new SRCREV

["William A. Kennington III" <wak@google.com>]

Sorry, bad send from gmail which seems to want to wrap and deduplicate 
my text. You can apply the following patch to test against over pessimism.

diff --git a/lib/bb/tests/fetch.py b/lib/bb/tests/fetch.py
index 429998b3..718605e2 100644
--- a/lib/bb/tests/fetch.py
+++ b/lib/bb/tests/fetch.py
@@ -961,6 +961,26 @@ class FetcherNetworkTest(FetcherTest):
          self.assertTrue(os.path.exists(os.path.join(repo_path, 
'.git/modules/extern/json/config')), msg='Missing submodule config 
"extern/json"')
          self.assertTrue(os.path.exists(os.path.join(repo_path, 
'.git/modules/extern/sanitizers/config')), msg='Missing submodule config 
"extern/sanitizers"')

+    def test_git_submodule_no_update_CLI11(self):
+        """ Prevent regression on update detection not finding already 
downloaded commits """
+        url = 
"gitsm://github.com/CLIUtils/CLI11;protocol=git;rev=da901cca542612a133efcb04e8e78080186991e4"
+        fetcher = bb.fetch.Fetch([url], self.d)
+        fetcher.download()
+
+        self.d.setVar("BB_NO_NETWORK", 1)
+        # CLI11 that pulls in a newer nlohmann-json which we already 
fetched
+        url = 
"gitsm://github.com/CLIUtils/CLI11;protocol=git;rev=899100c1619ea1282620604ac9199010e65f907e"
+        fetcher = bb.fetch.Fetch([url], self.d)
+        fetcher.download()
+        # Previous cwd has been deleted
+        os.chdir(os.path.dirname(self.unpackdir))
+        fetcher.unpack(self.unpackdir)
+
+        repo_path = os.path.join(self.tempdir, 'unpacked', 'git')
+        self.assertTrue(os.path.exists(os.path.join(repo_path, 
'.git/modules/extern/googletest/config')), msg='Missing submodule config 
"extern/googletest"')
+        self.assertTrue(os.path.exists(os.path.join(repo_path, 
'.git/modules/extern/json/config')), msg='Missing submodule config 
"extern/json"')
+        self.assertTrue(os.path.exists(os.path.join(repo_path, 
'.git/modules/extern/sanitizers/config')), msg='Missing submodule config 
"extern/sanitizers"')
+
      def test_git_submodule_aktualizr(self):
          url = 
"gitsm://github.com/advancedtelematic/aktualizr;branch=master;protocol=git;rev=d00d1a04cc2366d1a5f143b84b9f507f8bd32c44"
          fetcher = bb.fetch.Fetch([url], self.d)