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

recurse clone submodules #2427

Merged
merged 1 commit into from
Jun 10, 2020
Merged

recurse clone submodules #2427

merged 1 commit into from
Jun 10, 2020

Conversation

matemax
Copy link
Contributor

@matemax matemax commented May 15, 2020

Fixes: #1828
Closes: #611

Pull Request Check List

  • Added tests for changed code.
    I not found how implement test for these case
  • Updated documentation for changed code.
    I not found docs about clone git dependencies.

@matemax
Copy link
Contributor Author

matemax commented May 15, 2020

closes #1828

@matemax
Copy link
Contributor Author

matemax commented May 15, 2020

@finswimmer, i rebase and reopen(#1826 )

@matemax matemax changed the title recurse-submodules recurse clone submodules May 15, 2020
@finswimmer finswimmer requested a review from a team May 17, 2020 15:27
d3rp pushed a commit to d3rp/clima that referenced this pull request May 25, 2020
context parameters is not implemented, so the order is the same for all
and ordering is based naively on what's been given in the configuration.

V0.5.0-v0.5.0 bump version to 0.5.1

Included python-fire inside the package directory until
python-poetry/poetry#2427 gets merged and a
submodule can be used instead
@matemax
Copy link
Contributor Author

matemax commented Jun 9, 2020

Any Feedback?

Copy link
Member

@finswimmer finswimmer left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Thanks a lot.

@finswimmer finswimmer merged commit d037b59 into python-poetry:master Jun 10, 2020
@fabiopedrosa
Copy link

@finswimmer @matemax for me this merge has broken the use of https:// .... .git schema. e.g. this now fails:

[CalledProcessError]
Command '['git', 'clone', '--recurse-submodules', 'https://github.com/darklow/django-suit.git', 'C:\\Temp\\pypoetry-git-django-su0mrul0xy']' returned non-zero exit status 128.

@finswimmer
Copy link
Member

Hello @fabiopedrosa ,

could you please open a new issue for this and describe how one can reproduce the problem?

fin swimmer

@matemax
Copy link
Contributor Author

matemax commented Aug 4, 2020

Hello @fabiopedrosa, the command

git clone --recurse-submodules https://github.com/darklow/django-suit.git C:\\Temp\\pypoetry-git-django-su0mrul0xy

executes correct

@luisacabs-out
Copy link

Can you please add documentation on how to use this?

@Zeskbest
Copy link

Can you please add documentation on how to use this?

This pr fixes complicated libraries installation.
If a git-sourced library has a submodule, this submodule is not cloned during the install, so library is installed incorrectly. Don't use it. It must be used by default.

@luisacabs-out
Copy link

luisacabs-out commented Aug 26, 2021

@Zeskbest what do you mean it must be used by default?
What I'm trying to do is install a package from a specific module in my private repo. This repo has multiple independent packages. I can do this by running:

poetry run pip git+ssh:/git@github.com/myrepo#subdirectory=my_pkg

But I would like to specify this dependency in my pyproject.toml. Not sure how I should do it, this is not documented as far as I know

@finswimmer
Copy link
Member

@luisacabs-out: What you try is to install a package from a subfolder within a git repository. This is not supported at the moment by poetry. Some links you can follow:

@Zeskbest
Copy link

Zeskbest commented Aug 26, 2021

@Zeskbest what do you mean it must be used by default?
What I'm trying to do is install a package from a specific module in my private repo. This repo has multiple independent packages. I can do this by running:

poetry run pip git+ssh:/git@github.com/myrepo#subdirectory=my_pkg

But I would like to specify this dependency in my pyproject.toml. Not sure how I should do it, this is not documented as far as I know

You should make a library from your repo (see here).
Make a submodule from the subdirectory my_pkg if needed.
Than add it to the .toml file:

[tool.poetry.dependencies.myrepo]
git = "ssh:/git@github.com/myrepo.git"
tag = "v.0.0.1"

Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed add dependency with submodule Add key to allow fetching submodules for git repos
5 participants