chiark / gitweb /
Tone down makebuildserver terminal spamming
authorCiaran Gultnieks <ciaran@ciarang.com>
Fri, 17 Jun 2016 12:39:01 +0000 (13:39 +0100)
committerCiaran Gultnieks <ciaran@ciarang.com>
Fri, 17 Jun 2016 12:39:01 +0000 (13:39 +0100)
The progress bar updating at each 1k chunk spams the terminal to such an
extent it stops responding to keypresses (at least with a fast
connection). It also seems extremely inefficient to be writing the file
in 1k chunks. This makes it 64k, which is more reasonable but quite
probably still too small.

makebuildserver

index 0f5cb86e54f6caec9f8b9cf9c0c57ca82386d01b..0561645263e20855735ebf0e40f4768b78bb19a1 100755 (executable)
@@ -292,8 +292,8 @@ for srcurl, shasum in cachefiles:
                          stream=True, verify=False, allow_redirects=True)
         content_length = int(r.headers.get('content-length'))
         with open(local_filename, 'ab') as f:
-            for chunk in progress.bar(r.iter_content(chunk_size=1024),
-                                      expected_size=(content_length / 1024) + 1):
+            for chunk in progress.bar(r.iter_content(chunk_size=65536),
+                                      expected_size=(content_length / 65536) + 1):
                 if chunk:  # filter out keep-alive new chunks
                     f.write(chunk)