CEDET

Hi Klaus,

[...]
 > 3. With CVS GNU Emacs installing latest beta fails - not fine :-(
 >    So i can not test and run latest cedet-beta with CVS GNU Emacs.
 >
[...]
 > Does this report help you - what can be the problem?  Note: The
 > first beta of cedet 1.0 has installed fine with the same CVS GNU
 > Emacs! Can there be a problem in the changes added to cedet in the
 > meanwhile?
[...]

Thanks for your report.  You're right I introduced the bug(?) in my
last changes to cedet-autogen.el.  In fact, the "Local variable:"
string in `cedet-autogen-ensure-default-file' confused the standard file
function `hack-local-variables' that failed with that strange
"Local variables entry is missing the prefix" error :-(

The following trivial patch should fix the problem.  Does it?

David

Index: cedet/common/cedet-autogen.el
===================================================================
RCS file: /cvsroot/cedet/cedet/common/cedet-autogen.el,v
retrieving revision 1.3.2.1
diff -c -r1.3.2.1 cedet-autogen.el
*** cedet/common/cedet-autogen.el	14 Nov 2003 06:56:05 -0000	1.3.2.1
--- cedet/common/cedet-autogen.el	16 Nov 2003 12:59:58 -0000
***************
*** 170,176 ****
                 " --- automatically extracted autoloads\n"
                 ";;\n"
                 ";;; Code:\n\n"
!               "\n;; Local Variables:\n"
                 ";; version-control: never\n"
                 ";; no-byte-compile: t\n"
                 ";; no-update-autoloads: t\n"
--- 170,176 ----
                 " --- automatically extracted autoloads\n"
                 ";;\n"
                 ";;; Code:\n\n"
!               "\n;; Local" " Variables:\n"
                 ";; version-control: never\n"
                 ";; no-byte-compile: t\n"
                 ";; no-update-autoloads: t\n"

Hi David,

thanks for your fast responese and help...yes, your patch fix it - thanks?
BTW: Indeed strange: Why
the line
  \n;; Local" " Variables:\n"
is better than
  \n;; Local Variables:\n"
is really hard to understand??!!

What do you think about my suggestion about -no-site-file?

Thanks again,
Klaus

-----Original Message-----
From: David Ponce [mailto:david.ponce@...] 
Sent: Sunday, November 16, 2003 2:06 PM
To: Berndl, Klaus
Cc: 'Eric M. Ludlam'; cedet-devel@...
Subject: Re: [CEDET-devel] beta1c again - fails with CVS GNU Emacs


Hi Klaus,

[...]
 > 3. With CVS GNU Emacs installing latest beta fails - not fine :-(
 >    So i can not test and run latest cedet-beta with CVS GNU Emacs.
 >
[...]
 > Does this report help you - what can be the problem?  Note: The
 > first beta of cedet 1.0 has installed fine with the same CVS GNU
 > Emacs! Can there be a problem in the changes added to cedet in the
 > meanwhile?
[...]

Thanks for your report.  You're right I introduced the bug(?) in my
last changes to cedet-autogen.el.  In fact, the "Local variable:"
string in `cedet-autogen-ensure-default-file' confused the standard file
function `hack-local-variables' that failed with that strange
"Local variables entry is missing the prefix" error :-(

The following trivial patch should fix the problem.  Does it?

David

Index: cedet/common/cedet-autogen.el
===================================================================
RCS file: /cvsroot/cedet/cedet/common/cedet-autogen.el,v
retrieving revision 1.3.2.1
diff -c -r1.3.2.1 cedet-autogen.el
*** cedet/common/cedet-autogen.el	14 Nov 2003 06:56:05 -0000	1.3.2.1
--- cedet/common/cedet-autogen.el	16 Nov 2003 12:59:58 -0000
***************
*** 170,176 ****
                 " --- automatically extracted autoloads\n"
                 ";;\n"
                 ";;; Code:\n\n"
!               "\n;; Local Variables:\n"
                 ";; version-control: never\n"
                 ";; no-byte-compile: t\n"
                 ";; no-update-autoloads: t\n"
--- 170,176 ----
                 " --- automatically extracted autoloads\n"
                 ";;\n"
                 ";;; Code:\n\n"
!               "\n;; Local" " Variables:\n"
                 ";; version-control: never\n"
                 ";; no-byte-compile: t\n"
                 ";; no-update-autoloads: t\n"