<br><br><div class="gmail_quote">On Sun, Jul 11, 2010 at 12:06 PM, Romain Francois <span dir="ltr"><<a href="mailto:romain@r-enthusiasts.com">romain@r-enthusiasts.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Le 11/07/10 17:51, Dominick Samperi a écrit :<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
On Sun, Jul 11, 2010 at 4:58 AM, Romain Francois<br></div>
<<a href="mailto:romain@r-enthusiasts.com" target="_blank">romain@r-enthusiasts.com</a> <mailto:<a href="mailto:romain@r-enthusiasts.com" target="_blank">romain@r-enthusiasts.com</a>>> wrote:Le<div class="im">
<br>
11/07/10 02:14, Dominick Samperi a écrit :<br>
<br>
Romain,<br>
<br>
I made two small changes that should have no impact at all since<br>
they only<br>
apply when you build using windows DLL's (not static libs).<br>
Obviously these<br>
are not tested by a win-builder submission.<br>
<br>
<br>
Why not.<br>
<br>
<br>
Because testing this would require PATH to be changed at CRAN to include<br>
the directory containing Rcpp.dll, and this must be done BEFORE the package<br>
build or check process. This would have to be done using a Windows batch<br>
file as part of the initialization. Doing it in configure.win will not<br>
work. Also,<br>
a real test would require another package (like cxxPack) that links against<br>
Rcpp dynamically.<br>
</div></blockquote>
<br>
As I said, check the runit.client.package.R test that does exactly this: checking that a client package works.<br>
<br>
We do test that documented ways to include Rcpp headers and link against the user libraries work.<br>
<br>
While on the subject, the current best way to pull header files in is to use LinkingTo: Rcpp and not:<br>
<br>
PKG_CPPFLAGS = $(shell $(R_HOME)/bin/Rscript --vanilla -e "Rcpp:::CxxFlags()")<br>
<br>
which is what several packages are still using, including cxxPack.<br></blockquote><div><br>Why are you telling me what I obviously already know? Your reply does not address<br>any of the points that I raised here about Windows dynamic linking. In particular, <br>
runit.client.package.R does not address these issues at all (only the STATIC<br>library libRcpp.a is used).<br><br>Dominick<br><br> <br></div></div><br>