Skip to content

Commit

Permalink
allow creating wheels for editable packages
Browse files Browse the repository at this point in the history
  • Loading branch information
sbidoul committed May 19, 2016
1 parent 5685e01 commit e023083
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 15 deletions.
7 changes: 2 additions & 5 deletions pip/wheel.py
Original file line number Diff line number Diff line change
Expand Up @@ -759,11 +759,8 @@ def build(self, autobuilding=False):
if not autobuilding:
logger.info(
'Skipping %s, due to already being wheel.', req.name)
elif req.editable:
if not autobuilding:
logger.info(
'Skipping bdist_wheel for %s, due to being editable',
req.name)
elif autobuilding and req.editable:
pass
elif autobuilding and req.link and not req.link.is_artifact:
pass
elif autobuilding and not req.source_dir:
Expand Down
10 changes: 0 additions & 10 deletions tests/unit/test_wheel.py
Original file line number Diff line number Diff line change
Expand Up @@ -380,16 +380,6 @@ def test_skip_building_wheels(self, caplog):
assert "due to already being wheel" in caplog.text()
assert mock_build_one.mock_calls == []

def test_skip_building_editables(self, caplog):
with patch('pip.wheel.WheelBuilder._build_one') as mock_build_one:
editable = Mock(editable=True, is_wheel=False, constraint=False)
reqset = Mock(requirements=Mock(values=lambda: [editable]),
wheel_download_dir='/wheel/dir')
wb = wheel.WheelBuilder(reqset, Mock())
wb.build()
assert "due to being editable" in caplog.text()
assert mock_build_one.mock_calls == []


class TestWheelCache:

Expand Down

0 comments on commit e023083

Please sign in to comment.