GNU bug report logs - #3597
23.0.94; [PATCH] org-store-link broken within kbd macros

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 3597 in the body.
You can then email your comments to 3597 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Andreas Seltenreich <seltenreich <at> gmx.de>
To: emacs-pretest-bug <at> gnu.org
Cc: emacs-orgmode <at> gnu.org
Subject: 23.0.94; [PATCH] org-store-link broken within kbd macros
Date: Wed, 17 Jun 2009 13:15:38 +0200

> Your bug report will be posted to the emacs-pretest-bug <at> gnu.org mailing list.

> Please describe exactly what actions triggered the bug
> and the precise symptoms of the bug:

org-store-link works fine when recording a keyboard macro but it is a
NOP when executing it.  The following change appears to fix it.

regards,
andreas

> In GNU Emacs 23.0.94.1 (x86_64-unknown-linux-gnu)
>  of 2009-05-28 on tengen

--8<---------------cut here---------------start------------->8---
*** org.el.~1.48.~	2009-02-20 10:02:49.000000000 +0100
--- org.el	2009-06-17 12:55:15.000000000 +0200
***************
*** 6594,6600 ****
  	  desc (or desc cpltxt))
      (if (equal desc "NONE") (setq desc nil))
  
!     (if (and (interactive-p) link)
  	(progn
  	  (setq org-stored-links
  		(cons (list link desc) org-stored-links))
--- 6594,6600 ----
  	  desc (or desc cpltxt))
      (if (equal desc "NONE") (setq desc nil))
  
!     (if (and (or (interactive-p) executing-kbd-macro) link)
  	(progn
  	  (setq org-stored-links
  		(cons (list link desc) org-stored-links))
--8<---------------cut here---------------end--------------->8---

Message #10 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Andreas Seltenreich <seltenreich <at> gmx.de>
To: Carsten Dominik <carsten.dominik <at> gmail.com>
Cc: 3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd macros
Date: Wed, 24 Jun 2009 14:39:10 +0200

[Sorry about setting Cc instead X-Debbugs-Cc and causing the duplicate
report.  I set a proper Mail-Followup-To header now.]

Carsten Dominik writes:

> This looks to me as if it was a bug in keyboard macro execution,
> which should call functions interactively.

The docstring for interactive-p has the following paragraph:

,----[ C-h f interactive-p RET ]
| The only known proper use of `interactive-p' is in deciding whether to
| display a helpful message, or how to display it.  If you're thinking
| of using it for any other purpose, it is quite likely that you're
| making a mistake.  Think: what do you want to do when the command is
| called from a keyboard macro?
`----

The elisp reference states:

,----[ (info "(elisp)Distinguish Interactive") ]
|  -- Function: interactive-p
|      This function returns `t' if the containing function (the one
|      whose code includes the call to `interactive-p') was called in
|      direct response to user input.  This means that it was called with
|      the function `call-interactively', and that a keyboard macro is
|      not running, and that Emacs is not running in batch mode.
`----

This sounds like interactive-p is working as documented and the
excplicit check I proposed would be neccessary.

Thanks,
andreas

Message #15 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Carsten Dominik <carsten.dominik <at> gmail.com>
To: Andreas Seltenreich <seltenreich <at> gmx.de>
Cc: 3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd macros
Date: Wed, 24 Jun 2009 14:44:23 +0200

On Jun 24, 2009, at 2:39 PM, Andreas Seltenreich wrote:

>
> [Sorry about setting Cc instead X-Debbugs-Cc and causing the duplicate
> report.  I set a proper Mail-Followup-To header now.]
>
> Carsten Dominik writes:
>
>> This looks to me as if it was a bug in keyboard macro execution,
>> which should call functions interactively.
>
> The docstring for interactive-p has the following paragraph:
>
> ,----[ C-h f interactive-p RET ]
> | The only known proper use of `interactive-p' is in deciding  
> whether to
> | display a helpful message, or how to display it.  If you're thinking
> | of using it for any other purpose, it is quite likely that you're
> | making a mistake.  Think: what do you want to do when the command is
> | called from a keyboard macro?
> `----
>
> The elisp reference states:
>
> ,----[ (info "(elisp)Distinguish Interactive") ]
> |  -- Function: interactive-p
> |      This function returns `t' if the containing function (the one
> |      whose code includes the call to `interactive-p') was called in
> |      direct response to user input.  This means that it was called  
> with
> |      the function `call-interactively', and that a keyboard macro is
> |      not running, and that Emacs is not running in batch mode.
> `----
>
> This sounds like interactive-p is working as documented and the
> excplicit check I proposed would be neccessary.

True enough, and I am applying your fix.

Thanks.

- Carsten

>
> Thanks,
> andreas

Message #22 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Carsten Dominik <carsten.dominik <at> gmail.com>
Cc: 3597 <at> debbugs.gnu.org, Andreas Seltenreich <seltenreich <at> gmx.de>,
        emacs-orgmode <at> gnu.org
Subject: Re: bug#3597: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd macros
Date: Sat, 27 Jun 2009 02:43:11 +0200

>> This sounds like interactive-p is working as documented and the
>> excplicit check I proposed would be neccessary.
> True enough, and I am applying your fix.

IIUC you may want to use called-interactively-p instead.


        Stefan

Message #27 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Carsten Dominik <carsten.dominik <at> gmail.com>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 3597 <at> debbugs.gnu.org, Andreas Seltenreich <seltenreich <at> gmx.de>,
        emacs-orgmode <at> gnu.org
Subject: Re: bug#3597: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd macros
Date: Sat, 27 Jun 2009 11:38:37 +0200

Thanks, I'll check it out.

- Carsten

On Jun 27, 2009, at 2:43 AM, Stefan Monnier wrote:

>>> This sounds like interactive-p is working as documented and the
>>> excplicit check I proposed would be neccessary.
>> True enough, and I am applying your fix.
>
> IIUC you may want to use called-interactively-p instead.
>
>
>        Stefan

Message #32 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Samuel Wales <samologist <at> gmail.com>
To: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd 
	macros
Date: Wed, 1 Jul 2009 20:39:39 -0700

I cannot use org-remember to save all my w3m tabs using remember and
keyboard macros.

Says command rang the bell, but I don't know which even with stack trace on.

  call-last-kbd-macro(nil kmacro-loop-setup-function)
  kmacro-call-macro(nil nil)
  kmacro-end-and-call-macro(nil)
  call-interactively(kmacro-end-and-call-macro)

Is this related?

Message #37 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Bastien <bastienguerry <at> googlemail.com>
To: Samuel Wales <samologist <at> gmail.com>
Cc: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd  macros
Date: Thu, 02 Jul 2009 07:52:59 +0200

Samuel Wales <samologist <at> gmail.com> writes:

> I cannot use org-remember to save all my w3m tabs using remember and
> keyboard macros.

Are you using a specific remember template when remembering those 
w3m tabs?  My guess is that your remember template tries to fetch
a title or same tab-related info that isn't available for some tab.

> Says command rang the bell, but I don't know which even with stack trace on.
>
>   call-last-kbd-macro(nil kmacro-loop-setup-function)
>   kmacro-call-macro(nil nil)
>   kmacro-end-and-call-macro(nil)
>   call-interactively(kmacro-end-and-call-macro)
>
> Is this related?

Looks like some tab prevents the macro from being repeated correctly.  

Maybe you can try to debug this like this:

1. use several identic tabs (and see if the error occurs);
2. use two different tabs, then three, etc.

HTH

-- 
 Bastien

Message #42 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Samuel Wales <samologist <at> gmail.com>
To: Bastien <bastienguerry <at> googlemail.com>
Cc: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd 
	macros
Date: Wed, 1 Jul 2009 23:05:23 -0700

Hi Bastien,

It happens on all tabs.

          ("note" ?n      "* %? %i -- %a %U\n%i")

Thanks.

On Wed, Jul 1, 2009 at 22:52, Bastien<bastienguerry <at> googlemail.com> wrote:
> Samuel Wales <samologist <at> gmail.com> writes:
>
>> I cannot use org-remember to save all my w3m tabs using remember and
>> keyboard macros.
>
> Are you using a specific remember template when remembering those
> w3m tabs?  My guess is that your remember template tries to fetch
> a title or same tab-related info that isn't available for some tab.
>
>> Says command rang the bell, but I don't know which even with stack trace on.
>>
>>   call-last-kbd-macro(nil kmacro-loop-setup-function)
>>   kmacro-call-macro(nil nil)
>>   kmacro-end-and-call-macro(nil)
>>   call-interactively(kmacro-end-and-call-macro)
>>
>> Is this related?
>
> Looks like some tab prevents the macro from being repeated correctly.
>
> Maybe you can try to debug this like this:
>
> 1. use several identic tabs (and see if the error occurs);
> 2. use two different tabs, then three, etc.
>
> HTH
>
> --
>  Bastien
>



-- 
Myalgic encephalomyelitis is causing death and severe suffering
worse than MS.  Conflicts of interest are destroying research.
People get the disease at any time permanently.  Do science and
justice matter to you?  http://www.meactionuk.org.uk/What_Is_ME_What_Is_CFS.htm

Message #47 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Bastien <bastienguerry <at> googlemail.com>
To: Samuel Wales <samologist <at> gmail.com>
Cc: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd  macros
Date: Thu, 02 Jul 2009 09:08:41 +0200

Samuel Wales <samologist <at> gmail.com> writes:

> It happens on all tabs.

You mean on *each* tab?

>           ("note" ?n      "* %? %i -- %a %U\n%i")

Can you remember at least one tab with this template?

Can you send the full macro?

-- 
 Bastien

Message #52 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Samuel Wales <samologist <at> gmail.com>
To: Bastien <bastienguerry <at> googlemail.com>
Cc: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd 
	macros
Date: Thu, 2 Jul 2009 11:02:40 -0700

On each of my many emacs-w3m tabs, I can remember manually with f8 f8
n (my binding for org-remember and my ?n template which I sent).  It
grabs the title and URL perfectly.  I can also do m-right, which is my
binding for next tab.

To use the macro, I do c-x ( f8 f8 n m-right c-x ) .  To execute it I
do c-x e.  It does not get executed on any tabs.

Thanks.



On 2009-07-02, Bastien <bastienguerry <at> googlemail.com> wrote:
> Samuel Wales <samologist <at> gmail.com> writes:
>
>> It happens on all tabs.
>
> You mean on *each* tab?
>
>>           ("note" ?n      "* %? %i -- %a %U\n%i")
>
> Can you remember at least one tab with this template?
>
> Can you send the full macro?
>
> --
>  Bastien
>


-- 
Myalgic encephalomyelitis is causing death and severe suffering
worse than MS.  Conflicts of interest are destroying research.
People get the disease at any time permanently.  Do science and
justice matter to you?  http://www.meactionuk.org.uk/What_Is_ME_What_Is_CFS.htm

Message #57 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Bastien <bastienguerry <at> googlemail.com>
To: Samuel Wales <samologist <at> gmail.com>
Cc: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd  macros
Date: Fri, 03 Jul 2009 11:18:04 +0200

Samuel Wales <samologist <at> gmail.com> writes:

> On each of my many emacs-w3m tabs, I can remember manually with f8 f8
> n (my binding for org-remember and my ?n template which I sent).  It
> grabs the title and URL perfectly.  I can also do m-right, which is my
> binding for next tab.
>
> To use the macro, I do c-x ( f8 f8 n m-right c-x ) .  To execute it I
> do c-x e.  It does not get executed on any tabs.

My other guess would be that the macro execution is too fast for the
display engine.  Maybe the trick is to insert a command in your macro
sequence that makes sure the tab is displayed correctly before you try
to remember it.  

C-x ( f8 f8 n M-> M-< c-x )
              ^^^^^^^ <- this goes to the end and beginning of buffer

Anyway, you might have clearer/cleverer answers on help-gnu-emacs@

HTH,

-- 
 Bastien

Message #62 received at 3597 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Samuel Wales <samologist <at> gmail.com>
To: Bastien <bastienguerry <at> googlemail.com>
Cc: Carsten Dominik <carsten.dominik <at> gmail.com>,
        3597 <at> debbugs.gnu.org, emacs-orgmode <at> gnu.org,
        seltenreich <at> gmx.de
Subject: Re: [Orgmode] 23.0.94; [PATCH] org-store-link broken within kbd 
	macros
Date: Sat, 25 Jul 2009 17:44:12 -0700

Hi Bastien,

Thanks for the idea.

Doesn't work.

Maybe it's an emacs bug.

On Fri, Jul 3, 2009 at 02:18, Bastien<bastienguerry <at> googlemail.com> wrote:
> My other guess would be that the macro execution is too fast for the
> display engine.  Maybe the trick is to insert a command in your macro
> sequence that makes sure the tab is displayed correctly before you try
> to remember it.
>
> C-x ( f8 f8 n M-> M-< c-x )
>              ^^^^^^^ <- this goes to the end and beginning of buffer
>
> Anyway, you might have clearer/cleverer answers on help-gnu-emacs@
>
> HTH,
>
> --
>  Bastien
>

Message #69 received at 3597-done <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 3597-done <at> debbugs.gnu.org
Subject: Re: 23.0.94; [PATCH] org-store-link broken within kbd macros
Date: Thu, 03 Mar 2011 16:15:42 -0500

Version: 23.2

Looks like this patch was in 23.2.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.