Issue pypa#2677: Disable wheels for setup.py options.

Using --install-options, --build-options, --global-options changes
the way that setup.py behaves, and isn't honoured by the wheel code.
The new wheel autobuilding code made this very obvious - disable
the use of wheels when these options are supplied.

Author: rbtcollins

CHANGES.txt

@@ -28,6 +28,10 @@
 * Allow fine grained control over the use of wheels and source builds.
   (:pull:`2699`)
 
+* The use of ``--install-option``, ``--global-option`` or ``--build-option``
+  disable the use of wheels, and the autobuilding of wheels. (:pull:`2711`))
+  Fixes :issue:`2677`
+
 **6.1.1 (2015-04-07)**
 
 * No longer ignore dependencies which have been added to the standard library,

pip/cmdoptions.py

@@ -35,6 +35,24 @@ def resolve_wheel_no_use_binary(options):
         fmt_ctl_no_use_wheel(control)
 
 
+def check_install_build_global(options, getname=None, check_options=None):
+    """Disable wheels if per-setup.py call options are set.
+
+    :param options: The OptionParser options to update.
+    :param getname: If supplied what to use to get a name from check_options.
+    :param check_options: The options to check, if not supplied defaults to
+        options.
+    """
+    if getname is None:
+        getname = lambda n: getattr(options, n, None)
+    if check_options is None:
+        check_options = options
+    names = ["build_options", "global_options", "install_options"]
+    if any(map(getname, names)):
+        control = options.format_control
+        fmt_ctl_no_use_wheel(control)
+
+
 ###########
 # options #
 ###########

pip/commands/install.py

@@ -194,6 +194,7 @@ def _build_package_finder(self, options, index_urls, session):
 
     def run(self, options, args):
         cmdoptions.resolve_wheel_no_use_binary(options)
+        cmdoptions.check_install_build_global(options)
 
         if options.download_dir:
             options.ignore_installed = True

pip/commands/wheel.py

@@ -125,6 +125,7 @@ def check_required_packages(self):
     def run(self, options, args):
         self.check_required_packages()
         cmdoptions.resolve_wheel_no_use_binary(options)
+        cmdoptions.check_install_build_global(options)
 
         index_urls = [options.index_url] + options.extra_index_urls
         if options.no_index:

pip/req/req_file.py

@@ -4,6 +4,7 @@
 
 from __future__ import absolute_import
 
+from operator import attrgetter
 import os
 import re
 import shlex
@@ -140,6 +141,8 @@ def process_line(line, filename, line_number, finder=None, comes_from=None,
         args_line = ' '.join(args)
         comes_from = '-r %s (line %s)' % (filename, line_number)
         isolated = options.isolated_mode if options else False
+        if options:
+            cmdoptions.check_install_build_global(options, attrgetter, opts)
         # trim the None items
         keys = [opt for opt in opts.__dict__ if getattr(opts, opt) is None]
         for key in keys:

tests/unit/test_req_file.py

@@ -1,3 +1,4 @@
+from optparse import Values
 import os
 import subprocess
 from textwrap import dedent
@@ -11,7 +12,7 @@
                             ReqFileOnleOneOptionPerLineError,
                             ReqFileOptionNotAllowedWithReqError)
 from pip.download import PipSession
-from pip.index import PackageFinder
+from pip.index import FormatControl, PackageFinder
 from pip.req.req_install import InstallRequirement
 from pip.req.req_file import (parse_requirements, process_line, join_lines,
                               ignore_comments)
@@ -68,8 +69,10 @@ class TestProcessLine(object):
     """tests for `process_line`"""
 
     def setup(self):
-        self.options = stub(isolated_mode=False, default_vcs=None,
-                            skip_requirements_regex=False)
+        self.options = stub(
+            isolated_mode=False, default_vcs=None,
+            skip_requirements_regex=False,
+            format_control=pip.index.FormatControl(set(), set()))
 
     def test_parser_error(self):
         with pytest.raises(RequirementsFileParseError):
@@ -335,6 +338,7 @@ def test_install_requirements_with_options(self, tmpdir, finder, session):
         install_option = '--prefix=/opt'
 
         content = '''
+        --only-binary :all:
         INITools==2.0 --global-option="{global_option}" \
                         --install-option "{install_option}"
         '''.format(global_option=global_option, install_option=install_option)
@@ -343,8 +347,12 @@ def test_install_requirements_with_options(self, tmpdir, finder, session):
         with open(req_path, 'w') as fh:
             fh.write(content)
 
-        req = next(parse_requirements(req_path, finder=finder,
-                                      session=session))
+        options = Values()
+        options.format_control = FormatControl(set(), set())
+        options.skip_requirements_regex = None
+        options.isolated_mode = False
+        req = next(parse_requirements(
+            req_path, finder=finder, options=options, session=session))
 
         req.source_dir = os.curdir
         with patch.object(subprocess, 'Popen') as popen:
@@ -357,3 +365,5 @@ def test_install_requirements_with_options(self, tmpdir, finder, session):
             assert call.index(install_option) > \
                 call.index('install') > \
                 call.index(global_option) > 0
+        assert options.format_control.no_binary == set([':all:'])
+        assert options.format_control.only_binary == set([])