Skip to content

Commit 78e425d

Browse files
authored
remove tool-esp_install .piopm installation indicator
we dont need and it makes issues
1 parent b51adf8 commit 78e425d

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

platform.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -298,6 +298,9 @@ def _install_tl_install(self, version: str) -> bool:
298298
self.packages[tl_install_name]["optional"] = False
299299
self.packages[tl_install_name]["version"] = version
300300
pm.install(version)
301+
# Ensure backward compability by removing pio install status indicator
302+
tl_piopm_path = os.path.join(tl_install_path, ".piopm")
303+
safe_remove_file(tl_piopm_path)
301304

302305
if os.path.exists(os.path.join(tl_install_path, "package.json")):
303306
logger.info(f"{tl_install_name} successfully installed and verified")
@@ -308,10 +311,6 @@ def _install_tl_install(self, version: str) -> bool:
308311
# Copy tool-esp_install content to tl-install location
309312
if safe_copy_directory(tl_install_path, old_tl_install_path):
310313
logger.info(f"Content copied from {tl_install_name} to old tl-install location")
311-
old_tl_piopm_path = os.path.join(old_tl_install_path, ".piopm")
312-
safe_remove_file(old_tl_piopm_path)
313-
tl_piopm_path = os.path.join(tl_install_path, ".piopm")
314-
safe_remove_file(tl_piopm_path)
315314
else:
316315
logger.warning(f"Failed to copy content to old tl-install location")
317316
return True

0 commit comments

Comments
 (0)