summaryrefslogtreecommitdiff
path: root/lorry.gzip-importer
diff options
context:
space:
mode:
authorBen Hutchings <ben.hutchings@codethink.co.uk>2020-05-04 19:28:48 +0100
committerBen Hutchings <ben.hutchings@codethink.co.uk>2020-05-29 20:35:30 +0100
commit2234eae5baddeee7f41b41b40cf5cfb5d61436a1 (patch)
treef1f3edd98371398393557e25e8d6bcbdf489bd81 /lorry.gzip-importer
parentc64348cc872907fc711b756dbc73693402ee2fa1 (diff)
downloadlorry-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-xlorry.gzip-importer26
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)