Re: [cedet-semantic] Trouble compiling current CVS

[David E. <de...@ra...>]

Michael Reiher <re...@gm...> writes:
> On Thursday 16 July 2009 01:47:32 Eric M. Ludlam wrote:
>> Hi Michael,
>>
>>   I missed that you were using XEmacs.  I've tracked the problem down to
>> some recent changes we made in `mode-local.el'.
>>
>>   If you revert to CVS version 1.17 of cedet/common/mode-local.el, and
>> I'll guess that will get things working.
>>
>>   In the meantime, I need to debug what new thing in mode-local.el broke
>> XEmacs.  Perhaps an XEmacs expert can help me divine what went wrong
>> recently.

> I tried to narrow it down a bit, and it seems to be the latest revision of 
> mode-local.el (i.e. 1.22) which broke things. Using cedet HEAD with mode-
> local.el 1.21 lets it compile fine.

Yes, the hack introduced in 1.22 does not work with xemacs, since
load-file-name is always " /build/buildd/xemacsVERSION/lisp/loadup.el"
when activate-mode-local-bindings is called.

If you want to check in the first hack you posted in cedet-devel, you
could "adapt" it for xemacs by using

   (or  (and (featurep 'emacs) (boundp 'keep-mode-if-same))
        (and (featurep 'xemacs) (boundp 'just-from-file-name))

This is beyond ugly, though...

-David