Merge lp:~mguthriejr/phablet-tools/phablet-tools into lp:phablet-tools

Proposed by Mark
Status: Rejected
Rejected by: Sergio Schvezov
Proposed branch: lp:~mguthriejr/phablet-tools/phablet-tools
Merge into: lp:phablet-tools
Diff against target: 21 lines (+2/-2)
1 file modified
phabletutils/downloads.py (+2/-2)
To merge this branch: bzr merge lp:~mguthriejr/phablet-tools/phablet-tools
Reviewer Review Type Date Requested Status
Sergio Schvezov Needs Fixing
Ricardo Salveti Pending
Review via email: mp+166925@code.launchpad.net

Description of the change

This updates the terminal logging prompts in downloads.py. It fixes some misspellings.

To post a comment you must log in.
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

See if still applies

review: Needs Fixing

Unmerged revisions

103. By Mark

Updated the log to terminal outputs in downloads.py, fixed misspellings

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'phabletutils/downloads.py'
2--- phabletutils/downloads.py 2013-04-09 16:48:34 +0000
3+++ phabletutils/downloads.py 2013-06-01 01:40:34 +0000
4@@ -35,7 +35,7 @@
5 def __init__(self, base_uri, download_dir, artifact_list, offline=False):
6 if not os.path.isdir(download_dir):
7 raise RuntimeError('Directory %s does not exist or is not a '
8- 'direcotry' % download_dir)
9+ 'directory' % download_dir)
10 self._base_uri = base_uri
11 self._download_dir = download_dir
12 self._artifact_list = artifact_list
13@@ -63,7 +63,7 @@
14 continue
15 download(uri, target)
16 download(md5uri, md5file)
17- log.info('Validatind download for %s' % (artifact))
18+ log.info('Validating download for %s' % (artifact))
19 self.validate(md5file)
20 if artifact.endswith('.gz'):
21 subprocess.call(['gunzip', target])

Subscribers

People subscribed via source and target branches