From dtenenba at fhcrc.org  Sat Jul 12 01:49:09 2014
From: dtenenba at fhcrc.org (Dan Tenenbaum)
Date: Fri, 11 Jul 2014 16:49:09 -0700 (PDT)
Subject: [Rd] proposed change of check message when VIgnetteBuilder package
 is not declared as a dependency
In-Reply-To: <1141814523.922119.1405122039081.JavaMail.root@fhcrc.org>
Message-ID: <846333300.922260.1405122549438.JavaMail.root@fhcrc.org>

Hi,

I notice in R-3.1.1, if a package specifies a VignetteBuilder (such as knitr) in its DESCRIPTION file, but
does not also depend on the specified package (in either Depends, Imports, or Suggests), that R CMD check will say:

* checking package dependencies ... ERROR
VignetteBuilder package not declared: ?knitr?

I'm glad this situation is now caught by R CMD check, but I think the message text should be changed, because the message is confusing. Indeed, the VignetteBuilder package *was* declared, in the VignetteBuilder line. The problem is that there's no dependency on it, so no automated way to install it for building or checking.

Can the message instead say something like:

Package 'knitr' must be in Depends, Imports, or Suggests field of DESCRIPTION file.

?

A possible patch against trunk:

$ svn diff
Index: src/library/tools/R/QC.R
===================================================================
--- src/library/tools/R/QC.R	(revision 66127)
+++ src/library/tools/R/QC.R	(working copy)
@@ -3056,7 +3056,7 @@
       if(length(bad <- x$required_for_checking_but_not_installed) > 1L) {
           c(.pretty_format2("VignetteBuilder packages required for checking but not installed:", bad), "")
       } else if(length(bad)) {
-          c(sprintf("VignetteBuilder package required for checking but installed: %s", sQuote(bad)), "")
+          c(sprintf("Package %s must be in Depends, Imports, or Suggests field of DESCRIPTION file.", sQuote(bad)), "")
       },
       if(length(bad <- x$missing_vignette_depends)) {
           c(if(length(bad) > 1L) {


Thanks,
Dan



[Rd] proposed change of check message when VIgnetteBuilder package is not declared as a dependency
Dan Tenenbaum