[CEDET-devel] Patch for ede/linux

[刘智猷 <liu...@gm...>]

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 (24.1.50.1, 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.
+;;;###autoload
  (ede-add-project-autoload
   (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"

BTW. This is a resending.


Regards,
Zhiyou Liu