3D-printing/Cura
1
0
Fork 0
mirror of https://github.com/Ultimaker/Cura.git synced 2025-07-07 06:57:28 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

Use ContainerRegistry.lockFile in Cura package management

Browse source 
Prevent concurrent I/O issues.
This commit is contained in:
Lipu Fei 2018-05-01 10:31:54 +02:00
parent cff4dc8a0c
commit 1fb2edace5
1 changed files with 19 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

6
cura/CuraPackageManager.py
View file

@ -48,6 +48,9 @@ class CuraPackageManager(QObject):
Logger.log("i", "Package management file %s doesn't exist, do nothing", self._package_management_file_path) Logger.log("i", "Package management file %s doesn't exist, do nothing", self._package_management_file_path)
return return
# Need to use the file lock here to prevent concurrent I/O from other processes/threads
container_registry = self._application.getContainerRegistry()
with container_registry.lockFile():
with open(self._package_management_file_path, "r", encoding = "utf-8") as f: with open(self._package_management_file_path, "r", encoding = "utf-8") as f:
management_dict = json.load(f, encoding = "utf-8") management_dict = json.load(f, encoding = "utf-8")
@ -58,6 +61,9 @@ class CuraPackageManager(QObject):
Logger.log("i", "Package management file %s is loaded", self._package_management_file_path) Logger.log("i", "Package management file %s is loaded", self._package_management_file_path)
def _saveManagementData(self) -> None: def _saveManagementData(self) -> None:
# Need to use the file lock here to prevent concurrent I/O from other processes/threads
container_registry = self._application.getContainerRegistry()
with container_registry.lockFile():
with open(self._package_management_file_path, "w", encoding = "utf-8") as f: with open(self._package_management_file_path, "w", encoding = "utf-8") as f:
data_dict = {"installed": self._installed_package_dict, data_dict = {"installed": self._installed_package_dict,
"to_remove": list(self._to_remove_package_set), "to_remove": list(self._to_remove_package_set),