From keitt@nceas.ucsb.edu  Tue Feb  8 20:22:16 2000
From: keitt@nceas.ucsb.edu (Timothy H. Keitt)
Date: Tue, 08 Feb 2000 12:22:16 -0800
Subject: [Rd] 0.99
Message-ID: <38A07AF8.E9A7BFF5@nceas.ucsb.edu>

Congrats on the new release.  The new package management tools are
great!

A couple of quick comments:

1) it might be nice to add an optional "Autoloads:" field to the
description (the package does not depend on these other packages, but
will autoload them if certain functions are called).

2) on page 3 of the R-exts document, it says to set PKG_* flags in
'Makeconf'; I think this should read 'src/Makevars'  (this paragraph in
general is pretty awkward reading).

3) I prefer PKG_LDFLAGS for the "-L" stuff and PKG_LIBS for the "-l"
statements (currently the docs say to put it all in PKG_LIBS).

4) page 5 of R-exts gives an example of using autoconf to enable/disable
R functions depending on whether or not a library is available;
currently it suggests putting a test in the body of the function that is
executed every time the function is called, even if the external library
exists.  This is confusing flow control with conditional compilation. 
Perhaps the example could be reworked so that the function itself is
defined differently depending on whether or not the external library is
found. (If the library does not exist, the function is defined to return
an error; otherwise define the function normally.)

Cheers,
Tim

-- 
Timothy H. Keitt
National Center for Ecological Analysis and Synthesis
735 State Street, Suite 300, Santa Barbara, CA 93101
Phone: 805-892-2519, FAX: 805-892-2510
http://www.nceas.ucsb.edu/~keitt/
-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-
r-devel mailing list -- Read http://www.ci.tuwien.ac.at/~hornik/R/R-FAQ.html
Send "info", "help", or "[un]subscribe"
(in the "body", not the subject !)  To: r-devel-request@stat.math.ethz.ch
_._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._._


[Rd] 0.99
Timothy H. Keitt