On Mon, Jul 30, 2012 at 3:09 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Daniel Graña <dangra@xxxxxxxxx> writes: > >> * Check submodule is correctly initialized and updated after cloning .dotfiles >> >> Signed-off-by: Daniel Graña <dangra@xxxxxxxxx> >> --- > > Thanks. > >> @@ -23,14 +27,27 @@ test_expect_success 'submodule on detached working tree' ' >> git clone --bare ../remote .dotfiles && >> git submodule add ../bundle1 .vim/bundle/sogood && >> test_commit "sogood" && >> + ( >> + unset GIT_WORK_TREE GIT_DIR && >> + cd .vim/bundle/sogood && >> + git rev-parse --verify HEAD >actual && >> + test_cmp ../../../../expect actual >> + ) && >> git push origin master >> - ) && >> + ) > > I do not think you meant to break the && chain here on purpose. > I'll queue with a minor fix-up here. great, thanks. > >> mkdir home2 && >> ( >> cd home2 && >> - export GIT_WORK_TREE="$(pwd)" GIT_DIR="$(pwd)/.dotfiles" && >> git clone --bare ../remote .dotfiles && >> - git submodule update --init >> + export GIT_WORK_TREE="$(pwd)" GIT_DIR="$(pwd)/.dotfiles" && >> + git checkout master && >> + git submodule update --init && >> + ( >> + unset GIT_WORK_TREE GIT_DIR && >> + cd .vim/bundle/sogood && >> + git rev-parse --verify HEAD >actual && >> + test_cmp ../../../../expect actual >> + ) >> ) >> ' >> >> @@ -42,6 +59,7 @@ test_expect_success 'submodule on detached working pointed by core.worktree' ' >> git clone --bare ../remote "$GIT_DIR" && >> git config core.bare false && >> git config core.worktree .. && >> + git checkout master && >> git submodule add ../bundle1 .vim/bundle/dupe && >> test_commit "dupe" && >> git push origin master >> @@ -52,9 +70,8 @@ test_expect_success 'submodule on detached working pointed by core.worktree' ' >> git config core.bare false && >> git config core.worktree .. && >> git pull && >> - git submodule update && >> - git submodule status && >> - test -d .vim/bundle/dupe >> + git submodule update --init && >> + test -e .vim/bundle/dupe/shoot.t >> ) >> ' -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html