Re: [cedet-semantic] Still trying to build for MSWindows & emacs 24

[Vincent B. <vin...@ho...>]

Thank you for providing this clue.


Based on your suggestion I will try and propose a patch to jdee, so that CEDET/Emacs version compatibility are better handled.


VBR,
    Vincent.



> Date: Mon, 8 Apr 2013 20:57:41 -0400
> From: er...@si...
> To: vin...@ho...
> CC: jde...@li...; ced...@li...
> Subject: Re: [cedet-semantic] Still trying to build for MSWindows & emacs 24
> 
> Hi Vincent,
> 
> This probably doesn't help, but I think the problem you found below with 
> jde-bug-minor-mode is that in newer Emacs, :keymap is an option to 
> define-minor-mode, and was thus removed from the arglist of 
> semantic-add-minor-mode by the Emacs maintainers (presumably.)  My log 
> just says "imported from Emacs".
> 
> You can probably make it work in a range of versions of both Emacs and 
> CEDET by setting the keymap by hand.  To be more forward looking, just 
> use the :keymap part of define-minor-mode.
> 
> Eric
> 
> 
> On 04/07/2013 03:57 PM, Vincent Belaïche wrote:
> >> Hello,
> >>
> >> Sorry for the delay in feedback. I am still on that, I found an issue
> >> with build.el, require expects a symbol argument, not a string. Herein
> >> attached is a patch proposal.
> >>
> >> VBR,
> >>    Vincent
> >>
> >
> > Looping also through CEDET list.
> >
> > One more thing for those interested, here is the build log which I get
> > --- after the patch I sent in my previous email to jdee-users list is
> > applied, otherwise there would be even more errors.
> >
> > The problem one can see from this log is the following one: in
> > semantic, I have function
> >
> > (defun semantic-add-minor-mode (toggle name)
> >   ...)
> >
> > which takes *two* arguments. This function is defined in
> > cedet/lisp/cedet/semantic/util-modes.el at line 119 (Bzr-8556)
> >
> >
> > but in jde-bug.el (SVN-278), at line  this function is called at line
> > 573 like this with *three* arguments:
> >
> > (semantic-add-minor-mode 'jde-bug-minor-mode " JDEbug" jde-bug-mode-map)
> >
> > This seems to be one more compatibility issue of jdee with latest
> > cedet. Probably one should resort to some tricks to handle both older
> > and newer versions of CEDET.
> >
> > VBR,
> >     Vincent.
> >
> >
> >
> >
> > ------------------------------------------------------------------------------
> > Minimize network downtime and maximize team effectiveness.
> > Reduce network management and security costs.Learn how to hire
> > the most talented Cisco Certified professionals. Visit the
> > Employer Resources Portal
> > http://www.cisco.com/web/learning/employer_resources/index.html
> >
> >
> >
> > _______________________________________________
> > cedet-semantic mailing list
> > ced...@li...
> > https://lists.sourceforge.net/lists/listinfo/cedet-semantic