<div dir="ltr"><div>Thanks for the advice. It seems that R CMD check (locally and on winbuilder) throws the warning despite having SystemRequirements: GNU make. May not be a problem on CRAN itself.<br></div><div><br></div><div>I assumed that for CRAN one needed both Makevars and Makevars.win, so have always included both in submissions to CRAN.<br></div><div><br></div><div>I suspect Jeff's suggestion, to omit, does not apply to RcppParallel.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 24, 2018 at 3:31 AM, 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"><span class=""><br>
On 24 June 2018 at 03:22, Murray Efford wrote:<br>
| The RcppParallel introduction at <a href="https://rcppcore.github.io/RcppParallel/" rel="noreferrer" target="_blank">https://rcppcore.github.io/<wbr>RcppParallel/</a><br>
| instructs package writers to include a Makevars file with<br>
| <br>
| PKG_LIBS += $(shell ${R_HOME}/bin/Rscript -e "RcppParallel::<wbr>RcppParallelLibs()")<br>
| <br>
| However, R CMD check in R 3.5.0 generates a warning apparently because +=<br>
| and $(shell are non-portable GNU extensions. As a Windows user I'm not<br>
| familiar with makefiles and am not confident to work around this. Am I<br>
| missing something, or is there a simple solution? An update to the<br>
| introduction would be welcome.<br>
<br>
</span>Two things:<br>
<br>
i)  _Every_ R package which needs a Makevars _on Windows_ uses<br>
    src/Makevars.win -- see "Writing R Extensions"<br>
<br>
ii) If one needs GNU make, one can declare the dependency in DESCRIPTION via<br>
    SystemRequirements: -- again, see "Writing R Extensions"<br>
<br>
And we have just that:<br>
<br>
    SystemRequirements: GNU make, Windows: cmd.exe and cscript.exe, Solaris: g++ is required<br>
<br>
Does that answer your question?  If not, did you look at user reverse-depends<br>
of RcppParallel and examine what they do?  One can often copy working<br>
solution s from other packages...<br>
<span class="HOEnZb"><font color="#888888"><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>
</font></span></blockquote></div><br></div>