Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore pyproject.toml handling to its version 19.0.3 state #6449

Merged
merged 6 commits into from
Apr 29, 2019

Conversation

cjerdonek
Copy link
Member

@cjerdonek cjerdonek commented Apr 27, 2019

This is a candidate PR to restore pip's pyproject.toml handling to how it was with 19.0.3 in response to community feedback on the tracker and elsewhere -- namely so that --no-use-pep517 doesn't have to be added to use editable mode when pyproject.toml is present. It reverts all commits that touched pyproject.py.

As a side effect, this also undoes some refactoring that was done to make the pyproject.py code more testable, and it removes test cases that provided more test coverage. But those things can always be added back later. Merging this will also mean that issue #6314 will need to be addressed in a different way.

@cjerdonek cjerdonek added skip news Does not need a NEWS file entry (eg: trivial changes) C: PEP 517 impact Affected by PEP 517 processing labels Apr 27, 2019
@cjerdonek
Copy link
Member Author

(I had to add the trivial label since changing an existing news entry doesn't seem to satisfy the bot.)

Copy link
Member

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get 19.1.1 out 👍

@gaborbernat
Copy link

Shall we release this @cjerdonek?

openstack-gerrit pushed a commit to openstack/project-config that referenced this pull request May 6, 2019
As pointed out by ajaeger; the wheel build jobs are failing with

 Error installing 'pip from
 git+https://github.com/ianw/pip.git@path-and-hash#egg=pip': editable
 mode is not supported for pyproject.toml-style projects.

This appears to be an issue with pip 19.0 which would be installed on
the build hosts, interacting with projects that have a pyproject.toml
file.

The problem only seems to be related to installing in editable mode,
which we don't really need here on this ephemeral host.  So just
switch to a regular install, with upgrade/reinstall to be sure.

[1] pypa/pip#6449

Change-Id: I006500a2c98f512176fbac4ab426a8719243fc70
openstack-gerrit pushed a commit to openstack/openstack that referenced this pull request May 6, 2019
* Update project-config from branch 'master'
  - wheel-build: Work around pip 19.0 errors
    
    As pointed out by ajaeger; the wheel build jobs are failing with
    
     Error installing 'pip from
     git+https://github.com/ianw/pip.git@path-and-hash#egg=pip': editable
     mode is not supported for pyproject.toml-style projects.
    
    This appears to be an issue with pip 19.0 which would be installed on
    the build hosts, interacting with projects that have a pyproject.toml
    file.
    
    The problem only seems to be related to installing in editable mode,
    which we don't really need here on this ephemeral host.  So just
    switch to a regular install, with upgrade/reinstall to be sure.
    
    [1] pypa/pip#6449
    
    Change-Id: I006500a2c98f512176fbac4ab426a8719243fc70
@lock
Copy link

lock bot commented May 31, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label May 31, 2019
@lock lock bot locked as resolved and limited conversation to collaborators May 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation C: PEP 517 impact Affected by PEP 517 processing skip news Does not need a NEWS file entry (eg: trivial changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants