All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Burton <markb@ordern.com>
To: git@vger.kernel.org
Subject: Re: [RFC PATCH 0/4] deny push to current branch of non-bare repo
Date: Fri, 7 Nov 2008 22:39:26 +0000 (UTC)	[thread overview]
Message-ID: <loom.20081107T222744-932@post.gmane.org> (raw)
In-Reply-To: 20081107220730.GA15942@coredump.intra.peff.net

Jeff King <peff <at> peff.net> writes:

> 
> The short of it is that it's dangerous, we see people confused by it
> (there was another one just yesterday), and it's a FAQ:
> 
>   http://git.or.cz/gitwiki/GitFaq#head-b96f48bc9c925074be9f95c0fce69bcece5f6e73
> 
> The FAQ even says "don't do this until you know what you are doing." So
> the safety valve is configurable, so that those who know what they are
> doing can switch it off.

When I first tried to use git I was bitten by exactly this problem. I know,
RTFM, but when everything is new, it's easy to undervalue the words of wisdom
when you don't understand the bigger picture and the rational behind the advice.

I now happily work with non-bare repositories on my main machine that I push to
from my satellite development machines but, of course, I don't push to the head
branches but, instead, to remote branches and then merge on the main machine.

I wouldn't have wasted as much time getting my head around this if git had
refused to accept the push to the current branch but, instead, issued a suitable
message telling me I probably didn't want to be doing that.

So, from my own experience, I would say this would be a good feature to add.

Cheers,

Mark

  parent reply	other threads:[~2008-11-07 22:40 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-07 22:07 [RFC PATCH 0/4] deny push to current branch of non-bare repo Jeff King
2008-11-07 22:09 ` [PATCH 1/4] t5400: expect success for denying deletion Jeff King
2008-11-09 10:38   ` Jan Krüger
2008-11-07 22:20 ` [PATCH 2/4] t5516: refactor oddball tests Jeff King
2008-11-07 22:22 ` [PATCH 3/4] tests: avoid pushing to current branch of non-bare repo Jeff King
2008-11-07 22:28 ` [PATCH 4/4] receive-pack: deny push " Jeff King
2008-11-07 22:39 ` Mark Burton [this message]
2008-11-07 23:16 ` [RFC PATCH 0/4] " Junio C Hamano
2008-11-08 14:27   ` Jeff King
2008-11-08 15:12     ` Johannes Schindelin
2008-11-08 20:49     ` Junio C Hamano
2008-11-09  1:49       ` Jeff King
2008-11-09 22:12         ` Junio C Hamano
2008-11-12  0:44         ` Kyle Moffett
2008-11-12  8:44           ` Jeff King
2008-11-13  5:22             ` Kyle Moffett
2008-11-13  5:37               ` Jeff King
2008-11-13  6:14                 ` Junio C Hamano
2008-11-13 13:58                   ` Kyle Moffett
2008-11-14  6:37                     ` Jeff King
2008-12-02  2:22     ` Leo Razoumov
2008-12-02  2:29       ` Junio C Hamano
2008-12-02  2:41         ` Leo Razoumov
2008-12-02  2:48       ` Jeff King
2008-12-02  3:08         ` Leo Razoumov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=loom.20081107T222744-932@post.gmane.org \
    --to=markb@ordern.com \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.