Re: [PATCH v2 6/6] clone: print advice on checking out detached HEAD

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH v2 6/6] clone: print advice on checking out detached HEAD

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:52:45

This patch makes 100% sense _if_ we let clone result in a repository with
a detached HEAD, which I am not sure if it is a good idea, or if it is
better to fail the attempt to clone to give incentive to the owner of the
remote repository to fix it.

Re: [PATCH v2 6/6] clone: print advice on checking out detached HEAD

From: Nguyen Thai Ngoc Duy <hidden>
Date: 2016-06-15 22:52:45

2012/1/10 Junio C Hamano [off-list ref]:
This patch makes 100% sense _if_ we let clone result in a repository with
a detached HEAD, which I am not sure if it is a good idea, or if it is
better to fail the attempt to clone to give incentive to the owner of the
remote repository to fix it.
Then a hostile remote can stop users from cloning his repository by
detaching HEAD? That's not nice. On the other hand, if specifying
--branch=<wrong-branch> leads to detached case, then we should
probably refuse to clone. But that should happen before transferring
the pack.
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help