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

Re: [PATCH v2] Ignore dirty submodule states during stash

From
Junio C Hamano <gitster@pobox.com>
Date
May 17, 2016, 17:04 UTC
Message-ID
<xmqqa8joy5sv.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<CABSveYgsRU+L5qvY_evwekaRU7-oL_2Mmxt6L1u2dYM7Xfq2rw@mail.gmail.com>
Vasily Titskiy <qehgt0@gmail.com> writes:
> You're right, it's redundant here. Should I resubmit the path without this line?

I wasn't pointing out that it was not needed. I was only asking what it was meant to do.

If you now think it shouldn't have been there, that merely means your code was wrong. It does not mean I'm right ;-)

With that line removed, would the patch now becomes right? Are there other bugs?

Previous: Vasily TitskiyNext: Vasily Titskiy
Message 4 of 6 in “Ignore dirty submodule states during stash”
  1. Ignore dirty submodule states during stashVasily Titskiy, May 17, 2016
  2. Junio C HamanoMay 17, 2016
  3. Vasily TitskiyMay 17, 2016
  4. Junio C HamanoMay 17, 2016
  5. Vasily TitskiyMay 17, 2016
  6. Eric SunshineMay 17, 2016

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.