Re: JDE 2.1.9 checked into CVS

Andreas> Could you - or anybody else - please fix this asap? I'd Andreas> like to get a new package release out including a new Andreas> sumo and the broken JDE blocks the release (I'd like to Andreas> have JDE in - but if nobody can fix soon it, the sumo Andreas> will come with the old version). Stephen> I don't know if this will help all problems and I don't have time to Stephen> learn enough about the package to try to test it, but I did have a Stephen> chance to CVS up and look at it. Stephen> bash-2.04$ fgrep -l defmacro *.el Stephen> eieio.el Stephen> jde-db.el Stephen> jde-run.el Stephen> jde.el Stephen> bash-2.04$ fgrep -l defstruct *.el Stephen> bash-2.04$ fgrep PRELOAD ../Makefile Stephen> bash-2.04$ Stephen> strongly suggests that adding Stephen> PRELOADS = -l eieio.el -l jde.el -l jde-db.el -l jde-run.el Stephen> somewhere above the inclusion of XEmacs.rules should help a lot to Stephen> prevent broken .elcs. In fact, jde.el seems to use macros only for Stephen> compatibility with very old versions of customize, and jde-db.el Stephen> contains all calls to the single macro it defines; they can probably Stephen> be omitted from PRELOADS. But eieio.el is just full of macros, used Stephen> all over the place. Using: PRELOADS = -eval "(push (expand-file-name \"lisp\") load-path)" -l cl-macs \ -l cus-face -l eieio.el -l jde.el -l jde-db.el -l jde-run.el Let's me compile everything. I've installed the package locally and couldn't find a direct problem. I'm uploading a new package now for testing. Thanks, Andreas 2000-08-23 Andreas Jaeger <aj(a)suse.de&gt; * Makefile (PRELOADS): Added, as suggested by Stephen J. Turnbull <turnbull(a)sk.tsukuba.ac.jp&gt; Index: prog/jde/Makefile =================================================================== RCS file: /usr/CVSroot/XEmacs/xemacs-packages/prog/jde/Makefile,v retrieving revision 1.27 diff -u -u -r1.27 Makefile --- Makefile 2000/08/17 11:28:14 1.27 +++ Makefile 2000/08/23 08:35:49 ＠＠ -36,6 +36,9 ＠＠ lisp/beanshell.elc EXTRA_SOURCES = lisp/eieio.el +PRELOADS = -eval "(push (expand-file-name \"lisp\") load-path)" -l cl-macs \ + -l cus-face -l eieio.el -l jde.el -l jde-db.el -l jde-run.el + #INFO_FILES = $(PACKAGE).htm *.gif DATA_1_FILES = $(wildcard java/classes/jde/wizards/*.class) DATA_1_DEST = jde/java/classes/jde/wizards -- Andreas Jaeger SuSE Labs aj(a)suse.de private aj(a)arthur.inka.de http://www.suse.de/~aj