[Refdb-devel] refdb-elisp

[Markus H. <mar...@mh...>]

Hi David,

David Nebauer writes:
 > 1.  Makefile document directory
 > 
 > I had to alter Makefile.am: change
 >     docdir = $(datadir)/doc/$(PACKAGE)-$(VERSION)
 > to
 >     docdir = $(datadir)/doc/$(PACKAGE)
 > 
 > If you do not do this the documentation is split between 
 > /usr/share/doc/refdb-elisp and /usr/share/doc/refdb-elisp-1.0.  Further, 
 > the debian (dwww) document registration script gets confused.
 > 

Why is it "split"? What does end up in /usr/share/doc/refdb-elisp? I
thought this install directory is safe to use as we use the same in
libdbi and in RefDB.

 > 3.  Compile error
 > 
 > The following error occurs at compile time:
 > --------------------------------------------------------------------------------
 > While compiling refdb-getnote-by-field-on-region in file 
 > /usr/share/emacs21/site-lisp/refdb-elisp/refdb-mode.el:
 >   ** Malformed let binding: `(concat refdb-regexp-query-string (value 
 > (buffer-substring (mark) (point))) refdb-regexp-query-string)'
 > --------------------------------------------------------------------------------

Thanks for catching this. I didn't notice this between all those
harmless warnings.

 > 5.  Inappropriate hard-coded values in refdb-mode
 > 
 > 95:   ;;    load path (e.g. /usr/local/share/emacs/site-lisp), or add
 > to:   ;;    load path (e.g. /usr/share/emacs/site-lisp), or add
 > 
 > 877:  (defcustom refdb-rtf-view-program 
 > "/usr/local/OpenOffice.org1.1.4/program/swriter"
 > to:   (defcustom refdb-rtf-view-program "/usr/bin/oowriter"
 > 
 > 5125: (let* ((globalconfig-localpath (format "/usr/local/etc/refdb/%s" 
 > globalconfig-file))
 > to:   (let* ((globalconfig-localpath (format "/etc/refdb/%s" 
 > globalconfig-file))
 > 
 > 5205: (refdbdrc-localpath "/usr/local/etc/refdb/refdbdrc")
 > to:   (refdbdrc-localpath "/etc/refdb/refdbdrc")
 > 
 > 5446: (let* ((refdbdrc-localpath "/usr/local/etc/refdb/refdbdrc")
 > to:   (let* ((refdbdrc-localpath "/etc/refdb/refdbdrc")
 > 
 > 5618: (refdbdrc-localpath "/usr/local/etc/refdb/refdbdrc")
 > to:   (refdbdrc-localpath "/etc/refdb/refdbdrc")
 > 

Yeah, I was a bit lazy at this point to get refdb-mode done in time. I
think it's best to let configure check for the appropriate paths and
apps (e.g. I can check for either swriter, oowriter, abiword, or kword
as a rtf viewer, and use whichever is found first) and let the
Makefile preconfigure refdb-configure.el from an .in file.

 > 
 > 6.  ?? Failure
 > 
 > After installation of the package I was unable to load either mode.  
 > This could easily be due to my _complete_ ignorance of emacs.
 > 

Did you try to handle refdb-configure.el automatically? This needs to
be inserted into your site-lisp.el file, or into ~/.emacs. I believe
Debian has a site-lisp.d directory where apps can dump these startup
code fragments. Without the code in refdb-configure.el emacs does not
load the modes.
 > 
 > I'll send you the package off-list.  Please try it on your Debian system 
 > and let me know if it works.
 > 

I'll do this weekend. And thanks a lot for touching Emacs in the first
place.

regards,
Markus

-- 
Markus Hoenicka
mar...@ca...
(Spam-protected email: replace the quadrupeds with "mhoenicka")
http://www.mhoenicka.de