unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Kyle Meyer <kyle@kyleam.com>
To: emacs-devel@gnu.org
Cc: bzg@gnu.org
Subject: Re: org-merger questions
Date: Thu, 22 Jun 2017 14:11:11 -0400	[thread overview]
Message-ID: <87k2437v0g.fsf@kyleam.com> (raw)
In-Reply-To: <87mv906mfl.fsf@kyleam.com>

Hi Rasmus,

Kyle Meyer <kyle@kyleam.com> writes:

> Rasmus <rasmus@gmx.us> writes:
>
>> I had to do a few whitespace cleanup to be able to commit.  We should
>> backport those, I guess.
>
> Hmm, I was hoping I had already caught most of them, but maybe the
> issues you encountered were in the few files changed in
> scratch/org-mode-merge that I didn't look at.  I only checked
> lisp/org/*.el, lisp/doc/misc/org.texi, and etc/refcards/orgcard.tex.

It seems that etc/ORG-NEWS, od-manifest-schema-v1.2-os.rnc, and
od-schema-v1.2-os.rnc all required whitespace cleanups.  These cleanups
are now included in Org's maint (and cherry-picked to the emacs-sync branch).

> A few problems in scratch/org-mode-merge:

[...]

>   * I made the mistake of not looking at ORG-NEWS for backports, so
>     scratch/org-mode-merge is reverting a few Emacs commits here.  I'll
>     backport these

I've backported all the changes to ORG-NEWS in maint (and cherry-picked
them to the emacs-sync branch).

Please consider using Org files from the emacs-sync branch instead of
the release_9.0.9 tag.  The emacs-sync branch is a superset of the last
release, but it contains a few more changes that are specific for Emacs.

These are changes that should not be in the maint branch but should be
in the Emacs repo.  In other words, these are the changes that we should
make sure not to overwrite in a sync.  For example, Org org.texi has
"@include org-version.inc", but in the Emacs repo this should be "@set
VERSION 9.0.9" instead.

You can see these changes in the Org repo with

    git diff release_9.0.9..origin/emacs-sync

(This diff also includes the whitespace cleanup and ORG-NEWS backports
that were cherry-picked to emacs-sync because they are in the maint
branch after release_9.0.9 but should be in the 9.0.9 sync.)

> [...] and I can also add your ORG-NEWS formatting changes to the Org repo.

I don't understand the formatting changes made in 5cfdf8dd17 (; Fix
ORG-NEWS formatting in previous commit, 2017-06-22).  Aside from fixing
the alignment of a table that already seems correct in Org's ORG-NEWS,
all the changes seem to be space to tab conversions.  Are these
necessary?

-- 
Kyle



  reply	other threads:[~2017-06-22 18:11 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-21 15:09 org-merger questions Rasmus
2017-06-21 15:23 ` Kaushal Modi
2017-06-21 16:16   ` Bastien Guerry
2017-06-22 10:31     ` Rasmus
2017-06-22 10:46       ` Yuri Khan
2017-06-22 16:01       ` Kyle Meyer
2017-06-22 18:11         ` Kyle Meyer [this message]
2017-06-23  9:04           ` Rasmus
2017-06-23 14:19             ` Kyle Meyer
2017-06-23  8:47         ` Rasmus
2017-06-23 14:30           ` Kyle Meyer
2017-06-23 14:37             ` Rasmus
2017-06-25 16:10       ` Kaushal Modi
2017-06-29  8:54         ` Rasmus
2017-06-29  9:12           ` Bastien Guerry
2017-06-29 16:13             ` John Wiegley
2017-07-03  7:19               ` Bastien
2017-07-03 15:56                 ` Emacs master now updated to Org 9.0.9 (Was: org-merger questions) Kaushal Modi
2017-07-04  7:54                   ` Emacs master now updated to Org 9.0.9 Bastien Guerry
2017-07-13  0:36                     ` numbchild
2017-10-02  9:25         ` org-merger questions Bastien Guerry
2017-10-02  9:41           ` Rasmus
2017-10-02  9:46             ` Rasmus
2017-10-02 16:05               ` Eli Zaretskii
2017-10-11 22:31                 ` Org mode update missing in NEWS (Was: Re: org-merger questions) Kaushal Modi
2017-10-12  7:54                   ` Eli Zaretskii
2017-10-15 10:37                     ` Org mode update missing in NEWS Rasmus
2017-06-21 15:27 ` org-merger questions Kyle Meyer

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=87k2437v0g.fsf@kyleam.com \
    --to=kyle@kyleam.com \
    --cc=bzg@gnu.org \
    --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).