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

Re: [PATCH 0/2] stash: handle pathspec magic again

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 18, 2019, 07:02 UTC
Message-ID
<xmqqlg1c1x7v.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<xmqq8sxc6bjt.fsf@gitster-ct.c.googlers.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 5 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>
>> Fine. But in that case, I would appreciate not being reminded of the
>> messiness. Not unless you let me do something about it. Don't put me
>> between a rock and a hard place, please.

Perhaps your attitude is coming from a fundamental misunderstanding of the process. Don't be unnecessarily defensive, as nobody is being hostile or attacking you.

It's not like this is a competition to see if your taste as a developer is better or worse than others (including me). It is a cooperative process and everybody involved is "at fault" if we made a mistake. Ending up with a suboptimal history in 'next' is not creditable solely to you. I am as much to blame, so are others who advocated to merge it to 'next' even it were not ready. Or those who did not speak up before it was too late, for that matter.

> Gagging me won't change the fact that the history we ended up is
> messy.  Without getting reminded of our past mistake(s) ourselves,
> what else encourages us to do better the next time?
Previous: Junio C Hamano
Message 13 of 13 in “stash: handle pathspec magic again”
  1. 0/2 stash: handle pathspec magic againJohannes Schindelin via GitGitGadget, Mar 7, 2019
  2. 1/2 legacy stash: fix "rudimentary backport of -q"Johannes Schindelin via GitGitGadget, Mar 7, 2019
  3. Junio C HamanoMar 11, 2019
  4. 2/2 built-in stash: handle :(glob) pathspecs againJohannes Schindelin via GitGitGadget, Mar 7, 2019
  5. Junio C HamanoMar 11, 2019
  6. Johannes SchindelinMar 11, 2019
  7. Thomas GummererMar 11, 2019
  8. Junio C HamanoMar 8, 2019
  9. Johannes SchindelinMar 8, 2019
  10. Junio C HamanoMar 10, 2019
  11. Johannes SchindelinMar 11, 2019
  12. Junio C HamanoMar 18, 2019
  13. Junio C HamanoMar 18, 2019

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.