GNU bug report logs -
#72701
eglot crash when project-files-relative-names t
Previous Next
Reported by: Ship Mints <shipmints <at> gmail.com>
Date: Sun, 18 Aug 2024 15:47:02 UTC
Severity: normal
Done: Dmitry Gutov <dmitry <at> gutov.dev>
Bug is archived. No further changes may be made.
Full log
Message #32 received at 72701 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Already disabled. But still glad I experimented.
On Fri, Aug 23, 2024 at 11:06 AM João Távora <joaotavora <at> gmail.com> wrote:
> On Fri, Aug 23, 2024 at 3:56 PM Ship Mints <shipmints <at> gmail.com> wrote:
> >
> > Glad I set project-files-relative-names to t and have been testing. This
> side-effect in Eglot is sure to bite other people who are teased by the
> NEWS of this new option, despite not being a defcustom.
>
> If it's not a defcustom and, as Dmitry said, not meant to be set by the
> user,
> I suggest you _don't_ set it globally, but ask for it to be let-bound
> in whatever
> features you want to make use of it.
>
> Eglot could be one of those features if there's a performance advantage.
> But I doubt it, because server-supplied glob expressions may target the
> full file name (indeed likely the truename). Maybe it's worth it
> nevertheless,
> dunno. Anyway while let-binding p-f-r-names to nil in Eglot could work, I
> don't think it's the right solution, especially since it probably triggers
> a
> compilation warning in older Emacsen which don't have this.
>
> Unrelated, there are other enhancements to Eglot's file-watching
> machinery. One of them is about excluding files from over-eager watching
> by the server which could eventually bump into OS limits for these
> OS objects.
>
> João Távora
>
[Message part 2 (text/html, inline)]
This bug report was last modified 254 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.