diff options
author | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2015-01-09 12:02:15 +0000 |
---|---|---|
committer | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2015-01-09 12:18:50 +0000 |
commit | acf80a68ee41ec1d6bea531ea2fb00dc1227f2e3 (patch) | |
tree | 47fa37403262b00e50ec8e7b2c29719e10b7e19b | |
parent | 8bddb0996c44ce8ce989359a7b4c9ff102466ef2 (diff) | |
download | lorry-acf80a68ee41ec1d6bea531ea2fb00dc1227f2e3.tar.gz |
Redownload a tarball if it exists but is 0 bytes long
This is a safety measure. Commit
8bddb0996c44ce8ce989359a7b4c9ff102466ef2 should mean that this can't
happen, but 'lorry' runs unattended so we really don't want it to break.
-rwxr-xr-x | lorry | 8 |
1 files changed, 7 insertions, 1 deletions
@@ -30,6 +30,12 @@ __version__ = '0.0' lorry_path = os.path.realpath(__file__) + +def file_missing_or_empty(filename): + ''' A more comprehensive alternative to os.path.exists(). ''' + return (not os.path.isfile(filename)) or (os.path.getsize(filename) <= 0) + + def quote_url(url): ''' Convert URIs to strings that only contain digits, letters, % and _. @@ -475,7 +481,7 @@ class Lorry(cliapp.Application): basename = os.path.basename(url_path) tardest = os.path.join(dirname, basename) self.progress('.. checking if we need to fetch %s' % basename) - if not os.path.exists(tardest): + if file_missing_or_empty(tardest): self.progress('.. attempting to fetch.') try: with open(tardest, 'w') as tarfile: |