GUI: Remove extra loop

Should only need to unzip twice, if more than there’s a serious problem
This commit is contained in:
Mykola Grymalyuk
2023-05-16 10:30:53 -06:00
parent 54480c3776
commit f7d5b9345f

View File

@@ -167,7 +167,7 @@ class UpdateFrame(wx.Frame):
# Some hell spawn at Github decided to double zip our Github Actions artifacts
# So we need to unzip it twice
for i in range(3):
for i in range(2):
result = subprocess.run(
["ditto", "-xk", str(self.constants.payload_path / "OpenCore-Patcher-GUI.app.zip"), str(self.constants.payload_path)], capture_output=True
)
@@ -178,7 +178,7 @@ class UpdateFrame(wx.Frame):
if Path(self.application_path).exists():
break
if i == 2:
if i == 1:
wx.CallAfter(self.progress_bar.SetValue, 0)
wx.CallAfter(wx.MessageBox, "Failed to extract update. Error: Update file does not exist", "Critical Error!", wx.OK | wx.ICON_ERROR)
wx.CallAfter(sys.exit, 1)