git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/2 v3] push: better error messages when push.default = tracking

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 3, 2011, 01:43 UTC
Message-ID
<7vbp1t9dxf.fsf@alter.siamese.dyndns.org>
In-Reply-To
<1299096731-14194-1-git-send-email-Matthieu.Moy@imag.fr>
Matthieu Moy <Matthieu.Moy@imag.fr> writes:
> I applied Junio's suggestion to suggest pushing HEAD in detached HEAD
> state. I don't care very much either way indeed (and I didn't want to
> make the message too heavy, just give the user a way to do something).

If I were asked my honest opinion, I would have to say that I do not care too deeply about it either, in the sense that I won't claim that the workflow I thought was most likely to get a new user into the situation to get the message on the detached HEAD would be really the most likely one, but I wouldn't spend too much time on coming up with other scenarios myself even though I admit that there may be some other cases that the updated message is not optimal.

But one thing that I would value (and I would like to see others on the list to also value) is that every change we make is backed by solid thinking---at least I would like to see us being able to explain _why_ we thought that the message after applying this is better than the old one today. Others who will wonder why the message advises the action it advises three months down the road should be able to find out the thinking behind the change, and decide to agree or disagree with it. Without such a trace of clear and solid thinking, i.e. "even though we admit we may not have fully thought things through, at least we considered this and that when we came up with this solution", we would end up going circles, wasting everybody's time.

In any case, I've applied them and pushed the results out.  Thanks.
Previous: Matthieu Moy
Message 12 of 12 in “push: better error messages”
  1. 0/2 push: better error messagesMatthieu Moy, Feb 28, 2011
  2. 1/2 push: better error message when push.default = trackingMatthieu Moy, Feb 28, 2011
  3. 2/2 push: better error messages for detached HEAD and "no destination"Matthieu Moy, Feb 28, 2011
  4. Jay SoffianFeb 28, 2011
  5. Matthieu MoyFeb 28, 2011
  6. Junio C HamanoFeb 28, 2011
  7. 1/2 push: better error messages when push.default = trackingMatthieu Moy, Mar 1, 2011
  8. 2/2 push: better error message when no remote configuredMatthieu Moy, Mar 1, 2011
  9. Junio C HamanoMar 1, 2011
  10. 1/2 push: better error messages when push.default = trackingMatthieu Moy, Mar 2, 2011
  11. 2/2 push: better error message when no remote configuredMatthieu Moy, Mar 2, 2011
  12. Junio C HamanoMar 3, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.