unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Óscar Fuentes" <ofv@wanadoo.es>
To: emacs-devel@gnu.org
Cc: Dan Nicolaescu <dann@ics.uci.edu>
Subject: Re: log format for vc-bzr
Date: Tue, 08 Dec 2009 20:19:02 +0100	[thread overview]
Message-ID: <874oo1w9y1.fsf@telefonica.net> (raw)
In-Reply-To: 200912081747.nB8HlwPR021836@godzilla.ics.uci.edu

Dan Nicolaescu <dann@ics.uci.edu> writes:

> The default C-x v l format for bzr does not show merges.
>
> The default C-x v g format shows the version number from the merge:
> Here's an example from bzr itself:
>
>
>                   | from cStringIO import StringIO
> 0.16.5    aaron@a | import shutil
> 0.16.1    aaron@a | import sys
> 0.16.5    aaron@a | import tempfile
> 0.16.1    aaron@a | 
> 0.16.25   aaron@a | from bzrlib import (
>                   |     builtins,
>                   |     delta,
>                   |     diff,
>                   |     errors,
> 0.16.79   aaron@a |     osutils,
> 0.16.25   aaron@a |     patches,
> 0.16.74   aaron@a |     shelf,
> 0.16.72   aaron@a |     textfile,
> 0.16.54   aaron@a |     trace,
> 0.16.64   aaron@a |     ui,
> 0.16.102  aaron@a |     workingtree,
>
> The result is that when using the "l" key binding in vc-annotate emacs
> is not able to find the log for the revision mentioned in the annotate
> buffer. 
>
> This can be solved by passing the "--include-merges" to "bzr log".
> Any reason not to do that?

With the distributed development style that encourages committing often,
the history that comes from the merged branch can be very boring and
irrelevant. Something like this:

15.23 gone lunch
15.24 some refactoring
15.25 seems ready for testing

The important info is in the merge commit:

15 Added feature Foo

and that is what the log should show by default.

OTOH, for long-lived merged branches that implements complex features,
the merged history conveys useful information.

So, in the best Emacs tradition, both modes should be supported :-)

Personally, I'm against showing the merged history by default. If you
need to pass --include-merges for showing a merged commit, that's okay,
but not when the user asks for the log with C-x v l.

It would be interesting to have a method on the log buffer for showing
the merged history at some merge point. Some kind of "expand this one
level".

-- 
Óscar





  reply	other threads:[~2009-12-08 19:19 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-08 17:47 log format for vc-bzr Dan Nicolaescu
2009-12-08 19:19 ` Óscar Fuentes [this message]
2009-12-08 19:47   ` Andreas Schwab
2009-12-08 20:15     ` Óscar Fuentes
2009-12-08 21:10       ` Andreas Schwab
2009-12-08 22:03         ` Dan Nicolaescu
2009-12-08 22:36           ` Óscar Fuentes
2009-12-09  0:27           ` Jason Earl
2009-12-09  8:55             ` Andreas Schwab
2009-12-09  9:55               ` Stephen J. Turnbull
2009-12-09 14:41                 ` Stefan Monnier
2009-12-09 20:29               ` Óscar Fuentes
2009-12-09 21:49                 ` Andreas Schwab
2009-12-09 22:34                   ` Óscar Fuentes
2009-12-09 23:15                     ` Andreas Schwab
2009-12-10  2:03                       ` Stephen J. Turnbull
2009-12-10  9:12                         ` Andreas Schwab
2009-12-10 12:28                           ` Stephen J. Turnbull
2010-01-07 21:23                     ` Thien-Thi Nguyen
2010-01-07 21:47                       ` Óscar Fuentes
2010-01-08  0:40                         ` Thien-Thi Nguyen
2010-01-08  9:00                         ` Eli Zaretskii
2010-01-08 10:09                           ` reversion revulsion [was: log format for vc-bzr] Thien-Thi Nguyen
2010-01-08 10:53                             ` Juanma Barranquero
2010-01-08 13:41                               ` reversion revulsion Thien-Thi Nguyen
2010-01-08 14:32                                 ` Juanma Barranquero
2010-01-08 14:47                                   ` Thien-Thi Nguyen
2010-01-08 11:21                             ` reversion revulsion [was: log format for vc-bzr] Óscar Fuentes
2010-01-08 12:36                               ` Eli Zaretskii
2010-01-08 13:27                                 ` reversion revulsion Óscar Fuentes
2010-01-08 16:54                                   ` Eli Zaretskii
2010-01-08 17:57                                     ` Óscar Fuentes
2010-01-08 13:29                                 ` reversion revulsion [was: log format for vc-bzr] Juanma Barranquero
2010-01-08 13:32                                   ` Juanma Barranquero
2010-01-08 10:28                           ` log format for vc-bzr Juanma Barranquero
2010-01-08 12:01                             ` Eli Zaretskii
2010-01-08 12:24                               ` Juanma Barranquero
2010-01-08 12:52                                 ` Eli Zaretskii
2010-01-08 13:22                                   ` Juanma Barranquero
2010-01-08 13:50                                     ` Andreas Schwab
2010-01-08 14:28                                       ` Juanma Barranquero
2010-01-08 14:39                                   ` Thien-Thi Nguyen
2010-01-08 17:56                             ` bzr Q&A [was Re: log format for vc-bzr] Glenn Morris
2010-01-09 10:35                               ` Eli Zaretskii
2010-01-09 17:58                               ` Stephen J. Turnbull
2010-01-08 10:50                           ` log format for vc-bzr Stephen J. Turnbull
2010-01-08 12:20                             ` Eli Zaretskii
2010-01-08 12:29                               ` Juanma Barranquero
2010-01-08 13:39                               ` Stephen J. Turnbull
2010-01-09 10:51                                 ` Eli Zaretskii
2010-01-08 13:48                               ` Óscar Fuentes
2010-01-08 16:57                                 ` Eli Zaretskii
2010-01-08 17:45                                   ` Óscar Fuentes
2010-01-08 19:35                                     ` Eli Zaretskii
2010-01-08 19:41                                       ` Miles Bader
2010-01-08 21:22                                         ` Eli Zaretskii
2010-01-08 20:46                                       ` Óscar Fuentes
2010-01-08 21:29                                         ` Eli Zaretskii
2010-01-08 21:51                                           ` Óscar Fuentes
2010-01-08 22:24                                             ` Eli Zaretskii
2010-01-08 23:05                                               ` Óscar Fuentes
2010-01-09  8:50                                                 ` Thien-Thi Nguyen
2010-01-09  9:36                                                   ` Andreas Schwab
2010-01-09 13:35                                                     ` Thien-Thi Nguyen
2010-01-09  3:45                                             ` Miles Bader
2010-01-08 11:54                           ` Daniel Clemente
2010-01-08 16:45                             ` Eli Zaretskii
2010-01-09  3:18                               ` Daniel Clemente
2010-01-09  9:37                                 ` Eli Zaretskii
2009-12-09  1:32   ` Stefan Monnier

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=874oo1w9y1.fsf@telefonica.net \
    --to=ofv@wanadoo.es \
    --cc=dann@ics.uci.edu \
    --cc=emacs-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.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).