-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Description
- I am on the latest Poetry version.
- I have searched the issues of this repo and believe that this is not a duplicate.
- If an exception occurs when executing a command, I executed it again in debug mode (
-vvvoption).
- MaxOS 10.14:
- 1.0.0b8:
Issue
When working with git dependency such as [email protected]:sdispater/poetry.git. Working with poetry everything is totally fine things install and function properly and it makes it seem like you have configured poetry correctly.
This falls apart when you try and install that package in another repo. Basically the SdistBuilder outputs a string like [email protected]:sdispater/poetry.git@master. What it actually needs to be is git+ssh://[email protected]/sdispater/poetry.git.
This can easily be fixed in the package by changing the git url to be ssh://[email protected]/sdispater/poetry.git. The issue is there is no way to know that this is a problem until you are trying to consume your work and the terrible output messages from pip when you try and install this will leave you in debugging hell especially since poetry deletes the setup.py file it generates so you can't view it to rerun it.
There are two different ways to handle this problem as I see it, the first would be to make it so that poetry does not accept git urls that are not compliant with PEP-508.
The second option is to upgrade VCSDependency so that it detects urls in this format and converts them to a compliant format when dependency.to_pep_508() is called.
I personally prefer the second because it is easier to copy and paste out of GitHub and would hopefully make the friction less when new users are adopting poetry.
I am happy to implement either, I just need to know the preferred direction.