[PATCH 4/4] Don't do several times the same thing

Mathieu Bridon bochecha at fedoraproject.org
Tue Feb 17 03:33:46 UTC 2015


From: Mathieu Bridon <bochecha at daitauha.fr>

---
 src/pyrpkg/__init__.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/pyrpkg/__init__.py b/src/pyrpkg/__init__.py
index 57d9b75..a5f02ea 100644
--- a/src/pyrpkg/__init__.py
+++ b/src/pyrpkg/__init__.py
@@ -1578,10 +1578,9 @@ class Commands(object):
                 if self._verify_file(outfile, entry.hash, entry.hashtype):
                     continue
             self.log.info("Downloading %s" % (entry.file))
+            urled_file = entry.file.replace(' ', '%20')
             url = '%s/%s/%s/%s/%s' % (self.lookaside, self.module_name,
-                                      entry.file.replace(' ', '%20'),
-                                      entry.hash,
-                                      entry.file.replace(' ', '%20'))
+                                      urled_file, entry.hash, urled_file)
             # These options came from Makefile.common.
             # Probably need to support wget as well
             command = ['curl', '-H', 'Pragma:', '-o', outfile, '-R', '-S',
-- 
2.1.0



More information about the buildsys mailing list