GNU bug report logs -
#63870
29.0.90; project.el can't dynamically populate the project list
Previous Next
Full log
View this message in rfc822 format
> From: Spencer Baugh <sbaugh <at> janestreet.com>
> Cc: 63870 <at> debbugs.gnu.org
> Date: Wed, 28 Jun 2023 08:37:58 -0400
>
> Eli Zaretskii <eliz <at> gnu.org> writes:
>
> >> Absolutely. I am trying to be careful about this: project-watch
> >> shouldn't create watches on VCS directories.
> >
> > But below you explicitly give an example where it will. And given the
> > fact that the majority of project.el projects use VCS as its backend,
> > I'd say we are already there...
>
> No: the watch on a directory is removed once the directory becomes a VCS
> directory.
No, AFAIU the watch is removed once there is a project in the
directory. But a VCS can work on a directory regardles of any project
creation.
> > Once again, this is dangerous; users could easily shoot themselves in
> > the foot, because not many are aware of the pitfall of using file
> > notifications for many directories. It makes no sense to warn against
> > something and at the same time let callers easily stumble upon that.
>
> I agree with that, I suppose. Personally I would be fine with a
> mandatory 1 or 2 levels of recursion, since I only need 2. Do you have
> a suggestion for what that interface could look like? It feels a bit
> awkward...
I'd actually begin by not providing even 1 level. Let the callers
call this new function explicitly for every directory which they want
watching. If someone ever complains that this is somehow inconvenient
(although I don't see why: directory-files is simple to use), then we
could consider extending the API.
But that's MO; please wait for Dmitry to chime in.
This bug report was last modified 1 year and 325 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.