Make one script for automatic updates
Merge request reports
Activity
added 1 commit
- 2d8910d4 - Add IA session and run export_stats+add_links_to_db
added 1 commit
- 8670c691 - Upload to one IA item and don't cd into ~releases
added 1 commit
- ec5018ca - fixup! Upload to one IA item and don't cd into ~releases
requested review from @cohosh
Hey @meskio actually since you've been working on the GetTor rewrite, can you take a look at this? I'm not sure what your plans are for that rewrite and if this would be useful in the meantime/could be adapted for use alongside the rdsys changes.
@HackerNCoder nice work. Is pretty handy to have all the sites to update in once place.
We are currently in the process of rewriting gettor as part of rdsys and this script will be replaced by a daemon that will run in background checking for updates.
I think will be useful to merge this work, it might be useful if gettor in rdsys takes some time to finish. I'll do a review and leave few comments.
- Resolved by HackerNCoder
- Resolved by HackerNCoder
- Resolved by HackerNCoder
I like it. I left few comments with improvements to make. And few other things:
- Do you mind squashing all the commits into a single one? I'm happy to help if you don't know how to do that in git, poke me in irc, or I can do it myself before merging if needed.
- You will need to update
setup.py
to remove the deleted scripts and add yourupdate_all
script.
- Resolved by HackerNCoder
- Resolved by HackerNCoder
102 subprocess.check_call(["rclone", "delete gdrive:releases"]) 103 subprocess.check_call(["rclone", "cleanup gdrive:releases"]) 104 except: 105 print("Error: failed to delete or cleanup with rclone", file=sys.stderr) 106 return 1 107 108 def upload(self, filename, arch): 109 try: 110 subprocess.check_call(["rclone", "copy", filename, "gdrive:releases"]) 111 except: 112 print("Error: failed to rclone "+filename+" to Google Drive", file=sys.stderr) 113 114 class IARemote: 115 def __init__(self, token): 116 access = token.split("\n")[0] 117 secret = token.split("\n")[1] changed this line in version 6 of the diff
- Resolved by HackerNCoder
- Resolved by meskio