Re: [CEDET-devel] Re: Compile errors using Emacs 23.1.95
Brought to you by:
zappo
From: Richard K. <em...@gm...> - 2010-04-23 07:22:52
|
Eric, Thanks for the fixes. I picked up the latest cvs (I noted that you changed eieio version to 1.3), and retested, but the problem seems to remain. Following script demonstrates the problem. #!/bin/sh # # Executing this demonstrates byte compile problem with the latest bzr # emacs in either the emacs-23 branch or the trunk. # # Set this to your cedet cvs rot directory CEDET_CVS=/usr/local/src/cedet # Test elisp code to be created. TEST_FILE=cedet-compile-problem.el # go to my cedet cvs root directory cd ${CEDET_CVS} # create five line elisp file to be loaded by emacs cat > ${TEST_FILE} << 'EOF' (load-file "common/inversion.el") (load-file "common/cedet-compat.el") (add-to-list 'load-path (expand-file-name "eieio")) (load-file "eieio/eieio-comp.el") (byte-compile-file "eieio/eieio.el") EOF # # Run emacs (latest bzr code from either emacs-23 branch or from the # trunk) and load the test code just created which results in error such as # eieio.el:2516:13:Error: Symbol's value as variable is void: filename # /usr/local/src/emacs-bzr/emacs-23/src/emacs -Q -l ${TEST_FILE} -----Original Message----- From: Eric M. Ludlam <er...@si...> To: Richard Kim <em...@gm...> Cc: 丰宁宁 <fn...@ex...>, cedet-devel <ced...@li...> Subject: Re: [CEDET-devel] Re: Compile errors using Emacs 23.1.95 Date: Thu, 22 Apr 2010 20:34:18 -0400 Thanks for all the info. I hadn't realized that EIEIO had changed, which is why I hadn't updated the version number. I updated that number today. I also ran a full distribution test on emacs22 and 23 (by using cedet/testdist.sh) which builds with both the Makefile and cedet-build.el, and that all worked. I don't have an Emacs 24 handy. If the simple version update works, then that'll do for CEDET 1.0, as the next step is to change all the file names about. Eric On 04/20/2010 10:58 PM, Richard Kim wrote: > Eric, > > I don't know anything about the byte compiler change, so I can't comment > on it. You may well be right. > > It seems like cedet can be byte compiled without errors if the makefile > is used, but not if cedet-build.el is used. > > I tried to find out what the difference was between the two. > The first thing that I found odd was that eieio.el version is 1.2 in > both cedet-cvs as well as emacs-23 bzr branch. However the two eieio.el > files differ with each other! > > Second observation is that makefile based byte compilation makes use of > inversion.el. For emacs 23.1.9x, inversion.el detects that eieio that > comes with emacs is 1.2 which is the version requested in cedet-packages > constant. Hence eieio from emacs seems to be used and the one from > cedet is ignored. > > Could these difference contribute to the byte compilation errors? > > > -----Original Message----- > From: 丰宁宁<fn...@ex...> > To: Eric M. Ludlam<er...@si...> > Cc: cedet-devel<ced...@li...> > Subject: [CEDET-devel] Re: Compile errors using Emacs 23.1.95 > Date: Tue, 20 Apr 2010 10:54:49 +0800 > > > > > > I examine changes in > Emacs 23.2, I found: > " > ** Byte compilation > changes > --- > *** Changing the > file-names generated by > byte-compilation by > redefining > the function > `byte-compile-dest-file' > before loading > bytecomp.el is obsolete. > Instead, customize > byte-compile-dest-file-function. > " > And just at this point, > I have test cedet-cvs > with Emacs 23.1.96, the > compile errors go > on.(ede-gnustep.el:552:1:Error: Symbol's value as variable is void: filename) > So, maybe it's Emacs > 23.2 - Byte compilation > changes that lead to the > compile errors. > > > > > > > > > ------------------------------------------------------------------------------ > Download Intel® Parallel Studio Eval > Try the new software tools for yourself. Speed compiling, find bugs > proactively, and fine-tune applications for parallel performance. > See why Intel Parallel Studio got high marks during beta. > http://p.sf.net/sfu/intel-sw-dev > _______________________________________________ > Cedet-devel mailing list > Ced...@li... > https://lists.sourceforge.net/lists/listinfo/cedet-devel > > > |