All of lore.kernel.org
 help / color / mirror / Atom feed
From: Han-Wen Nienhuys <hanwen@xs4all.nl>
To: git@vger.kernel.org
Cc: lily-devel <lilypond-devel@gnu.org>
Subject: Re: git patch
Date: Wed, 06 Dec 2006 10:55:09 +0100	[thread overview]
Message-ID: <4576937D.1070402@xs4all.nl> (raw)
In-Reply-To: <Pine.LNX.4.63.0612060157020.28348@wbgn013.biozentrum.uni-wuerzburg.de>

Johannes Schindelin escreveu:
> The nice thing for me about Git: you never lose anything. Unless you say 
> "git prune" (in which case you really should know what you are doing), you 
> do not lose (committed) data.
> 
> Now, I promised to tell you what to do if all the files seem modified. Did 
> you look through "git -p diff"? (BTW with recent Git you only need "git 
> diff" and it will pipe the result into your pager automatically.)

This actually bothers me as well from a UI point of view.  Git-diff is
used both for generating diffs between versions that come from git and
the working tree/index.

I think it would be more logical to show those diffs as part of
git-status and perhaps git-commit, eg.

  git-commit --dry-run <commitoptions>

shows the diff of what would be committed

  git-status --diff

shows diffs of modified files in the working tree.

This makes it more clear what each diff means.

-- 
 Han-Wen Nienhuys - hanwen@xs4all.nl - http://www.xs4all.nl/~hanwen

  parent reply	other threads:[~2006-12-06  9:55 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <4574AC9E.3040506@gmail.com>
     [not found] ` <4574BF70.8070100@lilypond.org>
2006-12-06  0:41   ` git newbie problems Graham Percival
2006-12-06  1:23     ` Jakub Narebski
2006-12-06  9:39       ` Han-Wen Nienhuys
2006-12-06 10:17         ` Jakub Narebski
2006-12-06 10:21           ` Han-Wen Nienhuys
2006-12-06 10:42             ` Johannes Schindelin
2006-12-06  1:35     ` Junio C Hamano
2006-12-06 22:19       ` Daniel Barkalow
2006-12-06  1:52     ` Tom Prince
     [not found]     ` <el5608$952$2@sea.gmane.org>
2006-12-06  8:01       ` Graham Percival
     [not found]   ` <45760545.2010801@gmail.com>
     [not found]     ` <Pine.LNX.4.63.0612060053400.28348@wbgn013.biozentrum.uni-wuerzburg.de>
     [not found]       ` <45761451.8020006@gmail.com>
     [not found]         ` <Pine.LNX.4.63.0612060157020.28348@wbgn013.biozentrum.uni-wuerzburg.de>
2006-12-06  9:55           ` Han-Wen Nienhuys [this message]
2006-12-06 10:00             ` git patch Junio C Hamano
2006-12-06 10:22             ` Jakub Narebski
     [not found]             ` <Pine.LNX.4.63.0612061105220.28348@wbgn013.biozentrum.uni-wuerzburg.de>
     [not found]               ` <457698E0.80207@lilypond.org>
     [not found]                 ` <Pine.LNX.4.63.0612061147540.28348@wbgn013.biozentrum.uni-wuerzburg.de>
2006-12-06 11:27                   ` Han-Wen Nienhuys
     [not found]     ` <20061206.105251.144349770.wl@gnu.org>
2006-12-06 12:27       ` [PATCH] cvs-migration document: make the need for "push" more obvious Johannes Schindelin
2006-12-06 12:32         ` Jakub Narebski
2006-12-06 13:14           ` Johannes Schindelin
2006-12-06 13:27             ` Jakub Narebski
2006-12-06 13:32               ` New users, was " Johannes Schindelin
2006-12-06 14:52           ` J. Bruce Fields
2006-12-06 14:52         ` Han-Wen Nienhuys
2006-12-06 14:58           ` J. Bruce Fields
2006-12-06 15:05             ` Han-Wen Nienhuys
2006-12-06 15:16             ` Johannes Schindelin
2006-12-06 17:19               ` J. Bruce Fields
2006-12-06 17:24                 ` J. Bruce Fields
2006-12-06 17:44                   ` Junio C Hamano
2006-12-07  4:18                     ` [PATCH] Documentation: reorganize cvs-migration.txt J. Bruce Fields
2006-12-07  5:51                       ` Junio C Hamano
2006-12-07 15:21                         ` J. Bruce Fields
2006-12-07 14:28                       ` Johannes Schindelin
2006-12-07 17:43                         ` J. Bruce Fields
2006-12-07 17:50                           ` Johannes Schindelin
2006-12-08  3:34                             ` J. Bruce Fields
2006-12-08  7:25                               ` Junio C Hamano
2006-12-09  3:58                                 ` J. Bruce Fields
2006-12-06 21:02                   ` [PATCH] cvs-migration document: make the need for "push" more obvious Graham Percival
2006-12-06 21:45                     ` J. Bruce Fields

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4576937D.1070402@xs4all.nl \
    --to=hanwen@xs4all.nl \
    --cc=git@vger.kernel.org \
    --cc=lilypond-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.