unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Confusing "bzr log" as result of merges
@ 2011-05-21  7:33 Eli Zaretskii
  2011-05-21  8:36 ` Andreas Schwab
                   ` (4 more replies)
  0 siblings, 5 replies; 45+ messages in thread
From: Eli Zaretskii @ 2011-05-21  7:33 UTC (permalink / raw)
  To: emacs-devel

Sometimes, you want to know when was a certain file last modified.
This is when "bzr log" is useful, if invoked with the name of that
file.  However, it looks like the way we merge other branches, and in
particular emacs-23, makes this harder than it needs to be.

An example of this difficulty is below:

With the current trunk, type this command:

  bzr log -l1 --line --include-merges src/xdisp.c

You will see this as output:

  104201: Glenn Morris 2011-05-12 [merge] Merge from emacs-23; up to r100577.

However, neither "bzr status" nor "bzr diff" will show any changes for
xdisp.c in that revision.  The reason, it seems, is this:

  bzr log -l2 --line --include-merges src/xdisp.c

   => 104201: Glenn Morris 2011-05-12 [merge] Merge from emacs-23; up to r100577.
        99634.2.937: Eli Zaretskii 2011-05-09 Backport revisions 103939.1.41..103939.1.44 (inclusive) from trunk (bug#8623)

and revision 99634.2.937 indeed made a few changes to xdisp.c.  But
that version was merged to the trunk much earlier, in revision 104021,
and "bzr annotate" still shows that the changes attributed to revision
99634.2.937 were made in 103939.1.41, not in 104201 nor in 99634.2.937:

  103939.1.42  eggert@cs.ucla.edu            20110425 |       it->dpend = v->contents + v->header.size;

I think this happens because bzr is smart enough to know that the
revision which changed this line is already on the trunk, and so it
doesn't record any changes for xdisp.c in the merge-commit or 104201.
Bazaar is doing TRT, but the net result for the user is confusion and
extra time spent when trying to figure out who made this specific
change and why.

What can we do to avoid this confusion as result of merges?  And why
was 99634.2.937 merged to the trunk, even though the log message
clearly says it's a backport?  If we avoid merging such backported
revisions, will this problem go away?



^ permalink raw reply	[flat|nested] 45+ messages in thread

end of thread, other threads:[~2011-05-23 12:58 UTC | newest]

Thread overview: 45+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-21  7:33 Confusing "bzr log" as result of merges Eli Zaretskii
2011-05-21  8:36 ` Andreas Schwab
2011-05-21  8:44   ` Eli Zaretskii
2011-05-21  8:48     ` Eli Zaretskii
2011-05-21  9:01     ` Sven Joachim
2011-05-21  9:16       ` Eli Zaretskii
2011-05-21  9:27         ` Sven Joachim
2011-05-21  9:30           ` Eli Zaretskii
2011-05-21 10:27             ` Sven Joachim
2011-05-21  9:02     ` Andreas Schwab
2011-05-21  9:20       ` Eli Zaretskii
2011-05-21  9:38         ` Andreas Schwab
2011-05-21 10:33           ` Eli Zaretskii
2011-05-21 10:54             ` Andreas Schwab
2011-05-21 10:38 ` Eli Zaretskii
2011-05-21 10:56   ` Andreas Schwab
2011-05-21 11:27     ` Eli Zaretskii
2011-05-21 12:23       ` Andreas Schwab
2011-05-21 12:28         ` Eli Zaretskii
2011-05-21 10:57   ` Sven Joachim
2011-05-21 13:04   ` Eli Zaretskii
2011-05-21 13:37     ` Andreas Schwab
2011-05-21 14:46       ` Eli Zaretskii
2011-05-21 15:18         ` Andreas Schwab
2011-05-21 15:51           ` Eli Zaretskii
2011-05-21 16:39             ` Andreas Schwab
2011-05-21 16:21 ` Óscar Fuentes
2011-05-21 16:42   ` Andreas Schwab
2011-05-21 17:00     ` Óscar Fuentes
2011-05-21 17:06       ` Andreas Schwab
2011-05-21 17:19         ` Óscar Fuentes
2011-05-21 17:52           ` Andreas Schwab
2011-05-22 11:33           ` Juanma Barranquero
2011-05-21 16:48   ` Eli Zaretskii
2011-05-21 17:09     ` Óscar Fuentes
2011-05-21 18:56       ` Eli Zaretskii
2011-05-21 17:27 ` Glenn Morris
2011-05-21 19:02   ` Eli Zaretskii
2011-05-22  1:46     ` Glenn Morris
2011-05-22 14:14 ` Stefan Monnier
2011-05-22 17:28   ` Eli Zaretskii
2011-05-22 19:15     ` Stefan Monnier
2011-05-23  3:01       ` Eli Zaretskii
2011-05-23 12:02         ` Stefan Monnier
2011-05-23 12:58           ` Eli Zaretskii

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).