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

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

From
Vasily Titskiy <qehgt0@gmail.com>
Date
May 17, 2016, 17:12 UTC
Message-ID
<CABSveYjmqrPY4OVrYnWeCrjAqZdqOaciCzQ2_soB4akOcQgZpQ@mail.gmail.com>
In-Reply-To
<xmqqa8joy5sv.fsf@gitster.mtv.corp.google.com>
The command does nothing, so it's not needed. There is also a typo in
my patch description, so I'll resend it again with needed changes.
--
  Regards,
  Vasily Titskiy
On Tue, May 17, 2016 at 1:04 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 12 quoted lines
> 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: Junio C HamanoNext: Eric Sunshine
Message 5 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.