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

uv does not support --constraint with a URL in double quotes #11592

Closed
turbotankist opened this issue Feb 18, 2025 · 7 comments · Fixed by #11644
Closed

uv does not support --constraint with a URL in double quotes #11592

turbotankist opened this issue Feb 18, 2025 · 7 comments · Fixed by #11644
Assignees
Labels
compatibility Compatibility with a specification or another tool good first issue Good for newcomers help wanted Contribution especially encouraged

Comments

@turbotankist
Copy link

Summary

pip supports this syntax, but uv fails to parse it.

Steps to reproduce:

reqs.txt

--constraint "https://raw.githubusercontent.com/apache/airflow/constraints-2.10.4/constraints-3.11.txt"
...

Actual behavior

uv pip install -r reqs.txt
error: Error parsing included file in `./reqs.txt` at position 86
 Caused by: failed to read from file `"https://raw.githubusercontent.com/apache/airflow/constraints-2.10.4/constraints-3.11.txt"`

Platform

all

Version

0.6.1

Python version

3.11

@turbotankist turbotankist added the bug Something isn't working label Feb 18, 2025
@charliermarsh charliermarsh added compatibility Compatibility with a specification or another tool and removed bug Something isn't working labels Feb 18, 2025
@charliermarsh
Copy link
Member

Need to decide whether we want to support this. It's similar to #3621.

@charliermarsh charliermarsh added good first issue Good for newcomers help wanted Contribution especially encouraged labels Feb 19, 2025
@charliermarsh
Copy link
Member

I think it'd be ok to support this just for --constraint and the other flags that are supported here as a start, since those are intended to mirror the CLI.

@charliermarsh charliermarsh self-assigned this Feb 19, 2025
@charliermarsh
Copy link
Member

Gonna do this since it's easy to do on the plane, unlike things that require Internet.

@notatallshaw
Copy link
Collaborator

notatallshaw commented Feb 20, 2025

FYI, this will be a behavior difference with pip, which unfortunately reads in constraints files specified inside requirements files like this as though they were requirement files 😞

@charliermarsh
Copy link
Member

Sorry, what do you mean?

@charliermarsh
Copy link
Member

(I don't think this is a behavior change; it just allows users to surround things we already supported in quotes.)

@notatallshaw
Copy link
Collaborator

Apologies, I misunderstood the request, as you say this created no behavior difference for uv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility Compatibility with a specification or another tool good first issue Good for newcomers help wanted Contribution especially encouraged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants