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

Fix global project dependencies (wheels -> wheel) #182

Merged
merged 2 commits into from Nov 30, 2020
Merged

Fix global project dependencies (wheels -> wheel) #182

merged 2 commits into from Nov 30, 2020

Conversation

roywes
Copy link
Contributor

@roywes roywes commented Nov 30, 2020

wheels is among the minimum dependencies injected for global projects resulting in crashes.

This change corrects the typo to wheel.

Below is an example traceback of the error

======== Start resolving requirements ========
	pip
	setuptools
	wheels
	Adding requirement pip
	Adding requirement setuptools
	Adding requirement wheels
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/pdm/resolver/core.py", line 23, in _merge_into_criterion
    crit = self.state.criteria[name]
KeyError: 'wheels'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 310, in resolve
    name, crit = self._merge_into_criterion(r, parent=None)
  File "/usr/local/lib/python3.9/site-packages/pdm/resolver/core.py", line 25, in _merge_into_criterion
    crit = Criterion.from_requirement(self._p, requirement, parent)
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 83, in from_requirement
    raise RequirementsConflicted(criterion)
resolvelib.resolvers.RequirementsConflicted: Requirements conflict: <NamedRequirement wheels>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/pdm", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/pdm/core.py", line 71, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pdm/core.py", line 106, in main
    raise err.with_traceback(traceback)
  File "/usr/local/lib/python3.9/site-packages/pdm/core.py", line 102, in main
    f(options.project, options)
  File "/usr/local/lib/python3.9/site-packages/pdm/cli/commands/install.py", line 33, in handle
    actions.do_lock(project, strategy="all")
  File "/usr/local/lib/python3.9/site-packages/pdm/cli/actions.py", line 61, in do_lock
    mapping, dependencies, summaries = resolve(
  File "/usr/local/lib/python3.9/site-packages/pdm/resolver/core.py", line 135, in resolve
    result = resolver.resolve(requirements)
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 445, in resolve
    state = resolution.resolve(requirements, max_rounds=max_rounds)
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 312, in resolve
    raise ResolutionImpossible(e.criterion.information)
resolvelib.resolvers.ResolutionImpossible: [RequirementInformation(requirement=<NamedRequirement wheels>, parent=None)]

Pull Request Check List

  • A news fragment is added in news/ describing what is new.
  • Test cases added for changed code.

Describe what you have changed in this PR.

`wheels` is among the minimum dependencies injected for global projects resulting in crashes.

This change corrects the typo to `wheel`.

Below is an example traceback of the error

```
======== Start resolving requirements ========
	pip
	setuptools
	wheels
	Adding requirement pip
	Adding requirement setuptools
	Adding requirement wheels
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/pdm/resolver/core.py", line 23, in _merge_into_criterion
    crit = self.state.criteria[name]
KeyError: 'wheels'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 310, in resolve
    name, crit = self._merge_into_criterion(r, parent=None)
  File "/usr/local/lib/python3.9/site-packages/pdm/resolver/core.py", line 25, in _merge_into_criterion
    crit = Criterion.from_requirement(self._p, requirement, parent)
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 83, in from_requirement
    raise RequirementsConflicted(criterion)
resolvelib.resolvers.RequirementsConflicted: Requirements conflict: <NamedRequirement wheels>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/pdm", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/pdm/core.py", line 71, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pdm/core.py", line 106, in main
    raise err.with_traceback(traceback)
  File "/usr/local/lib/python3.9/site-packages/pdm/core.py", line 102, in main
    f(options.project, options)
  File "/usr/local/lib/python3.9/site-packages/pdm/cli/commands/install.py", line 33, in handle
    actions.do_lock(project, strategy="all")
  File "/usr/local/lib/python3.9/site-packages/pdm/cli/actions.py", line 61, in do_lock
    mapping, dependencies, summaries = resolve(
  File "/usr/local/lib/python3.9/site-packages/pdm/resolver/core.py", line 135, in resolve
    result = resolver.resolve(requirements)
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 445, in resolve
    state = resolution.resolve(requirements, max_rounds=max_rounds)
  File "/usr/local/lib/python3.9/site-packages/resolvelib/resolvers.py", line 312, in resolve
    raise ResolutionImpossible(e.criterion.information)
resolvelib.resolvers.ResolutionImpossible: [RequirementInformation(requirement=<NamedRequirement wheels>, parent=None)]
```
Copy link
Collaborator

@frostming frostming left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution

@frostming frostming merged commit fd2d9f4 into pdm-project:master Nov 30, 2020
@roywes roywes deleted the patch-1 branch November 30, 2020 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants