Today, a package that has an HTML vignette (but no PDF vignette) failed R CMD check --as-cran on a system without qpdf. I think the warning originates here [1], due to a premature check for the existence of qpdf [2]. Setting R_QPDF=true (as in /bin/true) helped, but perhaps it's possible to check qpdf existence only when it matters. I have attached a patch (untested) that could serve as a starting point. The code links correspond to SVN revision 69500. Thanks. Best regards Kirill [1] https://github.com/wch/r-source/blob/f42ee5e7ecf89a245afd6619b46483f1e3594ab7/src/library/tools/R/check.R#L322-L326, [2] https://github.com/wch/r-source/blob/f42ee5e7ecf89a245afd6619b46483f1e3594ab7/src/library/tools/R/check.R#L4426-L4428 -------------- next part -------------- A non-text attachment was scrubbed... Name: qpdf-missing.patch Type: text/x-patch Size: 2145 bytes Desc: not available URL: <https://stat.ethz.ch/pipermail/r-devel/attachments/20151010/2682c0ca/attachment.bin>
On 10/10/2015 6:30 AM, Kirill M?ller wrote:> Today, a package that has an HTML vignette (but no PDF vignette) failed > R CMD check --as-cran on a system without qpdf. I think the warning > originates here [1], due to a premature check for the existence of qpdf > [2]. Setting R_QPDF=true (as in /bin/true) helped, but perhaps it's > possible to check qpdf existence only when it matters. > > I have attached a patch (untested) that could serve as a starting point. > The code links correspond to SVN revision 69500. Thanks.The patch looks fine, and is now in R-devel and (in a few minutes) in R-patched. Thanks! Duncan Murdoch