unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Eric M. Ludlam" <eric@siege-engine.com>
To: emacs-devel@gnu.org, David Engster <deng@randomsample.de>
Subject: Re: A unified project root interface
Date: Fri, 15 Mar 2013 20:47:25 -0400	[thread overview]
Message-ID: <5143C11D.8070705@siege-engine.com> (raw)
In-Reply-To: <87vc8vyy66.fsf@engster.org>

On 03/13/2013 02:03 PM, David Engster wrote:
> Eric M. Ludlam writes:
>> On 03/11/2013 02:57 PM, Lluís wrote:
>>> CEDET provides this in two ways (as part of the EDE subsystem):
>>>
>>> * If a Project.ede file exists, that's the root (similar to .dir-locals.el in
>>>     this context)
>>>
>>> * If signs of a "project-like" structure exist (e.g., (auto)makefiles, scons,
>>>     java, etc), uses system-specific knowledge to automatically
>>> detect the project
>>>     root.
>>> I'm commenting this because, first, managing projects is the purpose of EDE
>>> (although it tries to do more than just identifying their root) and it's
>>> integrated in Emacs; and second, because the auto-detection could help in making
>>> the process simpler and, in the best case, auto-magical.
>>
>> I, of course, agree with Lluis.  EDE is already setup to automatically
>> find projects as was requested.  Adding new projects through the
>> generic' system is a pretty simple prospect.
>
> Yes, but we cannot envision all the kinds of projects people would want
> and create those for them beforehand. And defining own projects with
> ede-generic is simple, but not simple enough for end users. Let's take
> the example for defining those simple projects in project-roots.el,
> given by Sudish Joseph in this thread:
>
> 	("Perl Project"
>           :root-contains-files ("t" "lib")
>           :on-hit (lambda (p) (message (car p))))
>
> You can create a project like this with ede-generic, but then you have
> to write a little defclass inheriting from ede-generic, call
> ede-generic-new-autoloader, and to actually *do* something when the
> project is loaded you have to define methods like
> `ede-generic-setup-configuration'. This is just too much boilerplate for
> such a simple thing like the above. Also, even most Emacs developers are
> not familiar with the CLOS-like syntax that's needed to define those
> things.
>
> There's no doubt that EDE can do all what's needed, but is has to be
> wrapped in something that's easier to use, at least for simple stuff
> like what project-roots.el does.

Ok, I can buy that.  EDE was never simple.  I just know that if an 
"official" project mechanism is built, I'll have some work to do. ;)

In particular, if a system for detecting projects is developed, it would 
be great if EDE could stop detecting projects (ie - I delete some code), 
and instead use a hook to say "hey, let me know if you found a project", 
after which it can do its bit of magic for projects it also knows about.

Also be aware that I ran into a lot of performance issues I had to work 
through with EDE project detection.  Once other code starts asking where 
it is in a project, it's amazing how often that stuff gets called.  A 
project cache is a super handy thing, and was one of the key performance 
improvements I made back when i was trying to speed up smart completion.

Eric



  parent reply	other threads:[~2013-03-16  0:47 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-09 16:44 A unified project root interface Jorgen Schaefer
2013-03-09 17:12 ` Fabian Ezequiel Gallina
2013-03-10  5:38 ` Stefan Monnier
2013-03-10 10:06   ` Jorgen Schaefer
2013-03-11 18:57   ` Lluís
2013-03-12 23:28     ` Eric M. Ludlam
2013-03-12 23:42       ` Jorgen Schaefer
2013-03-13  2:02         ` Eric M. Ludlam
2013-03-13 18:03       ` David Engster
2013-03-13 19:11         ` Sudish Joseph
2013-03-16  0:47         ` Eric M. Ludlam [this message]
2013-03-16 14:18           ` David Engster
2013-03-16 15:02             ` Jorgen Schaefer
2013-03-16 22:27               ` Phil Hagelberg
2013-03-16 22:59               ` David Engster
2013-03-16 23:16                 ` Jorgen Schaefer
2013-03-17 17:40                   ` David Engster
2013-03-17 18:18                     ` Jorgen Schaefer
2013-03-18 22:50                       ` David Engster
2013-03-19  1:57                         ` John Yates
2013-03-19  7:18                           ` David Engster
2013-03-19 12:23                             ` Eric M. Ludlam
2013-03-19 13:06                             ` Stefan Monnier
2013-03-19 19:09                               ` David Engster
2013-03-20  3:21                                 ` Stefan Monnier
2013-03-20  4:48                                   ` Leo Liu
2013-03-20  7:04                                     ` joakim
2013-03-20  7:05                                     ` David Engster
2013-03-20  7:13                                   ` David Engster
2013-03-20 12:57                                     ` Stefan Monnier
2013-03-20 16:14                                       ` Davis Herring
2013-03-20 17:41                                         ` Stefan Monnier
2013-03-20 17:48                                         ` Stefan Monnier
2013-03-20 18:20                                         ` Bruce Korb
2013-03-20 22:14                                           ` Stefan Monnier
2013-03-20 16:34                                       ` David Engster
2013-03-20 17:47                                         ` Stefan Monnier
2013-03-21  0:55                                           ` Eric M. Ludlam
2013-03-21  3:27                                             ` Stefan Monnier
2013-03-21  4:07                                               ` Eric M. Ludlam
2013-03-21 14:33                                                 ` Stefan Monnier
2013-03-22  2:12                                                 ` Eric M. Ludlam
2013-03-23 11:04                                                   ` EIEIO split (was: A unified project root interface) David Engster
2013-03-21 16:32                                             ` A unified project root interface David Engster
2013-03-22  0:47                                               ` Eric M. Ludlam
2013-03-22 20:30                                                 ` David Engster
2013-03-23 17:10                                                   ` Eric M. Ludlam
2013-03-23 17:26                                                     ` Jorgen Schaefer
2013-03-23 18:02                                                       ` Dmitry Gutov
2013-03-23 20:51                                                         ` Pascal J. Bourguignon
2013-03-24  4:25                                                           ` Dmitry Gutov
2013-03-24 10:13                                                             ` Jorgen Schaefer
2013-04-06 13:25                                                               ` Jorgen Schaefer
2013-04-06 17:13                                                                 ` Eric M. Ludlam
2013-04-08 19:03                                                                   ` David Engster
2013-12-31 20:12                                                               ` Daniel Colascione
2013-03-20 17:49                                         ` Jorgen Schaefer
2013-03-19  7:33                         ` Jorgen Schaefer
2013-03-17  8:08                 ` joakim
2013-03-12 15:34   ` Sudish Joseph
2013-03-12 16:51     ` Dmitry Gutov
2013-03-12 18:23   ` Ted Zlatanov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5143C11D.8070705@siege-engine.com \
    --to=eric@siege-engine.com \
    --cc=deng@randomsample.de \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).