|
From: | Ergus |
Subject: | Re: Should EDE use project.el [PATCH] |
Date: | Mon, 16 May 2022 18:09:38 +0200 |
On Mon, May 16, 2022 at 06:50:59PM +0300, Dmitry Gutov wrote:
On 16.05.2022 18:00, Ergus wrote:Hi: I just attached an extremely simple (intested) attempt to make cedet use Project.el.. Maybe Dmitry or someone with more experience may complete this and extend it when possible?Looks okay to me.To give more detailed feedback, one might need more knowledge of CEDET, I think
Do you use it? CEDET, I mean. If yes, this can be a beneficial direction to go in. My observations say it's only falling out of favor, though.
Actually I don't know how to use it properly yet. I just tried this because I tried something with gtags and I found that it has its own backend for global/gtags BUT un-optimized and tramp and similes fail (as expected). The last commit in their git repo was in 2019 and it seems to be almost totally unmaintained (in a quick test I found some issues immediately)... which means I am not even sure if there is anyone still using it. So my first recommendation would be to remove/deprecate it in favor of project.el; but as I know that such a suggestion is impossible, I just recommend to make it a wrapper for project.el by default when possible. The only feature missing in project.el to totally overlap EDE is the concept of update; which cedet seems to use but project does not understand (probably because in the new backends may not be needed..) Best, Ergus
Best, Ergus On Mon, May 16, 2022 at 01:46:43PM +0200, Ergus wrote:Hi: Recently working in the gtags-mode package I thought to add a backend for ede/cedet... but I have found that most of the ede features overlap or coincide with ones in project.el. So my question is: Isn't it easier to add a backend in locate.el to make ede use project as one ede-locate-setup-options? I am not sure about how useful is ede or cedet these days as they seem to be a bit unmaintained, but IMHO, if we don't want to remove them (for some reason) but still keep project.el as the preferred way to go, then maybe cedet should use the project.el functions by default. The basic implementation seems to need only: initialize-instance ede-locate-ok-in-project ede-locate-file-in-project-impl ede-locate-create/update-root-database Implementing all these may be extremely simple but not sure if it worth the effort. What do you suggest? Best, Ergus
[Prev in Thread] | Current Thread | [Next in Thread] |