all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: David Engster <deng@randomsample.de>
To: "Eric M. Ludlam" <ericludlam@gmail.com>
Cc: "Eric M. Ludlam" <zappo@gnu.org>, "Chong Yidong" <cyd@gnu.org>,
	Lluís <xscript@gmx.net>,
	emacs-devel@gnu.org
Subject: Re: Merging CEDET
Date: Sun, 03 Jun 2012 20:00:53 +0200	[thread overview]
Message-ID: <87aa0kp962.fsf@engster.org> (raw)
In-Reply-To: <4FCB6B01.8020600@siege-engine.com> (Eric M. Ludlam's message of "Sun, 03 Jun 2012 09:47:45 -0400")

[Eric forgot to include emacs-devel in CC, which is why I'm quoting his
message fully at the end.]

Eric M. Ludlam writes:
> On 06/02/2012 03:47 AM, Chong Yidong wrote:
>> Hello CEDET folks,
>>
>> Is the CEDET file-rename branch ready?  If so, now is a good time to
>> start merging into Emacs.

>   From the perspective of content, the current trunk in CEDET bzr
> matches up with CEDET 1.1 closely.

To make things clear: I have merged our 'newtrunk' branch (which
included the changes in 'file-rename') into our development trunk. This
means we are now finally working directly with the new file and
directory structure from Emacs, and most changes from Emacs trunk are
now incorporated into CEDET.

As I've already written some time ago, I've started to write a special
package for merging Emacs<->CEDET. The most important goal is to keep
the granularity of commits in both repositories. For this to work, we
should get our work-flow straight before we start.

My idea is the following and I'd like to hear opinions from the VCS
gurus around here if this is the right approach:

  We use three branches:

    - CEDET trunk (in the following: 'cedet')
    - Emacs trunk (in the following: 'emacs'). Of course, we do not
      care for the full Emacs trunk, but only for the CEDET-related
      files (essentially: lisp/cedet and lisp/emacs-lisp/eieio*).
    - a special branch inside of CEDET upstream (in the following: 'merge')

  The 'merge' branch is special in that it follows Emacs *and* CEDET
  development. It is derived from the old 'file-rename' branch and thus
  has a common history with 'cedet', so that we can do proper merges
  from and to 'cedet'.

The main concept is this:

  - 'merge' follows 'emacs' as close as possible. That means:
    - It must *not* have any files from 'cedet' which should not end
      up in 'emacs'. Most importantly, this means that *everything*
      that is in 'merge' must have signed papers.
    - It follows exactly the Emacs directory structure, meaning that
      EIEIO is in lisp/emacs-lisp.

  - Syncing with 'cedet' happens through *merges*.

  - Syncing with 'emacs' happens through *cherry picking*, which with
    bzr just boils down to applying patches from "bzr diff -c
    <revision>". Alternatively, Lluís "bzr transfer" plugin can be used,
    but I couldn't get it to work. Either way, the special
    CEDET-Emacs-merge package I've written is used to track which
    commits have been merged and which not (and for what reason).

I could think of two alternatives to this approach:

  - Drop the special 'merge' branch and directly cherry-pick between the
    two repositories, hence essentially do what org and Gnus are
    currently doing. However, I think this can only work well if both
    repositories are very similar, and CEDET upstream still contains
    lots of stuff which isn't in Emacs.

  - Use *two* merge branches 'to-emacs' and 'from-emacs', that means one
    for each direction and both with a common history with 'cedet'.
    This was actually the initial idea, but by now I think this approach
    is just over-complicating things and could easily get pretty messy.

-David



Eric's full mail:


>  From a quality perspective it is pretty good in that it passes all
> the unit tests and the key interactive tests.  I am familiar with a
> couple typo type bugs I need to check in from the translation to the
> new file format still.  I'll do that today. From a copyright
> assignment perspective, all is good, though my last employer release
> for changes ends on July 3, so I can help out on any big changes for a
> month, and get another release soon if needed.
>
>   From the perspective of transplanting changes between our branches,
> David has merged many changes from Emacs into CEDET, and Lluis was
> working on a script to make it easy to do, so I added him to the CC
> list.  Since CEDET includes a 'contrib' area that doesn't have
> copyright assignments, you will still need to avoid that.  We have
> dropped explicit support for older Emacs versions so many previous
> conflicts have since been removed.  The test suites have all been
> converted to the new file system, so Emacs can use that if you'd like
> to enable the complete suite in Emacs core also.
>
> All-in-all I think you will be in good shape unless David or Lluis is
> familiar with something I am not.  We may need to do a second merge
> later, since our conversion to the new file name format is still quite
> fresh.
>
> Thanks!
> Eric



  parent reply	other threads:[~2012-06-03 18:00 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-02  7:47 Merging CEDET Chong Yidong
     [not found] ` <4FCB6B01.8020600@siege-engine.com>
2012-06-03 18:00   ` David Engster [this message]
2012-06-03 21:51     ` Lluís
2012-06-04 13:34     ` Stefan Monnier
2012-07-28 15:32     ` Chong Yidong
2012-07-28 15:38       ` Emacs 24.2 merge window (was: Merging CEDET) Bastien
2012-07-28 16:23       ` Merging CEDET David Engster
2012-07-29  3:21         ` Chong Yidong

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=87aa0kp962.fsf@engster.org \
    --to=deng@randomsample.de \
    --cc=cyd@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=ericludlam@gmail.com \
    --cc=xscript@gmx.net \
    --cc=zappo@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 external index

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

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.