* ubuntutools.archive:

- Add quiet option to silence downloading.
  - Use wget-style progress bar (fixed width) (LP: #845787)

debian/changelog

@@ -13,6 +13,9 @@ ubuntu-dev-tools (0.131) UNRELEASED; urgency=low
     past illegal version numbers (LP: #727314)
   * ubuntutools.update_maintainer: Don't use strict changelog parsing
     (LP: #806633)
+  * ubuntutools.archive:
+    - Add quiet option to silence downloading.
+    - Use wget-style progress bar (fixed width) (LP: #845787)
 
   [ Colin Watson ]
   * syncpackage: Fix typo.

ubuntutools/archive.py

@@ -128,7 +128,7 @@ class SourcePackage(object):
     distribution = None
 
     def __init__(self, package=None, version=None, component=None,
-                 dscfile=None, lp=None, mirrors=(), workdir='.'):
+                 dscfile=None, lp=None, mirrors=(), workdir='.', quiet=False):
         "Can be initialised either using package, version or dscfile"
         assert ((package is not None and version is not None)
                 or dscfile is not None)
@@ -136,6 +136,7 @@ class SourcePackage(object):
         self.source = package
         self._lp = lp
         self.workdir = workdir
+        self.quiet = quiet
 
         # Cached values:
         self._component = component
@@ -305,11 +306,7 @@ class SourcePackage(object):
 
     def _download_file(self, url, filename):
         "Download url to filename in workdir."
-        logurl = url
-        if os.path.basename(url) != filename:
-            logurl += ' -> ' + filename
         pathname = os.path.join(self.workdir, filename)
-        if self.dsc:
         if self.dsc.verify_file(pathname):
             Logger.debug('Using existing %s', filename)
             return True
@@ -317,30 +314,37 @@ class SourcePackage(object):
                 if entry['name'] == filename]
         assert len(size) == 1
         size = int(size[0])
-            Logger.normal('Downloading %s (%0.3f MiB)', logurl,
+        host = urlparse.urlparse(url).hostname
-                          size / 1024.0 / 1024)
+        if not self.quiet:
-        else:
+            Logger.normal('Downloading %s from %s (%0.3f MiB)',
-            Logger.normal('Downloading %s', logurl)
+                          filename, host, size / 1024.0 / 1024)
 
         try:
             in_ = urllib2.build_opener().open(url)
         except urllib2.URLError:
             return False
 
+        downloaded = 0
+        bar_width = 60
         with open(pathname, 'wb') as out:
             while True:
                 block = in_.read(10240)
                 if block == '':
                     break
+                downloaded += len(block)
                 out.write(block)
-                Logger.stdout.write('.')
+                if not self.quiet:
+                    percent = downloaded * 100 // size
+                    bar = '=' * int(round(downloaded * bar_width / size))
+                    bar = (bar + '>' + ' ' * bar_width)[:bar_width]
+                    Logger.stdout.write('[%s] %#3i%%\r' % (bar, percent))
                     Logger.stdout.flush()
         in_.close()
-        Logger.stdout.write(' done\n')
+        if not self.quiet:
+            Logger.stdout.write(' ' * (bar_width + 7) + '\r')
             Logger.stdout.flush()
-        if self.dsc:
         if not self.dsc.verify_file(pathname):
-                Logger.error('Checksum does not match.')
+            Logger.error('Checksum for %s does not match.', filename)
             return False
         return True