GNU bug report logs - #62155
30.0.50; elixir-ts-mode.elc warning

Previous Next

Package: emacs;

Reported by: Po Lu <luangruo <at> yahoo.com>

Date: Mon, 13 Mar 2023 02:08:01 UTC

Severity: normal

Found in version 30.0.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #20 received at 62155 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Wilhelm Kirschbaum <wkirschbaum <at> gmail.com>
Cc: ruijie <at> netyu.xyz, luangruo <at> yahoo.com, 62155 <at> debbugs.gnu.org
Subject: Re: bug#62155: 30.0.50; elixir-ts-mode.elc warning
Date: Mon, 13 Mar 2023 16:33:18 +0200
> Cc: luangruo <at> yahoo.com, 62155 <at> debbugs.gnu.org
> From: Wilhelm Kirschbaum <wkirschbaum <at> gmail.com>
> Date: Mon, 13 Mar 2023 15:08:10 +0200
> 
> I can see there is still an issue with loading heex-ts-mode from 
> elixir-ts-mode, but not quite sure how to
> immediately resolve it. I will have a look in a couple of hours.

One possibility is to add defvar for each variable defined by
heex-ts-mode that elixir-ts-mode needs to use, and remove the require
of heex-ts-mode.  Since heex-ts-mode is loaded at run time, I think
this should be enough.

> The problem is in the top of elixir-ts-mode.el: (require 
> 'heex-ts-mode) and then we call
> (treesit-ready-p 'heex) in heex-ts-mode.

Yes.

> I guess the intention with the convention here was to only load the
> -ts-mode when the user explicitly requires, but it breaks the way I
> called (require 'heex-ts-mode) from elixir-ts-mode.el.

Yes.




This bug report was last modified 2 years and 70 days ago.

Previous Next


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