From 80a6f985571943b4e20c45ae75df8979dbfd9ddb Mon Sep 17 00:00:00 2001
From: Sudharshan S <ssreenivasan@apigee.com>
Date: Tue, 25 Jun 2013 20:38:25 +0000
Subject: [PATCH] Use requests instead of urllib.urlretrieve for plugins

---
 jenkinsapi_tests/systests/test_scm.py | 3 +--
 jenkinsapi_utils/jenkins_launcher.py  | 7 +++++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/jenkinsapi_tests/systests/test_scm.py b/jenkinsapi_tests/systests/test_scm.py
index c5fcfb1..225c308 100644
--- a/jenkinsapi_tests/systests/test_scm.py
+++ b/jenkinsapi_tests/systests/test_scm.py
@@ -22,8 +22,7 @@ class TestSCMGIT(BaseSystemTest):
         ii = job.invoke()
         ii.block(until='completed')
         self.assertFalse(ii.is_running())
-        bn = ii.get_build_number()
-        b = job.get_build(bn)
+        b = ii.get_build()
         self.assertIsInstance(b.get_revision(), basestring)
 
 if __name__ == '__main__':
diff --git a/jenkinsapi_utils/jenkins_launcher.py b/jenkinsapi_utils/jenkins_launcher.py
index 9c1a938..4fffc72 100644
--- a/jenkinsapi_utils/jenkins_launcher.py
+++ b/jenkinsapi_utils/jenkins_launcher.py
@@ -1,6 +1,6 @@
 import os
 import time
-import urllib
+import requests
 import Queue
 import shutil
 import logging
@@ -80,7 +80,10 @@ class JenkinsLancher(object):
             log.info("Plugins will be installed in '%s'" % plugin_dir)
             # FIXME: This is kinda ugly but works
             filename = "plugin_%s.hpi" % i
-            urllib.urlretrieve(url, os.path.join(plugin_dir, filename))
+            plugin_path = os.path.join(plugin_dir, filename)
+            with open(plugin_path, 'wb') as h:
+                request = requests.get(url)
+                h.write(request.content)
         log.info("Restarting Jenkins after installing the plugins")
         self.jenkins_process.terminate()
         self.jenkins_process.wait()
-- 
2.34.1