project-switch-project stuck in current Clojure project

[manuel-uberti]

Expected behavior

Once in a Clojure project, I am able to use C-x p p (project-switch-project) to select a different (non-Clojure) project and move there.

Actual behavior

C-x p p and the following f to find a file in another project display the files of the current Clojure project.

Steps to reproduce the problem

From emacs -Q, after having installed clojure-mode:

• (require 'clojure-mode)
• C-x p p
• Pick a project (I chose a Git-versioned .emacs.d)
• f
• Type init.el and hit RET
• C-x p p
• Pick a Clojure project (I used my own boodle)
• f
• Pick a file (I chose deps.edn)
• C-x p f
• Type a random string (I typed aassdd) and hit RET
• Hit C-g
• C-x p p
• Pick the other project (in my case, .emacs.d) and hit RET
• f
• You are now proposed to select a file from the Clojure project instead of the other one you selected when prompted by C-x p p

Environment & Version information

clojure-mode version

Probably it's because I am using straight but I get clojure-mode (version nil) when I use clojure-mode-display-version. In any case, I am on the commit 84ed16c * origin/master grafted Mention my ko-fi account.

System information

GNU Emacs 28.0.50

➤ Master branch commit:
92f342f38dd82aae4a662708dd6280fdfb2e013b
D-Bus: keep type information in D-Bus events

➤ Configured with the following options:
--with-harfbuzz
--with-json
--with-mailutils

➤ Running on:
Ubuntu 20.04 LTS
GNOME Shell 3.36.4 (x11)
GTK+ Version 3.24.18, cairo version 1.16.0

[bbatsov]

project.el is a bit light on documentation, so it's not very obvious what needs to be implemented exactly, other than project-roots. (if anything)

I can't really test this, as it seems it's something added in Emacs 27 or 28, and I'm still on 26. Not to mention I use only Projectile. 😆 Anyways, I'm assuming it won't be hard to fix.

Probably it's because I am using straight but I get clojure-mode (version nil) when I use clojure-mode-display-version. In any case, I am on the commit 84ed16c * origin/master grafted Mention my ko-fi account.

I guess you can file a separate ticket for this. I assumed this lm-version would work everywhere, but maybe that's not the case.

[manuel-uberti]

I could try my way around this, but I'd probably need an input from @dgutov to understand the inner workings of project.el.
For instance: Is project-switch-project setting the project root every time a project is selected?

@bbatsov if you feel this discussion should be taken somewhere else, feel free to close this ticket and I'll see if I can find my way through some Emacs mailing list.

[bbatsov]

It's fine to keep the ticket open, but we definitely need more data points, and I don't have much time to investigate anything these days.

[manuel-uberti]

FTR, I sent an email on emacs-devel just to avoid missing the core devs not active on GitHub: https://lists.gnu.org/archive/html/emacs-devel/2020-09/msg01590.html

[muffinmad]

The clojure-project-dir function (used in the clojure-current-project project-find function) always returns cached dir value:

C-x C-f ~/workspace/testclojure/deps.edn
M-: (project-current) => (clojure . "~/workspace/testclojure/")
M-: (project-current nil "~/workspace/gittest") => (clojure . "~/workspace/testclojure/")

After swithing to *scratch* buffer:

M-: (project-current nil "~/workspace/gittest") => (vc . "~/workspace/gittest/")

Maybe clojure-project-dir must use buffer-local cached value only if the buffer file is a child of passed dir-name.

Or maybe add the clojure-current-project to the very end of project-find-functions.

[manuel-uberti]

Thank you for looking into it. Those "maybes" are what I hope to clear out and understand. :)

[muffinmad]

The project-switch-project function just ask for directory, set default-directory to it and run one of the action functions. The action function (e.g. project-find-file) then call the project-current function to find the project in that default-directory by calling project-find-functions hooks.

Each functions on this hook is called in turn with one
argument, the directory in which to look, and should return
either nil to mean that it is not applicable, or a project instance.

I hope this answer you questions on emacs-devel ;)

[manuel-uberti]

Great, thanks. But what is the best approach here, then? You found two possible solutions, but which one is the right thing to do?

[muffinmad]

IMO the clojure-project-dir must pay attention to the directory argument before returning cached value.

[manuel-uberti]

Oh, I did open an MR with your other suggestion though. :)

[muffinmad]

Oh, I did open an MR with your other suggestion though. :)

That way clojure-project-dir will still return the wrong result.

[manuel-uberti]

Do you suggest not using clojure-cached-project-dir?