unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juanma Barranquero <lekktu@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: "Óscar Fuentes" <ofv@wanadoo.es>, emacs-devel@gnu.org
Subject: Re: log format for vc-bzr
Date: Fri, 8 Jan 2010 11:28:08 +0100	[thread overview]
Message-ID: <f7ccd24b1001080228p197940e4ob2c78a2e40ac06c@mail.gmail.com> (raw)
In-Reply-To: <83ljg9as4g.fsf@gnu.org>

On Fri, Jan 8, 2010 at 10:00, Eli Zaretskii <eliz@gnu.org> wrote:

> As usual, the Bazaar documentation doesn't say anything about this
> option that can be grokked by Bazaar non-experts.
>
>    --forget-merges
>      Remove pending merge marker, without changing any files.
>
> What is a ``pending merge marker''?

After doing a merge to a branch, bzr status shows the pending merges:

  C:\emacs\trunk> bzr status
  modified:
    myfile.el
  pending merge tips: (use -v to see all merge revisions)
    Juanma Barranquero 2010-01-02 myfile.el: New file.

That means that the next commit will store the changes as a merge:
"[merge]" will appear in the commit description, and you will be able
to use log -n0 to see the history of the merge.

> And how removing it resolves the
> problem at hand?

"bzr revert --pending-merges" removes the info about the local changes
being a merge. The working copy remains as it is (i.e, it includes all
the changes from the merge), but Bazaar "forgets" that they came from
a merge operation. The next commit will be a normal, non-merge commit.

> And if this is the magic wand to leave personal
> commit comments out of the public repository, then shouldn't we add
> this to the recommended workflow on the wiki?

IIUC what Óscar was saying, he meant that --pending-merges can be used
to sanitize a branch, by selectively copying and squashing commits
from one branch into another before merging the changes back into the
trunk. That's not necessarily an easily automated process.

At some point, people has to try and learn, we cannot give everything
as recipes, because there's no one single good way to do it all. With
--pending-merges you can leave personal commit comments out, or
depending of the developer's habits, *all* the branch development
comments out. Imagine that the unicode-2 branch were merged with

   bzr merge ../unicode-2
   bzr revert --pending-merges
   bzr commit -m "Merge unicode-2 branch."

That's much less useful that the normal merge / commit, where the
history of the branch is accessible. So encouraging people to use
--pending-merges as a recipe is much worse than encouraging them to
just read the docs, try things and ask when they do not understand
something.

All this in my VHO, of course.

    Juanma




  parent reply	other threads:[~2010-01-08 10:28 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
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                           ` Juanma Barranquero [this message]
2010-01-08 12:01                             ` log format for vc-bzr 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=f7ccd24b1001080228p197940e4ob2c78a2e40ac06c@mail.gmail.com \
    --to=lekktu@gmail.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=ofv@wanadoo.es \
    /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).