JDEE

Will you please send the entire file?  Patch on OSX barfs on 46.

Thanks.


On Sep 10, 2012, at 4:17 PM, shyamal@... wrote:

> 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/jdee-users@...>).
> 
> 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 ":"))
> 
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and 
> threat landscape has changed and how IT managers can respond. Discussions 
> will include endpoint security, mobile security and the latest in malware 
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> jdee-users mailing list
> jdee-users@...
> https://lists.sourceforge.net/lists/listinfo/jdee-users

>>>>> "Paul" ==   <landes@.....> writes:

    Paul> Will you please send the entire file?  Patch on OSX barfs on
    Paul> 46.

Here is the entire file (I hope the attachment makes it).

>>>>> "Shyamal" == Shyamal Prasad <shyamal@...> writes:

>>>>> "Paul" ==   <landes@.....> writes:
    Paul> Will you please send the entire file?  Patch on OSX barfs on
    Paul> 46.

    Shyamal> Here is the entire file (I hope the attachment makes it).

I'm having a bad day, lots of rookie mistakes. In the process of trying
to figure out why the patch in my original email was bad I reverted
build.el and sent the unmodified version of build.el (a unwitting tab to
space conversion and this mailing list's addition of trailing text
destroyed the original patch email).


So, again, here is the modified build.el and a patch that actually works
(patch -p1) as an attachment.

Cheers!
Shyamal