From 24708c00d41e93aa756bce33b0a04775ec4977ea Mon Sep 17 00:00:00 2001 From: Michael Goderbauer Date: Mon, 16 Dec 2024 10:48:07 -0800 Subject: [PATCH] Reland 'Delete stale package_config.json in gclient sync hook' (flutter/engine#57218) https://github.com/flutter/engine/pull/57195 was reverted because it tried to delete non-existing files. This adds a check to only attempt to delete the file if it exists. --- engine/src/flutter/tools/pub_get_offline.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/engine/src/flutter/tools/pub_get_offline.py b/engine/src/flutter/tools/pub_get_offline.py index 154c08e6d5..c56d74f805 100644 --- a/engine/src/flutter/tools/pub_get_offline.py +++ b/engine/src/flutter/tools/pub_get_offline.py @@ -134,8 +134,10 @@ def delete_config_files(): gitcmd, cwd=ENGINE_DIR, stderr=subprocess.STDOUT, text=True ).splitlines() for file in files_to_delete: - print('Deleting %s...' % file) - os.remove(os.path.join(ENGINE_DIR, file)) + file_path = os.path.join(ENGINE_DIR, file) + if os.path.exists(file_path): + print('Deleting %s...' % file) + os.remove(file_path) def main():