Thread: [cedet-eieio] Test suite failure
Brought to you by:
zappo
From: <lu...@gn...> - 2009-02-25 10:40:10
|
Hi, EIEIO's test suite from CEDET 1.0pre4 fails: for loadpath in . ../common/ ../speedbar/ ; do \ echo "(add-to-list 'load-path \"$loadpath\")" >> test-compile-script; \ done; "emacs" -batch --no-site-file -l test-compile-script -f batch-byte-compile eieio-tests.el eieio-test-methodinvoke.el !! File eieio uses old-style backquotes !! Class allocated slots do not need :initarg In toplevel form: eieio-tests.el:474:1:Error: Symbol's value as variable is void: class-typep-var This is with GNU Emacs 22.3. Any idea? Thanks, Ludo'. |
From: Eric M. L. <er...@si...> - 2009-02-25 13:36:05
|
>>> lu...@gn... (Ludovic =?iso-8859-1?Q?Court=E8s?=) seems to think that: >Hi, > >EIEIO's test suite from CEDET 1.0pre4 fails: > > for loadpath in . ../common/ ../speedbar/ ; do \ > echo "(add-to-list 'load-path \"$loadpath\")" >> test-compile-script; \ > done; > "emacs" -batch --no-site-file -l test-compile-script -f batch-byte-compile eieio-tests.el eieio-test-methodinvoke.el > !! File eieio uses old-style backquotes !! > Class allocated slots do not need :initarg > > In toplevel form: > eieio-tests.el:474:1:Error: Symbol's value as variable is void: class-typep-var > >This is with GNU Emacs 22.3. > >Any idea? [ ... ] Hi, I have made a lot of changes to the test harness since CEDET 1.0pre4, and am in the process of trying to fix the last couple bugs I know of for a new release. I tried out the CVS version with an Emacs 22.1 install I found, but did not see that problem. If you have interests specifically in EIEIO, I would suggest getting the latest from CVS as it has some warnings about an EIEIO feature that needs to be deprecated to make it more compatible with CLOS. Eric -- Eric Ludlam: er...@si... Siege: www.siege-engine.com Emacs: http://cedet.sourceforge.net |
From: <lu...@gn...> - 2009-03-02 14:51:15
|
Hi, lu...@gn... (Ludovic Courtès) writes: > Hi, > > EIEIO's test suite from CEDET 1.0pre4 fails: > > for loadpath in . ../common/ ../speedbar/ ; do \ > echo "(add-to-list 'load-path \"$loadpath\")" >> test-compile-script; \ > done; > "emacs" -batch --no-site-file -l test-compile-script -f batch-byte-compile eieio-tests.el eieio-test-methodinvoke.el > !! File eieio uses old-style backquotes !! > Class allocated slots do not need :initarg > > In toplevel form: > eieio-tests.el:474:1:Error: Symbol's value as variable is void: class-typep-var > > This is with GNU Emacs 22.3. FWIW the issue still shows up with CEDET 1.0pre6, followed by another one: In toplevel form: eieio-tests.el:521:1:Error: Symbol's value as variable is void: class-typep-var Wrote /tmp/nix-build-hajhxyz4l5pd8lsl89df3ina4fyd9vfi-cedet-1.0pre6.drv-0/cedet-1.0pre6/eieio/eieio-test-methodinvoke.elc In end of data: eieio-perftest.el:149:1:Warning: the function `semantic-elapsed-time' is not known to be defined. Thanks, Ludo'. |
From: Eric M. L. <er...@si...> - 2009-03-02 22:32:18
|
>>> lu...@gn... (Ludovic =?iso-8859-1?Q?Court=E8s?=) seems to think that: >Hi, > >lu...@gn... (Ludovic Courtès) writes: > >> Hi, >> >> EIEIO's test suite from CEDET 1.0pre4 fails: >> >> for loadpath in . ../common/ ../speedbar/ ; do \ >> echo "(add-to-list 'load-path \"$loadpath\")" >> test-compile-script; \ >> done; >> "emacs" -batch --no-site-file -l test-compile-script -f batch-byte-compile eieio-tests.el eieio-test-methodinvoke.el >> !! File eieio uses old-style backquotes !! >> Class allocated slots do not need :initarg >> >> In toplevel form: >> eieio-tests.el:474:1:Error: Symbol's value as variable is void: class-typep-var >> >> This is with GNU Emacs 22.3. > >FWIW the issue still shows up with CEDET 1.0pre6, followed by another >one: > > In toplevel form: > eieio-tests.el:521:1:Error: Symbol's value as variable is void: class-typep-var > Wrote /tmp/nix-build-hajhxyz4l5pd8lsl89df3ina4fyd9vfi-cedet-1.0pre6.drv-0/cedet-1.0pre6/eieio/eieio-test-methodinvoke.elc > > In end of data: > eieio-perftest.el:149:1:Warning: the function `semantic-elapsed-time' is not > known to be defined. [ ... ] Hi, In my previous reply, there is one thing I hadn't noticed before, which is that you must have typed "make test" in the eieio directory to do this. The command "make test" in the EIEIO directory is not actually supposed to be run. I hadn't realized that is how you got into this state. I thought you were running "make utest" from the cedet directory, not "make test" from the eieio directory. The "make test" command attempts to compile the tests file, which cannot be compiled. Was there something that made you think this should be run to test something? The fact that it exists is a side effect of the makefile generator being used, but it should not be run during normal compilation. Eric -- Eric Ludlam: er...@si... Siege: www.siege-engine.com Emacs: http://cedet.sourceforge.net |
From: Eric M. L. <er...@si...> - 2009-03-02 20:10:43
|
>>> lu...@gn... (Ludovic =?iso-8859-1?Q?Court=E8s?=) seems to think that: >Hi, > >lu...@gn... (Ludovic Courtès) writes: > >> Hi, >> >> EIEIO's test suite from CEDET 1.0pre4 fails: >> >> for loadpath in . ../common/ ../speedbar/ ; do \ >> echo "(add-to-list 'load-path \"$loadpath\")" >> test-compile-script; \ >> done; >> "emacs" -batch --no-site-file -l test-compile-script -f batch-byte-compile eieio-tests.el eieio-test-methodinvoke.el >> !! File eieio uses old-style backquotes !! >> Class allocated slots do not need :initarg >> >> In toplevel form: >> eieio-tests.el:474:1:Error: Symbol's value as variable is void: class-typep-var >> >> This is with GNU Emacs 22.3. > >FWIW the issue still shows up with CEDET 1.0pre6, followed by another >one: > > In toplevel form: > eieio-tests.el:521:1:Error: Symbol's value as variable is void: class-typep-var > Wrote /tmp/nix-build-hajhxyz4l5pd8lsl89df3ina4fyd9vfi-cedet-1.0pre6.drv-0/cedet-1.0pre6/eieio/eieio-test-methodinvoke.elc > > In end of data: > eieio-perftest.el:149:1:Warning: the function `semantic-elapsed-time' is not > known to be defined. > [ ... ] Hi, I was not able to reproduce this issue. I've run these tests on several Emacsen, though not 22.3. I did use 22.1 though. It might be as simple as needing a (require 'cl-macs) somewhere. Eric -- Eric Ludlam: er...@si... Siege: www.siege-engine.com Emacs: http://cedet.sourceforge.net |
From: <lu...@gn...> - 2009-03-03 19:48:32
|
Hi, "Eric M. Ludlam" <er...@si...> writes: > In my previous reply, there is one thing I hadn't noticed before, > which is that you must have typed "make test" in the eieio directory > to do this. > > The command "make test" in the EIEIO directory is not actually > supposed to be run. I hadn't realized that is how you got into this > state. I thought you were running "make utest" from the cedet > directory, not "make test" from the eieio directory. The "make test" > command attempts to compile the tests file, which cannot be compiled. > > Was there something that made you think this should be run to test > something? I think I initially looked for a `check' target (as with GNU standard compliant makefiles), then tried `test', and finally found out there was such a rule in sub-directory makefiles. Anyway, "make utest" in the top-level directory works like a charm. (I'm actually working on a package for NixOS.) Thanks! Ludo'. |