all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
       [not found] ` <20240322014425.180EEC12C52@vcs2.savannah.gnu.org>
@ 2024-03-30 11:19   ` Arash Esbati
  2024-03-30 13:05     ` Eli Zaretskii
  0 siblings, 1 reply; 6+ messages in thread
From: Arash Esbati @ 2024-03-30 11:19 UTC (permalink / raw)
  To: emacs-devel; +Cc: Stefan Monnier

Stefan Monnier via Mailing list for Emacs changes <emacs-diffs@gnu.org> writes:

> branch: master
> commit 60c9702972f3cef9e6dbbce5eaad8cc90ea7f8e8
> Author: Stefan Monnier <monnier@iro.umontreal.ca>
> Commit: Stefan Monnier <monnier@iro.umontreal.ca>
>
>     * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
> ---
>  lisp/help.el | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lisp/help.el b/lisp/help.el
> index 4171d0c57c7..bafe6032942 100644
> --- a/lisp/help.el
> +++ b/lisp/help.el
> @@ -926,7 +926,9 @@ in the selected window."
>       (let ((key-desc (help-key-description key untranslated)))
>         (if (help--binding-undefined-p defn)
>             (format "%s%s is undefined" key-desc mouse-msg)
> -         (format "%s%s runs the command %S" key-desc mouse-msg defn)))
> +         (format "%s%s runs the command %s" key-desc mouse-msg
> +                 (if (symbolp defn) (prin1-to-string defn)
> +                   (help-fns-function-name defn)))))
>       defn event mouse-msg)))
>  
>  (defun help--filter-info-list (info-list i)

I think this change now produces:

  In end of data:
  help.el:931:21: Warning: the function ‘help-fns-function-name’ is not
  known to be defined.

This is with Emacs 30 (87be53846b) on macOS.

Best, Arash



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
  2024-03-30 11:19   ` master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name` Arash Esbati
@ 2024-03-30 13:05     ` Eli Zaretskii
  2024-03-30 15:45       ` Arash Esbati
  0 siblings, 1 reply; 6+ messages in thread
From: Eli Zaretskii @ 2024-03-30 13:05 UTC (permalink / raw)
  To: Arash Esbati; +Cc: emacs-devel, monnier

> From: Arash Esbati <arash@gnu.org>
> Cc: Stefan Monnier <monnier@iro.umontreal.ca>
> Date: Sat, 30 Mar 2024 12:19:23 +0100
> 
> I think this change now produces:
> 
>   In end of data:
>   help.el:931:21: Warning: the function ‘help-fns-function-name’ is not
>   known to be defined.

This is not a real problem: after you build once, touch lisp.help.el
and build again: the warning will be gone.  That's because the
autoload form for the function will be written to loaddefs.el.




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
  2024-03-30 13:05     ` Eli Zaretskii
@ 2024-03-30 15:45       ` Arash Esbati
  2024-03-30 16:40         ` Eli Zaretskii
  2024-03-31  2:55         ` Stefan Monnier
  0 siblings, 2 replies; 6+ messages in thread
From: Arash Esbati @ 2024-03-30 15:45 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel, monnier

Eli Zaretskii <eliz@gnu.org> writes:

> This is not a real problem: after you build once, touch lisp.help.el
> and build again: the warning will be gone.  That's because the
> autoload form for the function will be written to loaddefs.el.

Thanks, does this also apply to a clean state repo?  I use a small
script to build Emacs which does:

  git clean -fdx --exclude=ChangeLog
  ./autogen.sh
  ./configure ...

My lisp/loaddefs.el already has an entry:

  $ grep help-fns-function loaddefs.el
  (autoload 'help-fns-function-name "help-fns" "\

I didn't change anything after my initial build where I reported the
warning.

Best, Arash



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
  2024-03-30 15:45       ` Arash Esbati
@ 2024-03-30 16:40         ` Eli Zaretskii
  2024-03-31  2:55         ` Stefan Monnier
  1 sibling, 0 replies; 6+ messages in thread
From: Eli Zaretskii @ 2024-03-30 16:40 UTC (permalink / raw)
  To: Arash Esbati; +Cc: emacs-devel, monnier

> From: Arash Esbati <arash@gnu.org>
> Cc: emacs-devel@gnu.org,  monnier@iro.umontreal.ca
> Date: Sat, 30 Mar 2024 16:45:02 +0100
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > This is not a real problem: after you build once, touch lisp.help.el
> > and build again: the warning will be gone.  That's because the
> > autoload form for the function will be written to loaddefs.el.
> 
> Thanks, does this also apply to a clean state repo?  I use a small
> script to build Emacs which does:
> 
>   git clean -fdx --exclude=ChangeLog
>   ./autogen.sh
>   ./configure ...
> 
> My lisp/loaddefs.el already has an entry:
> 
>   $ grep help-fns-function loaddefs.el
>   (autoload 'help-fns-function-name "help-fns" "\
> 
> I didn't change anything after my initial build where I reported the
> warning.

No clue, sorry.  I got the warning upon the first build after update
from upstream, then the next build didn't have the warning.



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
  2024-03-30 15:45       ` Arash Esbati
  2024-03-30 16:40         ` Eli Zaretskii
@ 2024-03-31  2:55         ` Stefan Monnier
  2024-03-31 14:28           ` Arash Esbati
  1 sibling, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2024-03-31  2:55 UTC (permalink / raw)
  To: Arash Esbati; +Cc: Eli Zaretskii, emacs-devel

>> This is not a real problem: after you build once, touch lisp.help.el
>> and build again: the warning will be gone.  That's because the
>> autoload form for the function will be written to loaddefs.el.
> Thanks, does this also apply to a clean state repo?

What Eli is saying is that the warning will disappear the next time
`ldefs-boot.el` is updated with a more current copy of `loaddefs.el`.


        Stefan




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name`
  2024-03-31  2:55         ` Stefan Monnier
@ 2024-03-31 14:28           ` Arash Esbati
  0 siblings, 0 replies; 6+ messages in thread
From: Arash Esbati @ 2024-03-31 14:28 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Eli Zaretskii, emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> What Eli is saying is that the warning will disappear the next time
> `ldefs-boot.el` is updated with a more current copy of `loaddefs.el`.

Thanks Stefan, so I'll wait for that.

Best, Arash



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-03-31 14:28 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <171107186434.18812.2389706590800223024@vcs2.savannah.gnu.org>
     [not found] ` <20240322014425.180EEC12C52@vcs2.savannah.gnu.org>
2024-03-30 11:19   ` master 60c9702972f: * lisp/help.el (help--analyze-key): Use `help-fns-function-name` Arash Esbati
2024-03-30 13:05     ` Eli Zaretskii
2024-03-30 15:45       ` Arash Esbati
2024-03-30 16:40         ` Eli Zaretskii
2024-03-31  2:55         ` Stefan Monnier
2024-03-31 14:28           ` Arash Esbati

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.