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

Re: [PATCH 3/6] git-reset.txt: reset --soft is not a no-op

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 13, 2010, 23:19 UTC
Message-ID
<7v39tdb59e.fsf@alter.siamese.dyndns.org>
In-Reply-To
<AANLkTin1Ht+2Q4v5pyt8p11VmQ275C41rJLMLLuLkEfA@mail.gmail.com>
Jay Soffian <jaysoffian@gmail.com> writes:
Show 26 quoted lines
> On Mon, Sep 13, 2010 at 4:06 AM, Michael J Gruber
> <git@drmicha.warpmail.net> wrote:
>> Make it clearer that git reset --soft actually does something (changing
>> HEAD). While it is meantioned in the previous paragraph already it can
>> be easily overlooked otherwise.
>>
>> Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
>> ---
>>  Documentation/git-reset.txt |    3 ++-
>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/Documentation/git-reset.txt b/Documentation/git-reset.txt
>> index 8fb871c..65f5d07 100644
>> --- a/Documentation/git-reset.txt
>> +++ b/Documentation/git-reset.txt
>> @@ -43,7 +43,8 @@ linkgit:git-add[1]).
>>  --
>>  --soft::
>>        Does not touch the index file nor the working tree at all, but
>> -       requires them to be in a good order. This leaves all your changed
>> +       requires them to be in good order (and sets the head to <commit>,
>> +       just like all modes do). This leaves all your changed
>>        files "Changes to be committed", as 'git status' would
>>        put it.
>
> What does "requires them to be in good order" mean anyway?

Good point. I don't think it means anything now; for that matter I do not think it meant anything even back when that sentence was added in f67545e (Docs for git-reset-script., 2005-09-07). A soft reset doesn't even look at the index nor the working tree, so there is no good nor bad order there.

Just remove it.
Previous: Jay SoffianNext: Michael J Gruber
Message 8 of 22 in “git-reset.txt improvements”
  1. 0/6 git-reset.txt improvementsMichael J Gruber, Sep 13, 2010
  2. 1/6 git-reset.txt: clarify branch vs. branch headMichael J Gruber, Sep 13, 2010
  3. Junio C HamanoSep 14, 2010
  4. 2/6 git-reset.txt: reset does not change files in targetMichael J Gruber, Sep 13, 2010
  5. Junio C HamanoSep 14, 2010
  6. 3/6 git-reset.txt: reset --soft is not a no-opMichael J Gruber, Sep 13, 2010
  7. Jay SoffianSep 13, 2010
  8. Junio C HamanoSep 13, 2010
  9. 4/6 git-reset.txt: use "working tree" consistentlyMichael J Gruber, Sep 13, 2010
  10. 5/6 git-reset.txt: point to git-checkoutMichael J Gruber, Sep 13, 2010
  11. Junio C HamanoSep 14, 2010
  12. 6/6 git-reset.txt: make modes description more consistentMichael J Gruber, Sep 13, 2010
  13. Junio C HamanoSep 14, 2010
  14. Michael J GruberSep 15, 2010
  15. Jay SoffianSep 13, 2010
  16. 0/6 git-reset.txt improvementsMichael J Gruber, Sep 15, 2010
  17. 1/6 git-reset.txt: clarify branch vs. branch headMichael J Gruber, Sep 15, 2010
  18. 2/6 git-reset.txt: reset does not change files in targetMichael J Gruber, Sep 15, 2010
  19. 3/6 git-reset.txt: reset --soft is not a no-opMichael J Gruber, Sep 15, 2010
  20. 4/6 git-reset.txt: use "working tree" consistentlyMichael J Gruber, Sep 15, 2010
  21. 5/6 git-reset.txt: point to git-checkoutMichael J Gruber, Sep 15, 2010
  22. 6/6 git-reset.txt: make modes description more consistentMichael J Gruber, Sep 15, 2010

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.