Re: [PATCH 1/2 (v2)] submodule: no [--merge|--rebase] when newly cloned
- From
Jens Lehmann <jens.lehmann@web.de>
- Date
- Feb 17, 2011, 19:41 UTC
- Message-ID
- <4D5D7A06.6050700@web.de>
- In-Reply-To
- <1297959526-8089-1-git-send-email-olsonse@umich.edu>
Am 17.02.2011 17:18, schrieb Spencer E. Olson:
Show 5 quoted lines
> Previously, when a new submodule is cloned by running "git submodule update > [--merge|--rebase]", the newly cloned submodule does not get checked out and a > rebase or merge is incorrectly attempted against an empty working directory. > This patch ignores --rebase or --merge for new submodules and instead simply > checks out the appropriate revision.
Nice work, thanks!
Just a small thing: This problem also happens when the configuration "submodule.<name>.update" is set to either "rebase" or "merge", not only when using the command line options. So perhaps you could reword the topic to something like "submodule: don't merge or rebase when newly cloned" and adjust both commit messages a bit?
Apart from that: Acked-by: Jens Lehmann <Jens.Lehmann@web.de>