From: John Keeping <john@keeping.me.uk>
To: Junio C Hamano <gitster@pobox.com>
Cc: Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>,
Ramkumar Ramachandra <artagnon@gmail.com>,
Stefan Beller <stefanbeller@googlemail.com>,
Git List <git@vger.kernel.org>
Subject: Re: [PATCH] whatchanged: document its historical nature
Date: Fri, 9 Aug 2013 21:14:47 +0100 [thread overview]
Message-ID: <20130809201447.GH2337@serenity.lan> (raw)
In-Reply-To: <7v38qi63bn.fsf_-_@alter.siamese.dyndns.org>
On Fri, Aug 09, 2013 at 01:01:48PM -0700, Junio C Hamano wrote:
> After doing a bit of archaeology, I now know why "whatchanged" with
> an unwieldy long name persisted in the user's mindset for so long.
>
> My conclusions are:
>
> - It is better to encourage new users to use `log` very early in
> the document;
>
> - It is not sensible to remove the command at this point yet.
> After having used to `log` that does not take diff options for
> close to a year, it is understandable why there are many people
> who are used to type `whatchanged`.
>
> It could be argued that deprecation and retraining of fingers are
> doing favors to the long-time users. But the presense of the
> command is not hurting anybody, other than the new people who may
> stumble upon both and wonder what their differences are. By clearly
> indicating that these two are essentially the same, we would help
> the new people without harming anybody.
>
> -- >8 --
> Subject: [PATCH] whatchanged: document its historical nature
>
> Encourage new users to use 'log' instead. These days, these
> commands are unified and just have different defaults.
>
> 'git log' only allowed you to view the log messages and no diffs
> when it was added in early June 2005. It was only in early April
> 2006 that the command learned to take diff options. Because of
> this, power users tended to use 'whatchanged' that already existed
> since mid May 2005 and supported diff options.
>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> ---
>
> Documentation/git-whatchanged.txt | 41 ++++++++-------------------------------
> 1 file changed, 8 insertions(+), 33 deletions(-)
>
> diff --git a/Documentation/git-whatchanged.txt b/Documentation/git-whatchanged.txt
> index c600b61..6faa200 100644
> --- a/Documentation/git-whatchanged.txt
> +++ b/Documentation/git-whatchanged.txt
> @@ -13,43 +13,18 @@ SYNOPSIS
>
> DESCRIPTION
> -----------
> -Shows commit logs and diff output each commit introduces. The
> -command internally invokes 'git rev-list' piped to
> -'git diff-tree', and takes command line options for both of
> -these commands.
>
> -This manual page describes only the most frequently used options.
> +Shows commit logs and diff output each commit introduces.
>
> +New users are encouraged to use linkgit:git-log[1] instead. The
> +`whatchanged` command is essentially the same as linkgit:git-log[1]
> +run with different defaults that shows a --raw diff outputat the
s/outputat/output at/
Although I wonder if it would be better to say
New users are encouraged to use linkgit:git-log[1] instead. The
`whatchanged` command is essentially the same as linkgit:git-log[1]
with the `--raw` option specified.
> +end.
>
> -OPTIONS
> --------
> --p::
> - Show textual diffs, instead of the Git internal diff
> - output format that is useful only to tell the changed
> - paths and their nature of changes.
> +The command is kept primarily for historical reasons; fingers of
> +many people who learned Git long before `git log` was invented by
> +reading Linux kernel mailing list are trained to type it.
>
> --<n>::
> - Limit output to <n> commits.
> -
> -<since>..<until>::
> - Limit output to between the two named commits (bottom
> - exclusive, top inclusive).
> -
> --r::
> - Show Git internal diff output, but for the whole tree,
> - not just the top level.
> -
> --m::
> - By default, differences for merge commits are not shown.
> - With this flag, show differences to that commit from all
> - of its parents.
> -+
> -However, it is not very useful in general, although it
> -*is* useful on a file-by-file basis.
> -
> -include::pretty-options.txt[]
> -
> -include::pretty-formats.txt[]
>
> Examples
> --------
> --
> 1.8.4-rc2-195-gb76a8e9
next prev parent reply other threads:[~2013-08-09 20:15 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-07 16:00 Remove old forgotten command: whatchanged Ramkumar Ramachandra
2013-08-07 16:51 ` Stefan Beller
2013-08-07 17:50 ` Junio C Hamano
2013-08-07 21:50 ` Stefan Beller
2013-08-08 6:39 ` Junio C Hamano
2013-08-08 4:30 ` Ramkumar Ramachandra
2013-08-08 15:03 ` Matthieu Moy
2013-08-08 15:13 ` Ramkumar Ramachandra
2013-08-08 15:24 ` Matthieu Moy
2013-08-08 17:23 ` Junio C Hamano
2013-08-09 20:01 ` [PATCH] whatchanged: document its historical nature Junio C Hamano
2013-08-09 20:14 ` John Keeping [this message]
2013-08-09 20:57 ` Junio C Hamano
2013-08-12 7:50 ` John Keeping
2013-08-13 15:56 ` Junio C Hamano
2013-08-10 7:04 ` Ramkumar Ramachandra
2013-08-08 17:51 ` Remove old forgotten command: whatchanged Damien Robert
2013-08-08 18:05 ` Ramkumar Ramachandra
2013-08-08 18:06 ` Matthieu Moy
2013-08-08 19:09 ` John Keeping
2013-08-08 19:27 ` Junio C Hamano
2013-08-09 8:29 ` Matthieu Moy
2013-08-09 17:28 ` Junio C Hamano
2013-08-13 7:58 ` Matthieu Moy
2013-08-13 16:00 ` Junio C Hamano
2013-08-08 19:19 ` Junio C Hamano
2013-08-09 0:04 ` Damien Robert
2013-08-09 0:11 ` Junio C Hamano
2013-08-07 18:01 ` Kyle J. McKay
2013-08-07 18:31 ` John Keeping
2013-08-07 18:48 ` Kyle J. McKay
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=20130809201447.GH2337@serenity.lan \
--to=john@keeping.me.uk \
--cc=Matthieu.Moy@grenoble-inp.fr \
--cc=artagnon@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=stefanbeller@googlemail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).