diff options
author | Richard Ipsum <richard.ipsum@codethink.co.uk> | 2014-10-09 16:20:27 +0100 |
---|---|---|
committer | Richard Ipsum <richard.ipsum@codethink.co.uk> | 2014-11-04 12:40:54 +0000 |
commit | 761e41a7c1cffb17a27f6a3b1a82bfb8acfb2a63 (patch) | |
tree | 4436f839d85a1298a3f2fa361e4283fac00c46d7 | |
parent | bdde963ea7685feab9740dd21cc6e2ace3d0248e (diff) | |
download | import-761e41a7c1cffb17a27f6a3b1a82bfb8acfb2a63.tar.gz |
Change some of the err strings and format to 80 char
-rwxr-xr-x | import/pip.to_lorry | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/import/pip.to_lorry b/import/pip.to_lorry index 214f840..20278db 100755 --- a/import/pip.to_lorry +++ b/import/pip.to_lorry @@ -25,7 +25,6 @@ import json import sys import shutil import tempfile - import xmlrpclib PYPI_URL = 'http://pypi.python.org/pypi' @@ -55,9 +54,6 @@ def find_repo_type(url): return None -# also we only allow tar urls >.> -# need a compression flag for bzip I think >.> - def get_compression(url): bzip = 'bzip2' gzip = 'gzip' @@ -73,6 +69,7 @@ def get_compression(url): return None +# Assumption: url passed to this function must have a 'standard' tar extension def make_tarball_lorry(name, url): lorry = {'type': 'tarball', 'url': url} compression = get_compression(url) @@ -91,7 +88,7 @@ def ask_user(client, xs, fn, prompt='--> '): choice = choice - 1 if choice != None and choice <= len(xs) else None if choice == None: - print("Invalid choice, exiting", file=sys.stderr) + print("Invalid choice", file=sys.stderr) sys.exit(1) return choice @@ -115,13 +112,15 @@ def str_tarball_lorry(package_name): sys.exit(1) if len(releases) == 0: - print("Couldn't find any releases for packge %s, exiting" % package_name) + print("Couldn't find any releases for package %s" % package_name) sys.exit(1) - def f(release): - return client.release_data(package_name, release)['name'] + ' ' + release + def get_description(release): + return client.release_data(package_name, + release)['name'] + ' ' + release - choice = (ask_user(client, releases, f, prompt='Select release: ') + choice = (ask_user(client, releases, + get_description, prompt='Select release: ') if len(releases) > 1 else 0) release_version = releases[choice] @@ -142,12 +141,14 @@ def str_tarball_lorry(package_name): print("None of these urls look like tarballs:", file=sys.stderr) for url in urls: print("\t%s", url, file=sys.stderr) - print("Cannot proceed, exiting", file=sys.stderr) + print("Cannot proceed", file=sys.stderr) sys.exit(1) else: - print("Couldn't find any download urls for package, exiting", file=sys.stderr) + print("Couldn't find any download urls for package", file=sys.stderr) sys.exit(1) + urls = [{'url': 'http://foobar/baz.tar'}] + choice = (ask_user(client, urls, lambda url: url['url'], prompt='Select url: ') if len(urls) > 1 else 0) url = urls[choice]['url'] |