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

Clone/Pull submodules recursively #1615

Merged
merged 7 commits into from Aug 1, 2020

Conversation

pachiras
Copy link
Contributor

Hello,

I found run command fails to download the codes of submodule of submodule (the grand daughter module) from the repository. pull and clone commands even fail to download the codes of submodule (the daughter module).

For most of the nextflow users need the submodules downloaded recursively when they run their scripts. So this is my attempt to achieve it.

version 20.05.0-SNAPSHOT build 5335
created 10-05-2020 21:32 UTC (11-05-2020 06:32 JDT)

Signed-off-by: Akira Sekiguchi <akira.sekiguchi@dnaform.jp>
Signed-off-by: Akira Sekiguchi <akira.sekiguchi@dnaform.jp>
Signed-off-by: Akira Sekiguchi <akira.sekiguchi@dnaform.jp>
@pachiras
Copy link
Contributor Author

Hi, Paolo,

I have added -recurse-submodules options to run/clone/pull sub commands, as the git command does. Nextflow's behaviors on those commands should not be affected without the option. Please consider about it.

@pditommaso
Copy link
Member

This looks doable, however currently submodules are controlled via the config file setting the attribute manifest.submodule.

For consistency, I would suggest adding recurseSubmodule. The new feature would be enabled setting

manifest {
  submodule=true
  recurseSubmodules=true
}

Would that work for you?

@pachiras
Copy link
Contributor Author

Thank you for your comment, Paolo.
I'll try to change the code as you said.
By the way our server crashed today. It could take some time for its recovery.
Please wait for a few days.

@pditommaso
Copy link
Member

No hurry!

Signed-off-by: Akira Sekiguchi <akira.sekiguchi@dnaform.jp>
@pachiras
Copy link
Contributor Author

pachiras commented Aug 1, 2020

Hi,

Sorry to be late, but I have changed the code to control the recursive behavior from manifest scope. I hope it is acceptable. Thanks.

@pditommaso
Copy link
Member

Nice thanks!

@pditommaso pditommaso merged commit 5f20ab6 into nextflow-io:master Aug 1, 2020
@pachiras pachiras deleted the submodule_recursive branch August 2, 2020 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants