diff options
author | Ben Hutchings <ben.hutchings@codethink.co.uk> | 2020-05-04 19:28:48 +0100 |
---|---|---|
committer | Ben Hutchings <ben.hutchings@codethink.co.uk> | 2020-05-29 20:35:30 +0100 |
commit | 2234eae5baddeee7f41b41b40cf5cfb5d61436a1 (patch) | |
tree | f1f3edd98371398393557e25e8d6bcbdf489bd81 /lorry.gzip-importer | |
parent | c64348cc872907fc711b756dbc73693402ee2fa1 (diff) | |
download | lorry-2234eae5baddeee7f41b41b40cf5cfb5d61436a1.tar.gz |
lorry.{g,}zip-importer: Only import modules and classes into module scope
Currently these scripts import a bunch of functions into module scope,
which is a bit confusing. Change them to access functions through their
enclosing module names.
In lorry.gzip-importer, hexversion and stderr weren't used at all.
Diffstat (limited to 'lorry.gzip-importer')
-rwxr-xr-x | lorry.gzip-importer | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/lorry.gzip-importer b/lorry.gzip-importer index 4248f8b..8bc18ed 100755 --- a/lorry.gzip-importer +++ b/lorry.gzip-importer @@ -2,23 +2,23 @@ # gzip archive frontend for git-fast-import -from os import popen, path -from os.path import splitext -from os.path import getmtime -from sys import argv, exit, hexversion, stderr from gzip import GzipFile +import os +import os.path import struct +import sys -if len(argv) < 2: - print('usage:', argv[0], '<gzipfile>...') - exit(1) + +if len(sys.argv) < 2: + print('usage:', sys.argv[0], '<gzipfile>...') + sys.exit(1) branch_name = 'master' branch_ref = 'refs/heads/%s' % branch_name committer_name = 'Lorry Gzip Importer' committer_email = 'lorry-gzip-importer@lorry' -fast_import = popen('git fast-import --quiet', 'w') +fast_import = os.popen('git fast-import --quiet', 'w') def printlines(list): for str in list: fast_import.write(str + "\n") @@ -29,11 +29,11 @@ def uncompressedsize(filename): f.seek(-4, 2) return struct.unpack('I', f.read(4))[0] -for zipfile in argv[1:]: +for zipfile in sys.argv[1:]: # Gzip does have an encoded mtime, however Python's GzipFile # just ignores it, so we just yank the mtime of the zip file itself. - mtime = getmtime (zipfile); + mtime = os.path.getmtime(zipfile); file_size = uncompressedsize (zipfile); zip = GzipFile(zipfile, 'rb') @@ -43,7 +43,7 @@ for zipfile in argv[1:]: committer = committer_name + ' <' + committer_email + '> %d +0000' % \ mtime - zipfile_basename = path.basename(zipfile) + zipfile_basename = os.path.basename(zipfile) printlines(('commit ' + branch_ref, 'committer ' + committer, \ 'data <<EOM', 'Imported from ' + zipfile_basename + '.', 'EOM', \ '', 'deleteall')) @@ -52,10 +52,10 @@ for zipfile in argv[1:]: unzipped_file = zipfile_basename[:last_dot] fast_import.write('M 100644 :1 ' + unzipped_file + '\n'); - zipname, _ = splitext(zipfile_basename) + zipname, _ = os.path.splitext(zipfile_basename) printlines(('', 'tag ' + zipname, \ 'from ' + branch_ref, 'tagger ' + committer, \ 'data <<EOM', 'Package ' + zipfile, 'EOM', '')) if fast_import.close(): - exit(1) + sys.exit(1) |