<div dir="ltr"><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Thank you for the feedback. It helped, although not in the direction I had in mind originally but the proposed directions are equally fine.<br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif"><br></div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">many thanks,</div><div class="gmail_default" style="font-family:trebuchet ms,sans-serif">Jan<br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><br><span style="font-family:trebuchet ms,sans-serif"></span><span style="font-family:trebuchet ms,sans-serif"><font face="trebuchet ms,sans-serif">Jan Wijffels<br>Statistician<br><a href="http://www.bnosac.be" target="_blank">www.bnosac.be</a>  | +32 486 611708</font><br></span></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On 11 September 2017 at 13:28, Dirk Eddelbuettel <span dir="ltr"><<a href="mailto:edd@debian.org" target="_blank">edd@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Jan: This is hard/impossible for me to read. I do not use a html email<br>
client, so if you used colour here, I don't see.  Please use indentation if<br>
you can.<br>
<span class=""><br>
On 10 September 2017 at 22:35, Jan Wijffels wrote:<br>
|  Solaris no longer really matters.<br>
| Does that mean that CRAN will accept a package which gives errors on<br>
| Solaris?<br>
<br>
</span>I don't know. Only CRAN can speak to that. But Solaris no exists (as an<br>
Oracle product) and is no longer maintained at U Oxford meaning CRAN may noe<br>
have test machines any more.<br>
<span class=""><br>
|<br>
| You could add C++11 conditionally, ie only on Linux/macOS/Windows<br>
| Yes, but I always need C++11 for this package. I just wanted to make a<br>
| small reproducible example. The bigger reproducible example is the package<br>
| at <a href="https://github.com/bnosac/udpipe" rel="noreferrer" target="_blank">https://github.com/bnosac/<wbr>udpipe</a><br>
<br>
</span>Fine, then use it. I use CXX_STD=CXX11 in many of my packages too.<br>
<span class=""><br>
| If I add the Rcpp.h header to file<br>
| <a href="https://github.com/bnosac/udpipe/blob/master/src/udpipe.cpp" rel="noreferrer" target="_blank">https://github.com/bnosac/<wbr>udpipe/blob/master/src/udpipe.<wbr>cpp</a>, I'm getting<br>
| issues similar to this<br>
|<br>
| udpipe.cpp:23406:30: error: macro "Free" passed 2 arguments, but takes just 1<br>
|    alloc->Free(alloc, p->probs);<br>
<br>
</span>Quite possibly not a Solaris issue but a simply _ordering issue_: place<br>
Rcpp.h last, not first, and/or tell R not to define macros. An `#undef`<br>
may help too.<br>
<span class=""><br>
| ​ Also [[Rcpp::plugins(cpp11)]] should not be needed in a package.<br>
| Thank you for the remark. The udpipe package currently uses CXX_STD=CXX11<br>
| in Makevars and specifies SystemRequirements: C++11 in DESCRIPTION. Example<br>
<br>
</span>You need one of these two, not both.<br>
<span class=""><br>
| was just put there to show a minimal example of the error, coming from when<br>
| Rcpp compiles the attributes.<br>
|<br>
| If you could provide a pointer where I should look for a solution, this<br>
| would be great otherwise the package which is now on CRAN will be rejected<br>
| any time soon I think.<br>
<br>
</span>Try reordering.<br>
<br>
Hth,  Dirk<br>
<div class="HOEnZb"><div class="h5"><br>
|<br>
| best,<br>
| Jan<br>
|<br>
|<br>
|<br>
|<br>
|<br>
|<br>
| Jan Wijffels<br>
| Statistician<br>
| <a href="http://www.bnosac.be" rel="noreferrer" target="_blank">www.bnosac.be</a>  | +32 486 611708<br>
|<br>
| On 6 September 2017 at 18:49, Dirk Eddelbuettel <<a href="mailto:edd@debian.org">edd@debian.org</a>> wrote:<br>
|<br>
| ><br>
| > Hi Jan,<br>
| ><br>
| > On 6 September 2017 at 15:03, Jan Wijffels wrote:<br>
| > | I'm trying to fix the installation issue I have with the package udpipe:<br>
| > | <a href="https://github.com/bnosac/udpipe" rel="noreferrer" target="_blank">https://github.com/bnosac/<wbr>udpipe</a> but I'm getting issues on Solaris<br>
| > giving<br>
| > | messages like this:<br>
| ><br>
| > Solaris no longer really matters.<br>
| ><br>
| > | If I do not add ​SystemRequirements: C++11 and remove the Makevars file,<br>
| > | this works fine on Solaris.<br>
| ><br>
| > You could add C++11 conditionally, ie only on Linux/macOS/Windows.  You<br>
| > could<br>
| > use (even a shell script named) configure for that.<br>
| ><br>
| > | I need however C++11 as I need it in the udpipe<br>
| > | R package for building udpipe.cpp but can't add<br>
| > | to that udpipe.cpp file the following types of statements due to<br>
| > conflicts<br>
| > | on the use of the Free macro.<br>
| > | #include <Rcpp.h><br>
| > | using namespace Rcpp;<br>
| > | // [[Rcpp::plugins(cpp11)]]<br>
| ><br>
| > Can you explain a bit more?  What Free macro?  Do you know about forcing R<br>
| > to<br>
| > add Rf_ to its macros?<br>
| ><br>
| > Also [[Rcpp::plugins(cpp11)]] should not be needed in a package.<br>
| ><br>
| > Dirk<br>
| ><br>
| > --<br>
| > <a href="http://dirk.eddelbuettel.com" rel="noreferrer" target="_blank">http://dirk.eddelbuettel.com</a> | @eddelbuettel | <a href="mailto:edd@debian.org">edd@debian.org</a><br>
| ><br>
<br>
--<br>
<a href="http://dirk.eddelbuettel.com" rel="noreferrer" target="_blank">http://dirk.eddelbuettel.com</a> | @eddelbuettel | <a href="mailto:edd@debian.org">edd@debian.org</a><br>
</div></div></blockquote></div><br></div>