[jdee-users] Bad compile load path for GNU emacs >= 23.3?

[<sh...@me...>]

Hi,

I depend on JDEE and when it went away as a Debian package I found
myself building my own from the subversion trunk. 

One thing I seem to have learned: compiling trunk code does NOT work out
of the box. The reason is the build.el places the downloaded CEDET at
the end of the load path, so the emacs included CEDET always goes
before it! This leads to incomplete compilation and associated non
expansion of macros (e.g. see http://www.mail-archive.com/jde...@li.../msg00600.html).

The patch below seemingly allows me to reliably build and use JDEE on
Debian Squeeze (JDK 1.6.0_26, GNU Emacs 23.4.1, CEDET 1.0 downloaded
locally).

I suspect we could just set the add-to-list append parameter to nil
across all versions, but I don't have all the history here about why it
goes to the end. Feedback would be much appreciated. This was the most
defensive change I could think of.

Sorry for the cross post; these lists seem so quiet I'm not sure who is
reading which list, but I certainly would not mind a (gentle) rap on my
fingers.

Cheers!
Shyamal

diff --git a/jdee/config/build.el b/jdee/config/build.el
index 0365e42..ceebb98 100644
--- a/jdee/config/build.el
+++ b/jdee/config/build.el
@@ -46,7 +46,10 @@ AUTOLOAD-LIBNAME the name of the generated autoload file."
        (add-to-list 'load-path elib-dir t))
     (when (and cedet-paths (> (length cedet-dir) 0))
       (dolist (path cedet-paths)
-       (add-to-list 'load-path (expand-file-name path cedet-dir) t)))
+       (add-to-list 'load-path (expand-file-name path cedet-dir)
+                     ;; emacs 23.2 added CEDET
+                     (and (< emacs-major-version 24)
+                          (< emacs-minor-version 2)))))
     (add-to-list 'load-path lisp-src-dir t)
     (eval-buffer autoload-buf)
     (message "load path: %s" (mapconcat #'identity load-path ":"))