R devel - Feb 2014 - R CMD check, NOTEs, & S4-dependent packages

As I was running "R CMD check" on one of my older packages
(matlab), I was attempting to fix one of the NOTEs which
bitches about attempting to change the search path using
require().

In my case, I am repeating a package dependency for "methods"
package in my .onLoad() function. Can this scenario (based
on something B. Ripley discussed many years ago) no longer
occur? Just another R package anachronism? Was I really
modifying the search path by repeating a DESCRIPTION dependency?


.onLoad <- function(libname, pkgname) {
    ## In case namespace is loaded (via import) by package that
    ## doesn't depend on S4 methods and used in a session with
    ## non-default set of packages.
    require(methods)

    ?
}

You don't need this .onLoad anymore.  Just Depends on methods and use
import(methods) (or perhaps be more specific) in NAMESPACE

Kasper


On Tue, Feb 4, 2014 at 7:13 PM, Roebuck,Paul L
<proebuck@mdanderson.org>wrote:
> As I was running "R CMD check" on one of my older packages
> (matlab), I was attempting to fix one of the NOTEs which
> bitches about attempting to change the search path using
> require().
>
> In my case, I am repeating a package dependency for "methods"
> package in my .onLoad() function. Can this scenario (based
> on something B. Ripley discussed many years ago) no longer
> occur? Just another R package anachronism? Was I really
> modifying the search path by repeating a DESCRIPTION dependency?
>
>
> .onLoad <- function(libname, pkgname) {
>     ## In case namespace is loaded (via import) by package that
>     ## doesn't depend on S4 methods and used in a session with
>     ## non-default set of packages.
>     require(methods)
>
>     Š
> }
>
> ______________________________________________
> R-devel@r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel
>
	[[alternative HTML version deleted]]