[Bf-blender-cvs] [5527cd4ba0b] master: Codesign: Attempt to make macOS codesign more reliable

[Sergey Sharybin]

Commit: 5527cd4ba0b1d5e5721708b0859b6483070f3b44
Author: Sergey Sharybin
Date:   Wed Feb 12 18:57:07 2020 +0100
Branches: master
https://developer.blender.org/rB5527cd4ba0b1d5e5721708b0859b6483070f3b44

Codesign: Attempt to make macOS codesign more reliable

Is still sometimes .ready file appears prior to an actual archive.

===================================================================

M	build_files/buildbot/codesign/archive_with_indicator.py

===================================================================

diff --git a/build_files/buildbot/codesign/archive_with_indicator.py b/build_files/buildbot/codesign/archive_with_indicator.py
index d417d4560d6..ad3fe1c6ac7 100644
--- a/build_files/buildbot/codesign/archive_with_indicator.py
+++ b/build_files/buildbot/codesign/archive_with_indicator.py
@@ -72,7 +72,15 @@ class ArchiveWithIndicator:
 
     def is_ready(self) -> bool:
         """Check whether the archive is ready for access."""
-        return self.ready_indicator_filepath.exists()
+        if not self.ready_indicator_filepath.exists():
+            return False
+        # Sometimes on macOS indicator file appears prior to the actual archive
+        # despite the order of creation and os.sync() used in tag_ready().
+        # So consider archive not ready if there is an indicator without an
+        # actual archive.
+        if not self.archive_filepath.exists():
+            return False
+        return True
 
     def tag_ready(self) -> None:
         """