unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Dmitry Gutov <dgutov@yandex.ru>
To: Juri Linkov <juri@linkov.net>
Cc: "Kévin Le Gouguec" <kevin.legouguec@gmail.com>,
	"Eli Zaretskii" <eliz@gnu.org>,
	"Stephen Leake" <stephen_leake@stephe-leake.org>,
	emacs-devel@gnu.org, joaotavora@gmail.com
Subject: Re: Add cl-defgeneric project-name; first use case eglot
Date: Wed, 23 Nov 2022 15:46:50 +0200	[thread overview]
Message-ID: <bb083bad-7a26-655c-53b9-18e571004ec3@yandex.ru> (raw)
In-Reply-To: <86sfiarsmw.fsf@mail.linkov.net>

On 23/11/22 10:33, Juri Linkov wrote:
>>> (1) C-x p p emacs TAB is currently rather crowded, because I stuff a lot
>>> of things under ~/src/emacs: emacs.git worktrees, elpa.git, upstream
>>> repos for *ELPA packages…
>>> If I could "name" projects such that only emacs.git worktrees included
>>> the string "emacs" (rather than all repos under ~/src/emacs), that'd
>>> make completion more efficient.
>> You're welcome to experiment with project-prompt-project-dir's code. But
>> note that until now that function didn't require to "materialize" project
>> instances for every entry, it just works off saved directory names.
>>
>> The feature you have in mind seems to require fetching a project instance
>> for every dir and calling 'project-name' on it. The apparent #1 gotcha
>> would be with remote filesystems where connection is slow/impossible, but
>> it might be possible to skip those when computing the names.
> It's possible to add projects names to project--list that it saved in
> ~/.emacs.d/projects, e.g. '(("~/project/dir/" (name . "Project name")) ...)

I'm not sure it's a good idea because

a) That breakes the idea that every project type decides things (e.g. 
Projectile has projectile-project-name[-function]).

b) This file is transient. It should be allowed to clear off the 
projects that haven't been used for a while (or just all but 10-20 most 
recently used ones), and if the file contains important info, we won't 
be allowed to do that automatically.



  reply	other threads:[~2022-11-23 13:46 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-20 22:09 Add cl-defgeneric project-name; first use case eglot Stephen Leake
2022-11-20 22:14 ` [SPAM UNSURE] " Stephen Leake
2022-11-21  0:37 ` Stephen Leake
2022-11-21 13:07 ` Eli Zaretskii
2022-11-21 13:50   ` João Távora
2022-11-21 14:17     ` Eli Zaretskii
2022-11-21 21:29       ` João Távora
2022-11-22  9:56   ` Kévin Le Gouguec
2022-11-23  2:34     ` Dmitry Gutov
2022-11-23  8:33       ` Juri Linkov
2022-11-23 13:46         ` Dmitry Gutov [this message]
2022-11-27 18:47       ` Kévin Le Gouguec
2022-11-28  1:21         ` Dmitry Gutov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bb083bad-7a26-655c-53b9-18e571004ec3@yandex.ru \
    --to=dgutov@yandex.ru \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=joaotavora@gmail.com \
    --cc=juri@linkov.net \
    --cc=kevin.legouguec@gmail.com \
    --cc=stephen_leake@stephe-leake.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).