ESS: miscompiled macro

[Rodney Sparapani]

Jerry:

I had this:
>
>------------------------------------------------------------------------
>Compiling /home/james/xemacs/packages/xemacs-packages/ess/lisp/ess-menu.el...
>Loading /usr/local/test/lib/xemacs-21.5-b17/lisp/cl-macs.elc...
>Loading /home/james/xemacs/packages/xemacs-packages/xemacs-base/outline.elc...
>While compiling the end of the data in file 
/home/james/xemacs/packages/xemacs-p
>ackages/ess/lisp/ess-menu.el:
>  ** The following functions are not known to be defined:
>    speedbar-with-writable, speedbar-add-supported-extension
>Wrote /home/james/xemacs/packages/xemacs-packages/ess/lisp/ess-menu.elc
>Done
>------------------------------------------------------------------------
>

But I didn't have this:  
>
>speedbar-with-writable  xemacs-packages/speedbar/speedbar.el    defmacro
>
>Do you have that one in your macro.list?
>
>Thanks for helping me track this down.  Regards,
>-- 
>Jerry James

I was confused by what you meant by "the top-level package directory".  Here, I 
thought you meant the package you are inspecting, i.e. ESS.  However, you really 
meant /usr/local/lib/xemacs/xemacs-packages/lisp.  After all that, I checked 
Adrian's patch and I still get the same warning.  But, that gave me the hint I 
needed: 
(eval-when-compile
  (condition-case nil
      (progn
        (require 'speedbar)
        (when (featurep 'speedbar)
          (message "enabling speedbar support")

	    (defun S-speedbar-buttons (buffer)
		"attempted hack."

		;;(speedbar-make-tag-line)
		;;(speedbar-insert-button)
		(speedbar-with-writable))

	    (fset 'R-speedbar-buttons 'S-speedbar-buttons)

	    (defun S-speedbar-menu-items  ( )
		"Need to write.")

          (ess-S-initialize-speedbar)))
    (error nil)))

Rodney Sparapani              Medical College of Wisconsin
Sr. Biostatistician           Patient Care & Outcomes Research
rsparapa at mcw.edu              http://www.mcw.edu/pcor
Was 'Name That Tune' rigged?  WWLD -- What Would Lombardi Do