Thanks for revising, it is looking a bit better. A few comments still, though: On Tue, Aug 25, 2009 at 09:20:06PM +0400, Kirill A. Korinskiy wrote: > Sometimes (especially on production systems) we need to use only one > remote branch for building software. It really annoying to clone > origin and then swith branch by hand everytime. So this patch provide > functionality to clone remote branch with one command without using > checkout after clone. Typos: s/It/It's/ s/swith/switch/ s/provide/provides/ s/clone remote/clone a remote/ > t/t5706-clone-brnach.sh | 31 +++++++++++++++++++++++++++++++ Typo: s/brnach/branch/ :) > +test_expect_success 'clone' ' > + > + git clone parent clone && > + (cd clone && git rev-parse --verify refs/remotes/origin/master) > + > +' > + > +test_expect_success 'clone -b' ' > + > + git clone -b two parent clone-b && > + (cd clone && git rev-parse --verify refs/remotes/origin/two) > + > +' Is this really testing the right thing? Shouldn't you always have refs/remotes/origin/*, no matter what "-b" says? The difference should be that HEAD in the first test will point to 'master', and to 'two' in the second test. -Peff -- 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