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

Check for git submodule --recursive support #105

Merged
merged 1 commit into from Sep 24, 2011

Conversation

@nilbus
Copy link
Contributor

nilbus commented Sep 23, 2011

git submodule x --recursive is only supported after git 1.6.5.
See https://github.com/git/git/blob/master/Documentation/RelNotes/1.6.5.txt#L147

Users with older versions of git on the deploy target using remote-cache or similar deploy methods would see this error when deploying:

 ** [10.0.0.1 :: err] Usage: git submodule [--quiet] [--cached] [add [-b branch] <repo> <path>]|[status|init|update [-i|--init] [-N|--no-fetch]|summary [-n|--summary-limit <n>] [<commit>]] [--] [<path>...]|[foreach <command>]|[sync [--] [<path>...]]
    command finished in 27929ms
*** [deploy:update_code] rolling back

This fix was tested on an OSX 10.6 (git 1.7.4.1) and Ubuntu Linux 9.10 (git 1.6.3.3) deploy targets, and against both GNU bash and BSD dash (sh).

git submodule x --recursive is only supported after git 1.6.5.
@leehambley

This comment has been minimized.

Copy link

leehambley commented on a50a4fc Sep 23, 2011

@nilbus I really like this proposed solution, I'll respond to the pull request this evening, or tomorrow morning, but so far looks like a nice fix.

leehambley added a commit that referenced this pull request Sep 24, 2011
Check for Git submodule `--recursive` support.
@leehambley leehambley merged commit ad17ec9 into capistrano:master Sep 24, 2011
mattbrictson pushed a commit to mattbrictson/capistrano that referenced this pull request Aug 24, 2016
Mention do/end stabby lambda syntax
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.