all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: David Engster <deng@randomsample.de>
To: Chong Yidong <cyd@stupidchicken.com>
Cc: emacs-devel@gnu.org
Subject: Re: CEDET branch
Date: Fri, 11 Sep 2009 12:41:18 +0200	[thread overview]
Message-ID: <kzy6olbw6p.fsf@engster.org> (raw)
In-Reply-To: <8763bquvfh.fsf@stupidchicken.com> (Chong Yidong's message of "Thu, 10 Sep 2009 21:22:42 -0400")

Chong Yidong <cyd@stupidchicken.com> writes:
> I wonder, though, whether the merged version of CEDET should provide a
> different method for activating CEDET.  The out-of-tree version requires
> you to put something like
>
>     (setq semantic-load-turn-useful-things-on t)
>     (require 'cedet)
>
> in your init file, but a package that is included with Emacs should not
> have to be loaded this way.  Rather, there should be a command (or set
> of commands) that are autoloaded (in the global Emacs loaddefs.el), and
> calling the command should load the package.

The problem was always that CEDET provides a bunch of different
features. If you activate them all, people would complain that it is
'too much', and if you just activate the bare minimum, people would
wonder why certain stuff just doesn't work.

That's why in upstream there's semantic-load.el, which has several
functions to set up CEDET in various different ways, from the bare
minimum to absolutely everything. I guess we would have to find a
compromise for which features should be activated by default. Eric has
surely some ideas regarding this issue.

>> * I guess most of the search macros in semantic/find.el will need
>>   autoloads, most notably the ones under "Top level searches" and
>>   probably also the ones under "Deep searches". At the moment, I get an
>>   error that `semantic-find-tags-by-class' isn't defined.
>
> The semantic-find-tags-by-class macro should be byte-compiled in-place
> by any file that requires it.  Can you provide a recipe?

I guess a

(eval-when-compile (require 'semantic/find))

in semantic/analyze/fcn.el should do the trick?

I also noticed another error due to removed autoloads:
'semantic-tag-write-list-slot-value' isn't defined when the database is
saved upon exiting Emacs. An eval-when-compile won't work here, since
this is actually a slot value which is called from eieio. So I guess it
either needs an autoload, or 'semantic/tag-write' is required in
semantic/db-file.el ?

Regards,
David








  parent reply	other threads:[~2009-09-11 10:41 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-09  2:23 CEDET branch Chong Yidong
2009-09-09 16:19 ` David Engster
2009-09-11  1:22   ` Chong Yidong
2009-09-11  2:56     ` Stefan Monnier
2009-09-11 12:21       ` Jason Rumney
2009-09-11 10:41     ` David Engster [this message]
2009-09-13 16:19       ` Chong Yidong
2009-09-12 12:15     ` Eric M. Ludlam
2009-09-12 18:56       ` David Engster
2009-09-12 20:20         ` Eric M. Ludlam

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

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

  git send-email \
    --in-reply-to=kzy6olbw6p.fsf@engster.org \
    --to=deng@randomsample.de \
    --cc=cyd@stupidchicken.com \
    --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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.