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