Retry failed downloads

- also fixed a bug where releases won't be deleted untless the assets
are removed first

scripts/update_files

 #!/usr/bin/env python3
 
+#pip3 install PyGithub
+
 from github import Github
 
 import sys
@@ -11,8 +13,12 @@ REPO_NAME = "TheTorProject/gettorbrowser"
 
 failure = False
 
+failed_uploads = []
+
 def delete_old_releases(repo):
     for release in repo.get_releases():
+        for asset in release.get_assets():
+            asset.delete_asset()
         release.delete_release()
 
 #Download list of tor browser releases and upload them to github
@@ -28,10 +34,17 @@ def upload_files(release):
                     subprocess.check_call(["/usr/bin/wget", "--quiet", url])
                     release.upload_asset(filename)
                 except:
-                    print("Error: failed to download "+url)
-                    print("Please upload this file manually.")
-                    failure = True
-
+                    print("Error: failed to download "+url+". Will retry later.")
+                    failed_uploads.append(url)
+    #Retry failed uploads
+    for url in failed_uploads:
+        filename = url.split('/')[-1]
+        try:
+            subprocess.check_call(["/usr/bin/wget", "--quiet", url])
+            release.upload_asset(filename)
+        except:
+            print("Error: failed to download "+url+". Please upload this file manually.")
+            failure = True
 
 
 def main(token):

[Philipp Winter]

Nitpick: For a clean commit history, I suggest that each commit only contains what it must contain. For this change, which is unrelated, I would make a separate commit.

[Philipp Winter]

Very minor nitpick: Maybe have it print "failed to update" instead of "failed to download" because it could be either the download or the upload that failed?

[Philipp Winter]

Same nitpick as above.