Do not generate provenance if on Windows machine. The provenance generation on Windows machines for BCID compliance is not yet ready, but instead of creating a failure in that scenario, this change allows for the provenance generation to be skipped. After Windows provenance generation is available, we will uncomment this line. Change-Id: I203e267b0aed542e686f58eacdb4c150152e5d7c Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/37101 Commit-Queue: Jesse Seales <jseales@google.com> Reviewed-by: Godofredo Contreras <godofredoc@google.com>
diff --git a/recipe_modules/flutter_bcid/__init__.py b/recipe_modules/flutter_bcid/__init__.py index d148649..c790b26 100644 --- a/recipe_modules/flutter_bcid/__init__.py +++ b/recipe_modules/flutter_bcid/__init__.py
@@ -7,4 +7,5 @@ 'recipe_engine/bcid_reporter', 'recipe_engine/buildbucket', 'recipe_engine/file', + 'recipe_engine/platform', ]
diff --git a/recipe_modules/flutter_bcid/api.py b/recipe_modules/flutter_bcid/api.py index 8b84261..89018e9 100644 --- a/recipe_modules/flutter_bcid/api.py +++ b/recipe_modules/flutter_bcid/api.py
@@ -40,7 +40,11 @@ local_artifact_path: (str) path and filename of a specific file. remote_artifact_path: (str) path and filename of a specific file. """ - if self._is_official_build(): + if (self._is_official_build() and + # TODO(jseales): Uncomment next line after windows + # can generate provenance succesfully + # https://github.com/flutter/flutter/issues/116749 + not self.m.platform.is_win): sha256 = self.m.file.file_hash(local_artifact_path) self.m.bcid_reporter.report_gcs( sha256,