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
[fix] Support git ynh app with submodules #533 #174
Conversation
Looks good to me. |
Moul had made an answer about this pr. I need to take time to explore his proposal. |
The Moul answer is here https://dev.yunohost.org/issues/533#change-1803 |
I tried subtrees, it works without this change. But I suggest to merge this contribution to be able to support submodules too. |
I'm calling for a small decision on this one. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't test it, but seems good command and options.
I have merged it (5 positive vote since 5 days) |
I tried to use git submodule for assure the iso sources but yunohost doesn't support it with a non github url because the git clone is not recursive.
Since git 2.9 there is an option to force submodule to just download the last commit --shallow-submodules https://git-scm.com/docs/git-clone , but with the current yunohost (debian jessie) git is in version 2.1 so we should wait before adding this option.
This Merge Request is linked to the ticket https://dev.yunohost.org/issues/533