diff options
author | Sergey Sharybin <sergey.vfx@gmail.com> | 2020-02-12 20:57:07 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2020-02-12 20:57:57 +0300 |
commit | 5527cd4ba0b1d5e5721708b0859b6483070f3b44 (patch) | |
tree | 2f986c5f9dea4c89de5af9e508d897d9a9e78413 /build_files/buildbot/codesign | |
parent | 6da097136cf4c287ce1e08acdaa8376dfae9a898 (diff) |
Codesign: Attempt to make macOS codesign more reliable
Is still sometimes .ready file appears prior to an actual archive.
Diffstat (limited to 'build_files/buildbot/codesign')
-rw-r--r-- | build_files/buildbot/codesign/archive_with_indicator.py | 10 |
1 files changed, 9 insertions, 1 deletions
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: """ |