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

Add support for url dependencies #1260

Merged
merged 1 commit into from Aug 1, 2019
Merged

Add support for url dependencies #1260

merged 1 commit into from Aug 1, 2019

Conversation

@sdispater
Copy link
Member

sdispater commented Jul 28, 2019

Pull Request Check List

  • Added tests for changed code.
  • Updated documentation for changed code.

This PR adds support for URL dependencies.

Adding a URL dependency to the current project can be done in two ways: via the add command or by modifying the pyproject.toml file directly.

poetry add https://example.com/packages/my-package-1.0.0.tar.gz
[tool.poetry.dependencies]
my-package = {url = "https://example.com/packages/my-package-1.0.0.tar.gz"}
@sdispater sdispater added this to the 1.0 milestone Jul 28, 2019
@sdispater sdispater added this to In progress in 1.0 via automation Jul 28, 2019
@sdispater sdispater moved this from In progress to Done in 1.0 Jul 30, 2019
@sdispater sdispater force-pushed the url-dependencies branch from 15737fb to 53a3cae Aug 1, 2019
@sdispater sdispater merged commit f205ac7 into develop Aug 1, 2019
4 checks passed
4 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@sdispater sdispater deleted the url-dependencies branch Aug 1, 2019
@marctorrellas

This comment has been minimized.

Copy link

marctorrellas commented Aug 10, 2019

I tried to install spacy language model like this

(title_detector) MacBook-MacBook-Pro-de-Marc:title_detector marctorrellassocastro$ poetry add https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.1.0/en_core_web_sm-2.1.0.tar.gz

Updating dependencies
Resolving dependencies... (6.6s)


Package operations: 1 install, 0 updates, 0 removals

  - Installing en_core_web_sm (2.1.0 https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.1.0/en_core_web_sm-2.1.0.tar.gz)

[EnvCommandError]
Command ['/Users/marctorrellassocastro/Library/Caches/pypoetry/virtualenvs/title-detector-uYNvqQ-e-py3.7/bin/python', '-m', 'pip', 'install', '--no-deps', 'en-core-web-sm==2.1.0'] errored with the following return code 1, and output:
Collecting en-core-web-sm==2.1.0
  Could not find a version that satisfies the requirement en-core-web-sm==2.1.0 (from versions: )
No matching distribution found for en-core-web-sm==2.1.0
You are using pip version 19.0.3, however version 19.2.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

but didn't work. Doing pip install <url> it works. Am I doing something wrong?

@dpraul dpraul mentioned this pull request Aug 13, 2019
3 of 3 tasks complete
@abn abn mentioned this pull request Aug 20, 2019
1 of 1 task complete
@im-n1

This comment has been minimized.

Copy link

im-n1 commented Sep 26, 2019

Just tried this and doesnt work on tar/zip balls.

{ url = "https://github.com/im-n1/django-seed/archive/bf8b56d.zip" }

doesn't work because of

[SolverProblemError]
Because middleware depends on django-seed (*) which doesn't exist, version solving failed.

Maybe the version check can be skipped in this case or if you must know it we can come up with something like :

{ url = "https://github.com/im-n1/django-seed/archive/bf8b56d.zip", version = "my-epic-version" }

This happends if you fork someones repo on Github, do stuff but the original developer is not responding to your merge request. Then you need to add your repo as dependency which doesn't have PyPI link nor official version number.

@Jamim

This comment has been minimized.

Copy link
Contributor

Jamim commented Sep 26, 2019

Hello @im-n1,

Just tried this and doesnt work on tar/zip balls.

Could you please submit an issue?
Thank you!

@im-n1

This comment has been minimized.

Copy link

im-n1 commented Sep 26, 2019

I just did. Feel free to do your internal issue management. Bye.

@AdamGold

This comment has been minimized.

Copy link

AdamGold commented Nov 6, 2019

Any news on this?

@sdispater sdispater moved this from Done to Closed in 1.0 Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
1.0
  
Closed
5 participants
You can’t perform that action at this time.