Re: Should EDE use project.el [PATCH]

[Ergus]

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