all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: nljlistbox2@gmail.com (N. Jackson)
To: Eli Zaretskii <eliz@gnu.org>
Cc: 23917@debbugs.gnu.org, Robert Pluim <rpluim@gmail.com>,
	Stefan Monnier <monnier@iro.umontreal.ca>
Subject: bug#23917: 25.0.95; commit 3a9d6296b35e5317c497674d5725eb52699bd3b8 causing org-capture to error out
Date: Thu, 21 Jul 2016 04:52:42 -0300	[thread overview]
Message-ID: <87h9bj2zsl.fsf_-_@moondust.awandering> (raw)
In-Reply-To: <83lh0y24y6.fsf@gnu.org> (Eli Zaretskii's message of "Mon, 18 Jul 2016 21:09:53 +0300")

At 21:09 +0300 on Monday 2016-07-18, Eli Zaretskii wrote:

> My suggestion to fix this is below.  I ask for opinions on (1) whether
> this looks like TRT, (2) whether it is safe enough for emacs-25, and
> (3) whether someone has better ideas.  If someone thinks I've
> misunderstood the issue, don't hesitate to explain why, because
> frankly it feels very strange to find bugs that seem to have existed
> since 1990.
>
> diff --git a/lisp/subr.el b/lisp/subr.el
> index e9e19d3..1bb1cb3 100644
> --- a/lisp/subr.el
> +++ b/lisp/subr.el
> @@ -3466,7 +3466,7 @@ save-match-data
>    ;; if you need to recompile all the Lisp files using interpreted code.
>    (declare (indent 0) (debug t))
>    (list 'let
> -	'((save-match-data-internal (match-data)))
> +	'((save-match-data-internal (match-data 'integers)))
>  	(list 'unwind-protect
>  	      (cons 'progn body)
>  	      ;; It is safe to free (evaporate) markers immediately here,

FWIW on my system applying this patch does not resolve the org-capture
issue. I'm testing with org-20160718 from GNU Elpa and latest Emacs 25
branch from the git (Repository revision: 4157159a37b43712440da91a45a6d5f71eb96e8a).

With these versions of Org and Emacs and your patch applied, with a
recipe similar to that posted by Robert Pluim on 2016-07-18,
specifically

  src/emacs -Q

  M-: (custom-set-variables '(package-selected-packages (quote (org-20160718)))) RET
  M-x package-initialize RET

  C-x C-f		; find file.
  C-S-backspace		; kill-whole-line.
  ~/.notes RET		; Open the file expected by default capture template.
  M-x org-mode RET	; put the buffer into Org Mode.
  M-x org-capture RET t ; Run the default "Task" capture template bound to the t key.

I get the error: 

  org-capture: Capture template ‘t’: Match data clobbered by buffer modification hooks

. I also get a similar error with your patch and my full configuration
loaded and using my own capture templates:

  org-capture: Capture abort: (error Match data clobbered by buffer modification hooks)

.

The results above are same as I get if I do not apply your patch.

[On the other hand, with the same version of Org as above and Emacs from the
25.0.95 tarball, I do not see these error.]






  parent reply	other threads:[~2016-07-21  7:52 UTC|newest]

Thread overview: 109+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-15 16:39 org-capture: Capture template ‘g’: Match data clobbered by buffer modification hooks Alex Bennée
2016-07-15 17:39 ` Eli Zaretskii
2016-07-15 19:12   ` Alex Bennée
2016-07-15 19:37     ` Noam Postavsky
2016-07-15 21:56       ` Alex Bennée
2016-07-16 16:01       ` Please consider making Bug #23917 a blocker for 25.1 (was Re: org-capture: Capture template ‘g’: Match data clobbered by buffer modification hooks) N. Jackson
2016-07-16 16:11         ` Eli Zaretskii
2016-07-16 17:09           ` John Wiegley
2016-07-17 17:28             ` Alex Bennée
2016-07-17 17:41               ` Eli Zaretskii
2016-07-18 12:24                 ` bug#23917: " Robert Pluim
2016-07-18 12:24                 ` Robert Pluim
2016-07-18 18:09                   ` Eli Zaretskii
2016-07-18 18:09                   ` Eli Zaretskii
2016-07-18 19:04                     ` John Wiegley
2016-07-18 19:04                     ` John Wiegley
2016-07-18 19:10                       ` Eli Zaretskii
2016-07-18 19:10                       ` Eli Zaretskii
2016-07-19  0:58                     ` Stefan Monnier
2016-07-19  0:58                     ` Stefan Monnier
2016-07-19  2:40                       ` Eli Zaretskii
2016-07-19  4:48                         ` Stefan Monnier
2016-07-19  4:48                         ` Stefan Monnier
2016-07-19 15:35                           ` Eli Zaretskii
2016-07-19 15:35                           ` Eli Zaretskii
2016-07-19 16:03                             ` Stefan Monnier
2016-07-19 16:03                             ` Stefan Monnier
2016-07-19 16:13                               ` Eli Zaretskii
2016-07-19 16:13                               ` Eli Zaretskii
2016-07-19 17:05                                 ` bug#23917: [O] " Alex Bennée
2016-07-19 17:20                                   ` Eli Zaretskii
2016-07-19 17:45                                     ` bug#23917: " Alex Bennée
2016-07-19 17:45                                     ` bug#23917: [O] " Alex Bennée
2016-07-19 18:07                                       ` bug#23917: " Sebastian Wiesner
2016-07-19 18:07                                       ` bug#23917: [O] " Sebastian Wiesner
2016-07-19 18:44                                       ` Eli Zaretskii
2016-07-19 18:44                                       ` bug#23917: " Eli Zaretskii
2016-07-20  9:48                                         ` bug#23917: " Alex Bennée
2016-07-20  9:48                                         ` bug#23917: [O] " Alex Bennée
2016-07-20 14:59                                           ` bug#23917: " Eli Zaretskii
2016-07-20 14:59                                           ` bug#23917: [O] " Eli Zaretskii
2016-07-19 17:20                                   ` Eli Zaretskii
2016-07-19 17:05                                 ` Alex Bennée
2016-07-20  1:50                                 ` Stefan Monnier
2016-07-20 14:55                                   ` Eli Zaretskii
2016-07-20 14:55                                   ` Eli Zaretskii
2016-07-20 18:19                                     ` Stefan Monnier
2016-07-20 18:55                                       ` Eli Zaretskii
2016-07-20 18:55                                       ` Eli Zaretskii
2016-07-20 20:54                                         ` Stefan Monnier
2016-07-20 20:54                                         ` Stefan Monnier
2016-07-21  0:56                                           ` npostavs
2016-07-21  0:56                                           ` npostavs
2016-07-21  1:47                                             ` Stefan Monnier
2016-07-21  2:34                                               ` Noam Postavsky
2016-07-21  2:34                                               ` Noam Postavsky
2016-07-21  3:06                                                 ` Stefan Monnier
2016-07-21  3:06                                                 ` Stefan Monnier
2016-07-21  1:47                                             ` Stefan Monnier
2016-07-21  2:43                                             ` Eli Zaretskii
2016-07-21  3:00                                               ` npostavs
2016-07-21 14:26                                                 ` Eli Zaretskii
2016-07-22  1:08                                                   ` npostavs
2016-07-22  1:08                                                   ` npostavs
2016-07-22  6:43                                                     ` Eli Zaretskii
2016-07-22 14:01                                                       ` Robert Pluim
2016-07-22 14:01                                                       ` Robert Pluim
2016-07-22 19:30                                                       ` Nicolas Petton
2016-07-22 19:30                                                       ` Nicolas Petton
2016-07-23  4:19                                                       ` npostavs
2016-07-23  4:19                                                       ` npostavs
2016-07-22  6:43                                                     ` Eli Zaretskii
2016-07-23  0:42                                                     ` bug#23917: Re: bug#23917: 25.0.95; commit 3a9d6296b35e5317c497674d5725eb52699bd3b8 causing org-capture to error out N. Jackson
2016-07-23  7:38                                                       ` Eli Zaretskii
2016-07-23  7:38                                                       ` Eli Zaretskii
2016-07-23  0:42                                                     ` bug#23917: " N. Jackson
2016-07-21 14:26                                                 ` bug#23917: Please consider making Bug #23917 a blocker for 25.1 (was Re: org-capture: Capture template ‘g’: Match data clobbered by buffer modification hooks) Eli Zaretskii
2016-07-21  3:00                                               ` npostavs
2016-07-21  2:43                                             ` Eli Zaretskii
2016-07-21  7:59                                             ` bug#23917: 25.0.95; commit 3a9d6296b35e5317c497674d5725eb52699bd3b8 causing org-capture to error out N. Jackson
2016-07-21  7:59                                             ` N. Jackson
2016-07-21  8:19                                               ` Robert Pluim
2016-07-08 12:42                                                 ` Robert Pluim
2016-07-08 14:02                                                   ` Eli Zaretskii
2016-07-08 15:40                                                     ` Robert Pluim
2016-07-08 17:03                                                       ` Eli Zaretskii
2016-07-15 19:46                                                         ` Eli Zaretskii
2016-07-15 19:46                                                         ` Eli Zaretskii
2016-07-08 17:03                                                       ` Eli Zaretskii
2016-07-21 14:24                                                   ` N. Jackson
2016-07-21 14:24                                                   ` N. Jackson
2016-07-21  8:19                                               ` Robert Pluim
2016-07-20 18:19                                     ` bug#23917: Please consider making Bug #23917 a blocker for 25.1 (was Re: org-capture: Capture template ‘g’: Match data clobbered by buffer modification hooks) Stefan Monnier
2016-07-20  1:50                                 ` Stefan Monnier
2016-07-19 23:18                             ` npostavs
2016-07-19 23:18                             ` npostavs
2016-07-19  2:40                       ` Eli Zaretskii
2016-07-19 15:36                       ` Eli Zaretskii
2016-07-19 15:36                       ` Eli Zaretskii
2016-07-21  7:52                     ` N. Jackson [this message]
2016-07-21  8:08                       ` bug#23917: 25.0.95; commit 3a9d6296b35e5317c497674d5725eb52699bd3b8 causing org-capture to error out Robert Pluim
2016-07-21 13:19                         ` N. Jackson
2016-07-21 13:19                         ` N. Jackson
2016-07-21  8:08                       ` Robert Pluim
2016-07-21  7:52                     ` N. Jackson
2016-07-18 14:50                 ` bug#23917: Please consider making Bug #23917 a blocker for 25.1 (was Re: org-capture: Capture template ‘g’: Match data clobbered by buffer modification hooks) Kaushal Modi
2016-07-18 14:50                 ` Kaushal Modi
     [not found]                 ` <9613abde4dbd48fdb2b5e780101a13a0@HE1PR01MB1898.eurprd01.prod.exchangelabs.com>
2016-07-18 16:59                   ` Eric S Fraga
2016-07-18 16:59                   ` Eric S Fraga

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=87h9bj2zsl.fsf_-_@moondust.awandering \
    --to=nljlistbox2@gmail.com \
    --cc=23917@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=rpluim@gmail.com \
    /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.