On 07/13/2012 12:41 PM, 刘智猷 wrote:
> Hi, all
> I am trying to use the development version of cedet, i.e, cedet pulled
> from http://git.randomsample.de/cedet.git. I think the following patch
> it's needed to make ede-linux (or ede/linux) working on the newest
> emacs (126.96.36.199, pulled from repo.or.cz and compiled by myself).
> I make some comments on the patch to explain why.
> $ git diff -b
> diff --git a/lisp/cedet/ede/linux.el b/lisp/cedet/ede/linux.el
> index f18852a..613b8b0 100644
> --- a/lisp/cedet/ede/linux.el
> +++ b/lisp/cedet/ede/linux.el
> @@ -33,6 +33,8 @@
> ;; * Add website
> (require 'ede)
> +(require 'ede/make) ;; without it, `ede-make-command' is undefined
> (declare-function semanticdb-file-table-object "semantic/db")
> (declare-function semanticdb-needs-refresh-p "semantic/db")
> (declare-function semanticdb-refresh-table "semantic/db")
> @@ -129,6 +131,7 @@ ROOTPROJ is nil, since there is only one project."
> (ede-add-project-to-global-list proj))
> +;; without it, the following statement doesn't eval at the right time.
> (ede-project-autoload "linux"
> :name "LINUX ROOT"
> @@ -292,7 +295,7 @@ Argument COMMAND is the command to use for
> compiling the target."
> (compile command)))
> -(provide 'ede-linux)
> +(provide 'ede/linux) ;; IMHO, nobody require ede-linux now. and
> files.el indeed requires ede/linux
> ;; Local variables:
> ;; generated-autoload-file: "loaddefs.el"
> Zhiyou Liu
Sorry for sending to a wrong mailinglist, I will resend it to cedet-devel.