feedback on 2.3.4beta4

[Ed Mooney]

I installed 2.3.4beta4 on Solaris 9 and have these comments.
* I had to make this change to get lisp/makefile to work:
    all:
          test -d $(SEMANTIC) -a -d $(SPEEDBAR) -a -d $(ELIB) -a -d 
$(EIEIO)
  ***************
  *** 25,31 ****
          echo "(setq debug-on-error t)" >> jde-compile-script-init
          echo "(require 'jde-compat)" >> jde-compile-script-init
          echo "(require 'jde)" >>jde-compile-script-init
  ! 	if test `echo "$(EMACS)" | grep -w xemacs` ; \
            then $(EMACS) -batch -l jde-compile-script-init -f 
batch-byte-compile `echo *.el` ; \
            else $(EMACS) -batch -l jde-compile-script-init \
              -f batch-byte-compile `ls -1 *.el | egrep -v 
'jde-xemacs.el'`; \
  --- 25,31 ----
          echo "(setq debug-on-error t)" >> jde-compile-script-init
          echo "(require 'jde-compat)" >> jde-compile-script-init
          echo "(require 'jde)" >>jde-compile-script-init
  ! 	if echo "$(EMACS)" | grep -w xemacs ; \
            then $(EMACS) -batch -l jde-compile-script-init -f 
batch-byte-compile `echo *.el` ; \
            else $(EMACS) -batch -l jde-compile-script-init \
              -f batch-byte-compile `ls -1 *.el | egrep -v 
'jde-xemacs.el'`; \

* I had to make these changes to jde.el:
      *** jde.el        2004/06/01 18:17:56     1.1
      --- jde.el        2004/06/01 19:57:42
      ***************
      *** 38,43 ****
      --- 38,46 ----
        ;;;###autoload
      + (defconst cedet-version "1.0beta2b"
      +   "Cedet version number.")
      +
        (defconst jde-version "2.3.4beta4"
          "JDE version number.")
  Without this change, emacs complained that cedet-version is an
  undefined variable (or some such). This was true even though I
  had set jde-check-version-flag to nil.
      ***************
      *** 1252,1257 ****
      --- 1255,1262 ----
        (defvar jde-cygwin-root-cache nil
          "Cache of converted cygwin root directory paths.")
      + (defun jde-cygwin-path-converter-noop (path) path)
      +
        (defun jde-cygwin-path-converter-cygpath (path)
          (interactive "sPath: ")
          (if (string-match "^[a-zA-Z]:" path)
  This change lets me set jde-cygwin-path-converter to
  jde-cygwin-path-converter-noop, disabling conversion of paths
  beginning with /cygdrive to DOS format. Comments in jde.el led
  me to think that setting jde-cygwin-path-converter to
  jde-cygwin-path-converter-cygpath would disable such
  conversions on my Solaris system, but that seemed not to be
  the case.
With these changes, I'm up and running and intend to shake it
out some more. I'm delighted no longer to see:
      Wrong type argument: stringp, nil
when I load a second Java file (I'm running with cedet-1.0beta2b).
Regards,
--
Ed Mooney         |Sun Microsystems, Inc.|Time flies like
Java Web Services |UBUR02-201            |an arrow, but
[EMAIL PROTECTED] |1 Network Drive       |fruit flies like
781-442-0459      |Burlington, MA  01803 |a banana. Groucho