From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Glenn Morris Newsgroups: gmane.emacs.devel Subject: Re: /srv/bzr/emacs/emacs-24 r111150: Merge Org (commit c8c217) Date: Tue, 08 Jan 2013 15:53:32 -0500 Message-ID: <8zy5g3bdmb.fsf@fencepost.gnu.org> References: NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Trace: ger.gmane.org 1357678420 15367 80.91.229.3 (8 Jan 2013 20:53:40 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 8 Jan 2013 20:53:40 +0000 (UTC) Cc: Emacs developers To: Bastien Guerry Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jan 08 21:53:57 2013 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1TsgBF-0004nL-GI for ged-emacs-devel@m.gmane.org; Tue, 08 Jan 2013 21:53:53 +0100 Original-Received: from localhost ([::1]:57342 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TsgAz-0007LD-Vr for ged-emacs-devel@m.gmane.org; Tue, 08 Jan 2013 15:53:37 -0500 Original-Received: from eggs.gnu.org ([208.118.235.92]:58389) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TsgAx-0007KH-8h for emacs-devel@gnu.org; Tue, 08 Jan 2013 15:53:36 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TsgAv-0003pr-TE for emacs-devel@gnu.org; Tue, 08 Jan 2013 15:53:35 -0500 Original-Received: from fencepost.gnu.org ([2001:4830:134:3::e]:33591) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TsgAv-0003pn-QM for emacs-devel@gnu.org; Tue, 08 Jan 2013 15:53:33 -0500 Original-Received: from rgm by fencepost.gnu.org with local (Exim 4.71) (envelope-from ) id 1TsgAu-0006yT-Mv; Tue, 08 Jan 2013 15:53:32 -0500 X-Spook: Belknap Jyllandsposten Leitrim CESID Leuken-Baden fraud X-Ran: TP9aveU>z0hP-a|r/1E0]M-+Te;jZhV*~--`3}xICjE&$XJ@{mjs\>{$u|E List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:156160 Archived-At: Thanks for merging it. Now the usual petty griping.. :) Bastien Guerry wrote: > -Copyright @copyright{} 2004--2013 Free Software Foundation, Inc. > +Copyright @copyright{} 2004--2012 Free Software Foundation, Inc. Wrong-o. > =3D=3D=3D modified file 'lisp/org/ChangeLog' > --- a/lisp/org/ChangeLog 2013-01-01 09:11:05 +0000 > +++ b/lisp/org/ChangeLog 2013-01-08 15:31:30 +0000 > @@ -1,3 +1,412 @@ > +2013-01-08 Achim Gratz > + > + * org-compat.el (org-define-obsolete-function-alias) > + (org-define-obsolete-variable-alias): Introduce new compatibility > + macros to obsolete functions and variables. > + > + * org-agenda.el, org-clock.el, org-id.el, org-lparse.el, > + org-protocol.el org.el: Use > + `org-define-obsolete-{function,variable}-alias' instead of > + `define-obsolate{function,variable}-alias'. > + > + * org-faces.el: Define face alias mode-line for XEmacs (it's > + called modeline there). > + > + * org-compat.el (org-condition-case-unless-debug): Do not use > + defalias for special forms, the Emacs 22 byte-compiler does not > + recognize them correctly when compiling macros. Use a macro > + instead and rely on macro expansion. That however makes the > + decision at compile time, which should be acceptable in this case > + since it only affects debugging. > + > + * org-compat.el (org-no-popups): New wrapper macro which let-binds > + the correct variables to suppress popup windows depending on the > + Emacs version in use. This is a compile-time decision when > + byte-compiling. > + > + * org.el (org-get-location, org-switch-to-buffer-other-window): > + Use the wrapper `org-no-popups=C2=B4 to let-bind the correct variables > + for suppression of popup windows. > + > + * org-compat.el (user-error): Defalias to `error=C2=B4 for Emacsen that > + don't have it. > + > + * org-agenda.el (org-agenda-write): Use org-called-interactively-p > + instead of called-interactively-p. > + > + * org.el (org-find-invisible-foreground): Do not use the value of > + variables `default-frame-alist=C2=B4, `initial-frame-alist=C2=B4 and > + `window-system-default-frame-alist=C2=B4 when their symbol is not > + bound. These changes don't seem to fix bugs and serve no purpose for the Org mode that is distributed _with Emacs itself_. The only thing these could possibly do at this stage is (inadvertently) introduce bugs in Emacs 24.3. I'm not saying to remove the changes, just commenting that this is not compatible with the emacs-24 policy. ChangeLog formatting trivia (applies to several places): > * org-agenda.el (org-agenda-get-restriction-and-command): Use > `point-marker'. > > * org-capture.el (org-capture-place-template): Ditto. > > * org-colview-xemacs.el (org-dblock-write:columnview): Ditto. Related changes should not be separated by blank lines. Also, from admin/notes/changelogs: Preferred form for several entries with the same content: * help.el (view-lossage): * kmacro.el (kmacro-edit-lossage): * edmacro.el (edit-kbd-macro): Fix docstring, lossage is now 300 keys. (Rather than anything involving "ditto" and suchlike.) > -\def\orgversionnumber{7.9.2} > -\def\versionyear{2012} % latest update > -\input emacsver.tex > +\def\orgversionnumber{7.9.3} > +\def\versionyear{2013} % latest update > +\def\year{2013} % latest copyright year That's an (undocumented) reversion of a prior Emacs change (the use of emacsver.tex to set \year). I can understand the motivation, but it leaves us with more places in Emacs where the copyright year needs to be updated. If you don't want to copy emacsver.tex to the Org repo, set-copyright in admin/admin.el will need a special entry for orgcard.tex. > +** No GPL manual anymore > + > +There used to be a GPL version of the Org manual, but this is not the > +case anymore, the Free Software Foundation does not permit this. I'd remove that statement from the version in Emacs I were you, because that version has never been under GPL. Personally, I would find a more neutral phrasing as well. Eg: The Org manual is no longer dual-licensed GPL and GFDL, but only GFDL. (Since this is hiding a whole can of worms.) > +The GNU FDL license is now included in the manual directly. Not NEWS-worthy IMO.