Re: Should EDE use project.el [PATCH]

[Ergus <spacibba@aol.com>]

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.
>

I only get messages like this all the time:

Debug on Error enabled globally
<menu-bar> <cedet-menu> <ede-project-options> <Default configuration>-  which-key: There are no keys to show [2 times]
<menu-bar> <cedet-menu> <ede-project-options> <Default configuration> <menu-bar> is undefined
<cedet-menu> is undefined
<ede-new> is undefined
Remote file error: Forbidden reentrant call of Tramp [3 times]
<menu-bar> <cedet-menu> <ede-project-options> <Default configuration>-  which-key: There are no keys to show [2 times]
<menu-bar> <cedet-menu> <ede-project-options> <Default configuration> <menu-bar> is undefined
<cedet-menu> is undefined
<ede-new> is undefined
Remote file error: Forbidden reentrant call of Tramp [3 times]
<menu-bar> <cedet-menu> <ede-project-options> <Default configuration>-  which-key: There are no keys to show [2 times]
<menu-bar> <cedet-menu> <ede-project-options> <Default configuration> <menu-bar> is undefined
<cedet-menu> is undefined
<ede-project-options> is undefined
<Default configuration> is undefined

So I am 90% sure nobody is trying cedet with tramp.

>>
>>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
>>>
>
>