Re: [O] Problem at capture time
Bastien writes: > David Maus writes: >> I just pushed a fix for this problem > Thanks a lot for the fix. After removing my patch and installing the official code here, things continue to work nicely for the few tests I did. Let me thank you as well! François
Re: [O] Problem at capture time
Hi David, David Maus writes: > Sorry for my silence, haven't looked into the Orgmode mailing list for > a while. I just pushed a fix for this problem: If the :exact-position > for a list item is supplied we don't search for the position in an > existing list at all but insert the item whereever :exact-position > points to. Thanks a lot for the fix. > I'll mark the patch in patchwork as superseeded this evening as soon > as I recover my password. I just did so -- hope you'll recover your password! Otherwise John might help. Best, -- Bastien
Re: [O] Problem at capture time
Hi François, Hi Bastien, At Mon, 09 Jan 2012 07:50:54 -0500, François Pinard wrote: > > Bastien writes: > > > Hi François, please be patient -- your patches are under radar, > > resending them does not help. > > OK, sorry. I do not know enough, yet, how Org works. I sent a problem, > got a question, sent a reply which was rejected, then nothing. A mere > short acknowledgement would have told me the message was not lost. > > > What helps is to send them using git format-patch, to include a > > detailed and clean changelog entry, and to make it clear whether the > > patch is okay or for testing purpose first. When I read your message, > > I could not be sure you were yourself confident it was the right fix. > > I'm still not confident. And I'm not pedant enough -- yet! :-) -- to > write formal patches when I'm not at least rather sure. > > Besides the lack of confidence in a solution, however, the problem is > real. The patch I sent was merely a trail for thinking. But if nothing > else, it would do I presume. > > > I will look into this later on. > > Let me the one thanking you for supporting Org mode and its users, even > those like me! :-) Sorry for my silence, haven't looked into the Orgmode mailing list for a while. I just pushed a fix for this problem: If the :exact-position for a list item is supplied we don't search for the position in an existing list at all but insert the item whereever :exact-position points to. I'll mark the patch in patchwork as superseeded this evening as soon as I recover my password. Best, -- David -- OpenPGP... 0x99ADB83B5A4478E6 Jabber dmj...@jabber.org Email. dm...@ictsoc.de pgpXeMllWqvRS.pgp Description: PGP signature
Re: [O] Problem at capture time
Bastien writes: > Hi François, please be patient -- your patches are under radar, > resending them does not help. OK, sorry. I do not know enough, yet, how Org works. I sent a problem, got a question, sent a reply which was rejected, then nothing. A mere short acknowledgement would have told me the message was not lost. > What helps is to send them using git format-patch, to include a > detailed and clean changelog entry, and to make it clear whether the > patch is okay or for testing purpose first. When I read your message, > I could not be sure you were yourself confident it was the right fix. I'm still not confident. And I'm not pedant enough -- yet! :-) -- to write formal patches when I'm not at least rather sure. Besides the lack of confidence in a solution, however, the problem is real. The patch I sent was merely a trail for thinking. But if nothing else, it would do I presume. > I will look into this later on. Let me the one thanking you for supporting Org mode and its users, even those like me! :-) François
Re: [O] Problem at capture time
Hi François, please be patient -- your patches are under radar, resending them does not help. What helps is to send them using git format-patch, to include a detailed and clean changelog entry, and to make it clear whether the patch is okay or for testing purpose first. When I read your message, I could not be sure you were yourself confident it was the right fix. I will look into this later on. Thanks, -- Bastien
Re: [O] Problem at capture time
pin...@iro.umontreal.ca (François Pinard) writes: > Hi, Org people. [...] The problem, which I described two months ago, Hmph! My math is clearly wrong! One month ago! :-) Sorry. François P.S. Who never had the the memory of times...
[O] Problem at capture time
Hi, Org people. On 2011-12-11 23:24, I reported a problem met while using the Org capture facility, and a patch working around the problem. David Maus then requested for details, which I sent on 2011-12-19 00:59. David's mailer rejected my reply as being likely SPAM, but it made it to the emacs-orgmode@gnu.org mailing list, so I presume David got it. Here are references I have for both messages: - [[gnus:gmane.emacs.orgmode#8739cqo1s3@iro.umontreal.ca][Email from François Pinard: Capture failure {7.7}]] - [[gnus:gmane.emacs.orgmode#871us1f6fo@iro.umontreal.ca][Email from François Pinard: Re: Capture failure {7.7}]] This morning, I removed my patch locally. Using a fresh Git pull of Org mode, the problem is still there, showing as: Debugger entered--Lisp error: (error "Capture template `d': integer-or-marker-p") signal(error ("Capture template `d': integer-or-marker-p")) error("Capture template `%s': %s" "d" integer-or-marker-p) byte-code("\301p!\203. \302\303\304 \"\203. \305p!\210\306\307\310!!\210\300\311\307\312!A@#\207" [error buffer-base-buffer string-match "\\`CAPTURE-" buffer-name kill-buffer set-window-configuration org-capture-get :return-to-wconf "Capture template `%s': %s" :key] 4) org-capture(nil) call-interactively(org-capture nil nil) The problem, which I described two months ago, is that some variables in org-capture.el (org-capture-place-item) do not get properly initialized depending on the flow within the function. The following patch solves the problem for me: diff --git a/lisp/org-capture.el b/lisp/org-capture.el index 46202f8..508611d 100644 --- a/lisp/org-capture.el +++ b/lisp/org-capture.el @@ -913,7 +913,8 @@ it. When it is a variable, retrieve the value. Return whatever we get." beg end) (cond ((org-capture-get :exact-position) - (goto-char (org-capture-get :exact-position))) + (goto-char (org-capture-get :exact-position)) + (setq beg (point) end (point))) ((not target-entry-p) ;; Insert as top-level entry, either at beginning or at end of file (setq beg (point-min) end (point-max))) Could it be integrated in the main stream? I'm not sure that "(point)" is the best value to give for "beg" and "end" (I do not understand the code enough), but surely, this is better than capture just crashing. François