Vendor requests.

bootstrap/update_vendoring.py

Index: bootstrap/update_vendoring.py
diff --git a/bootstrap/update_vendoring.py b/bootstrap/update_vendoring.py
index 8ff8b9f140cb6f56bcb5445232486924b1e92371..bd64b19dd1fee718bc4f14825694143206f9cbe7 100755
--- a/bootstrap/update_vendoring.py
+++ b/bootstrap/update_vendoring.py
@@ -167,6 +167,7 @@ class _VendoredPipPackage(_ActionBase):
     install_dir = os.path.join(workdir, 'six')
     c.check_call(['pip', 'install', '--verbose', '-t', install_dir,
                   '%s==%s' % (self._name, self._version)])
+
     c.vendor_dir(install_dir, dest)
 
 
@@ -249,6 +250,11 @@ _ACTIONS = (
         name='six',
         version='1.10.0'),
 
+    _VendoredPipPackage(

[dnj, 2016/07/20 21:34:07]

You don't still want this then, right?

[martiniss, 2016/07/20 21:39:27]

No, I still do. We still want a vendored requests, and this vendoring is better
than the copy pasting we did before. If you look at the CL, you can see it
deletes tests and documentation and stuff, which we don't need

[dnj, 2016/07/20 21:59:44]

So you are virtualenv-bootstrapping "requests", but also vendoring?

Do you delete doc/tests after vendoring, or was that just part of the previous
vendoring scheme and this method doesn't include them?If the former is the case,
you should add that to the script.

[martiniss, 2016/07/20 22:26:20]

Didn't mean to. No virtualenv, only vendor.
No, it was part of the previous vendoring scheme (git clone requests.git).

+        'recipe_engine/third_party/requests',
+        name='requests',
+        version='2.10.0'),
+
     _VendoredGitRepo(
         'recipe_engine/third_party/client-py/libs',
         repo='https://github.com/luci/client-py',