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

Transitive dependencies via optional dependency aliases fails URL installation #4152

Closed
ringohoffman opened this issue Jun 7, 2024 · 1 comment · Fixed by #4155
Closed
Assignees
Labels
bug Something isn't working

Comments

@ringohoffman
Copy link

Continuing my report from the comment here:

I am still seeing a failure to install transitive URL dependencies when using optional dependency aliases as in:

[project]
name = "transitive_optional_dependency"
dynamic = ["version"]

[project.optional-dependencies]
all = [
    "transitive_optional_dependency[docs]",
]
docs = [
    "mike @ git+https://github.com/jimporter/mike@3f7d756#egg=mike",
]

I've provided a minimally reproducible repo here: https://github.com/ringohoffman/transitive-optional-dependency

With steps:

$ git clone git@github.com:ringohoffman/transitive-optional-dependency.git
$ cd transitive-optional-dependency
$ conda create -n tod python=3.10 -y
$ conda activate tod
$ pip install uv
Collecting uv
  Using cached uv-0.2.9-py3-none-macosx_11_0_arm64.whl.metadata (33 kB)
Using cached uv-0.2.9-py3-none-macosx_11_0_arm64.whl (9.7 MB)
Installing collected packages: uv
Successfully installed uv-0.2.9
$ uv pip install -e .[all]
error: Package `mike` attempted to resolve via URL: git+https://github.com/jimporter/mike@3f7d756#egg=mike. URL dependencies must be expressed as direct requirements or constraints. Consider adding `mike @ git+https://github.com/jimporter/mike@3f7d756#egg=mike` to your dependencies or constraints file.

cc: @charliermarsh

@charliermarsh
Copy link
Member

Ah in a recursive extra? Maybe we're missing that case, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants