unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Edward Steere <edward.steere@gmail.com>
To: David Engster <david@engster.org>
Cc: Eli Zaretskii <eliz@gnu.org>,
	Stephen Leake <stephen_leake@stephe-leake.org>,
	emacs-devel@gnu.org
Subject: Re: CEDET Merge
Date: Wed, 18 Jan 2017 12:12:10 +0200	[thread overview]
Message-ID: <E5263DEF-ACC0-4F6F-BF21-7D37945AE075@gmail.com> (raw)
In-Reply-To: <87k29tv0r4.fsf@engster.org>

Patching does sound easier.

Given that we might be moving from upstream entirely it might be worth it to keep the commit history if we can though.  I know that git won't make is easy.

If we do go with patching though, and the changes are going to appear in the changelog then how are we going to go about doing that?  Would we use the add-change-log command on all of the commits since the last merge?

One disadvantage of all of this is that git blame results would be more difficult to interpret because the change and author would be documented elsewhere.

> On 17 Jan 2017, at 11:23 PM, David Engster <david@engster.org> wrote:
> 
> Edward John Steere writes:
>> Our main concern is to maintain the commit history and I think that my
>> original approach of merging to CEDET and then back creates too much
>> noise.
> 
> Merging to the CEDET repo is not needed, since we will abandon it
> anyway.
> 
>> I think that the best way to go about this is to move everything in the
>> CEDET project into folders which mirror their destination in core.
>> Changes will be required for:
>> - The grammar files, which need to be in admin/grammars
>> - The tests, which need to be in test/manual/cedet
>> - The documentation files, which need to move to doc/misc (and
>>   should probably be flattened.)
>> 
>> Once moved we commit, add CEDET as a remote of Emacs and merge
>> CEDET/master allowing unrelated histories.
> 
> I think it will be less work to simply do the diff|patch game and fixing
> a few paths along the way. Since the histories are unrelated, git cannot
> really help you with the merges anyway.
> 
>> (I considered moving the tests, but they have history too and we'd have
>> to start splitting commits to get them across w/o the rest of upstream
>> CEDET.  Additionally there's nothing preventing the tests from being run
>> with CEDET in Emacs core.  Just start it with --no-init add the test
>> folder to the load path, load the relevant test file and run it.)
> 
> I wouldn't worry too much about the history of the tests. The authorship
> of the changes should be clear, but at least I don't care much for
> granularity here.
> 
> -David



  reply	other threads:[~2017-01-18 10:12 UTC|newest]

Thread overview: 153+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-12 19:32 CEDET Merge Edward John Steere
2017-01-12 19:45 ` Eli Zaretskii
2017-01-12 20:27   ` Edward John Steere
2017-01-12 20:10 ` Bastian Beischer
2017-01-12 20:40   ` Edward John Steere
2017-01-16 18:45     ` Edward John Steere
2017-01-16 19:30       ` Eli Zaretskii
2017-01-16 19:55         ` Edward John Steere
2017-01-16 20:06           ` Eli Zaretskii
2017-01-16 20:12             ` Edward John Steere
2017-01-17 15:59               ` Eli Zaretskii
2017-01-17 16:10                 ` Edward John Steere
2017-01-17 16:36                   ` Stephen Leake
2017-01-17 20:36                     ` Edward John Steere
2017-01-17 21:22                       ` Stephen Leake
2017-01-17 21:23                       ` David Engster
2017-01-18 10:12                         ` Edward Steere [this message]
2017-01-18 22:05                           ` David Engster
2017-01-19 18:01                             ` Edward John Steere
2017-01-19 21:57                               ` David Engster
2017-01-19 22:29                                 ` Karl Fogel
2017-01-20 22:20                                   ` David Engster
2017-01-20 22:40                                     ` Stefan Monnier
2017-01-20 22:57                                       ` David Engster
2017-01-21  0:08                                         ` Stefan Monnier
2017-01-21 12:02                                     ` Eli Zaretskii
2017-01-21 18:29                                       ` Glenn Morris
2017-01-21 18:37                                         ` Eli Zaretskii
2017-01-21 18:52                                           ` Glenn Morris
2017-01-21 19:50                                             ` Eli Zaretskii
2017-01-21 22:57                                               ` Paul Eggert
2017-01-22 16:08                                                 ` Eli Zaretskii
2017-01-22 22:00                                             ` David Engster
2017-01-23  1:37                                               ` Paul Eggert
2017-01-22 21:31                                 ` David Engster
2017-01-24 19:02                                   ` Edward John Steere
2017-01-26 19:54                                     ` Edward John Steere
2017-01-26 21:06                                       ` David Engster
2017-01-27  4:38                                         ` Edward Steere
2017-01-27 20:20                                   ` Edward John Steere
2017-01-27 21:04                                     ` David Engster
2017-01-28  9:23                                       ` Edward John Steere
2017-01-29 21:34                                         ` David Engster
2017-01-31 16:51                                           ` Lars Ingebrigtsen
2017-01-31 18:48                                             ` Ted Zlatanov
2017-01-28 13:45                                       ` Edward John Steere
2017-01-17 21:10                     ` David Engster
2017-01-17 21:25                       ` Stephen Leake
2017-01-16 20:26           ` David Engster
2017-01-16 20:37             ` Edward John Steere
2017-01-16 21:13               ` David Engster
2017-01-17  5:21                 ` Edward Steere
2017-01-17 21:06                   ` David Engster
2017-01-17 21:19                     ` Dmitry Gutov
2017-01-17 21:32                       ` David Engster
2017-01-18  3:10                         ` Lee Hinman
2017-01-18  5:31                           ` Edward John Steere
2017-01-18 12:58                             ` Stephen Leake
2017-01-18 21:57                             ` David Engster
2017-01-19 17:51                               ` Edward John Steere
2017-01-19 20:25                                 ` Lee Hinman
2017-01-29 19:25                                   ` John Wiegley
2017-01-21 18:06                               ` Eric Ludlam
2017-09-23 11:38 ` Charles A. Roelli
2017-09-23 12:55   ` Edward John Steere
2017-09-24  8:24     ` Charles A. Roelli
  -- strict thread matches above, loose matches on Subject: below --
2017-09-26 18:31 Edward John Steere
2012-10-02  2:44 CEDET merge Dmitry Gutov
2012-10-02  5:05 ` Chong Yidong
2012-10-02  5:56   ` David Engster
2012-09-16  1:55 Feature freeze on October 1 Chong Yidong
2012-09-16 15:30 ` David Engster
2012-09-16 19:04   ` Stefan Monnier
2012-09-26 20:24     ` CEDET merge (was: Feature freeze on October 1) David Engster
2012-09-30 13:55       ` CEDET merge David Engster
2012-09-30 14:10         ` David Engster
2012-09-30 18:58         ` Glenn Morris
2012-09-30 19:17           ` Paul Eggert
2012-10-01  0:16             ` Glenn Morris
2012-10-01  3:44         ` Chong Yidong
2012-10-01 11:44           ` Eric M. Ludlam
2012-10-01 15:17           ` David Engster
2012-10-01 17:48             ` Chong Yidong
2012-10-01 17:56               ` Chong Yidong
2012-10-02 15:24               ` Chong Yidong
2012-10-04 19:32         ` David Engster
2012-10-06 11:19           ` Chong Yidong
2012-10-06 11:30             ` David Engster
2012-10-06 14:24               ` Chong Yidong
2012-10-06 14:54                 ` Stefan Monnier
2012-10-06 17:29                   ` David Engster
2012-10-06 18:10                     ` Stefan Monnier
2012-10-07 11:19                       ` David Engster
2012-10-06 23:31                     ` Glenn Morris
2012-10-07  1:15                       ` Glenn Morris
2012-10-07 11:03                         ` David Engster
2012-10-27 14:40                           ` David Engster
2012-10-28 18:50                             ` Glenn Morris
2012-11-17  3:23                               ` Glenn Morris
2012-11-18 15:42                                 ` David Engster
2012-11-20  2:45                                   ` Glenn Morris
2012-11-21 13:09                                   ` Chong Yidong
2012-10-07 20:50                 ` David Engster
2012-11-08 12:32         ` Alex Ott
2009-09-28 15:31 Chong Yidong
2009-09-28 17:31 ` Ulrich Mueller
2009-09-28 17:55   ` Chong Yidong
2009-09-28 18:42     ` Ulrich Mueller
2009-09-28 19:30       ` Chong Yidong
2009-09-28 20:03         ` Ulrich Mueller
2009-09-28 20:20           ` Rupert Swarbrick
2009-09-28 22:16             ` Ulrich Mueller
2009-09-28 17:47 ` Eli Zaretskii
2009-09-28 18:00 ` Eli Zaretskii
2009-09-28 18:25   ` Chong Yidong
2009-09-28 19:23     ` Eli Zaretskii
2009-09-28 19:27       ` Andreas Schwab
2009-09-28 18:20 ` Phil Hagelberg
2009-09-28 22:10   ` Chong Yidong
2009-09-28 22:25     ` Phil Hagelberg
2009-09-30 16:38     ` Phil Hagelberg
2009-09-30 17:29       ` Chong Yidong
2009-09-30 21:43         ` Phil Hagelberg
2009-10-01  1:19           ` Chong Yidong
2009-10-01  3:20             ` Phil Hagelberg
2009-10-01  5:14               ` Phil Hagelberg
2009-10-01  7:07                 ` Stefan Monnier
2009-10-02 17:46                   ` Phil Hagelberg
2009-09-28 19:34 ` Andreas Schwab
2009-09-28 20:20   ` Andreas Schwab
2009-09-28 20:20 ` Alan Mackenzie
2009-09-28 21:57   ` Chong Yidong
2009-09-29  4:28 ` Glenn Morris
2009-09-29 11:28   ` Eric M. Ludlam
2009-09-30  9:32 ` Juanma Barranquero
2009-09-30 18:38   ` Eli Zaretskii
2009-09-30 19:32     ` Juanma Barranquero
2009-09-30 10:29 ` Sascha Wilde
2009-10-01 10:58   ` Sascha Wilde
2009-10-01 11:38     ` Eric M. Ludlam
2009-10-01 12:51       ` Sascha Wilde
2009-10-01 16:28         ` Sascha Wilde
2009-10-03 13:07           ` Eric M. Ludlam
2009-10-03 21:01             ` Sascha Wilde
2009-10-06 16:15               ` Sascha Wilde
2009-10-06 17:51                 ` Chong Yidong
2009-10-03 20:10       ` Chong Yidong
2009-10-03 20:31         ` Eric M. Ludlam
2009-10-04  1:44           ` Chong Yidong
2009-10-04  2:30             ` Eric M. Ludlam
2009-10-04  5:52               ` Chong Yidong
2009-10-01  3:58 ` Miles Bader
2009-10-01 11:31   ` Eric M. Ludlam
2009-10-01 14:48     ` Chong Yidong
2009-10-07  3:43 ` Phil Hagelberg
2009-10-07  5:37   ` Chong Yidong
2009-10-07 16:20     ` Eric M. Ludlam

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=E5263DEF-ACC0-4F6F-BF21-7D37945AE075@gmail.com \
    --to=edward.steere@gmail.com \
    --cc=david@engster.org \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=stephen_leake@stephe-leake.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).