warnings when loading CodeDepends

[wlandau]

After installing fe50582:

library(CodeDepends)
#> Warning: replacing previous import 'graph::addNode' by 'XML::addNode' when
#> loading 'CodeDepends'
#> Warning: replacing previous import 'graph::plot' by 'graphics::plot' when
#> loading 'CodeDepends'

This is affecting packages that import from CodeDepends.

[gmbecker]

So CodeDepends imports the XML for reasons which may or may not continue to be relevant to how @duncantl uses the package today. That could potentially be changed to a suggests or an importFrom, but it would take a bit of doing as the parts of the package that do touch XML functionality do so quite a bit, and adding all the ::'s after doing a requireNamespace or finding the full set of things for the importFrom is doable but will be annoying. The other one is a bit more puzzling, as CodeDepends doesn't import or depend on graphics. Off the top of my head I'm not really sure what's going on there. ~G

…

On Tue, Mar 13, 2018 at 12:50 PM, Will Landau ***@***.***> wrote: After installing fe50582 <fe50582> : library(CodeDepends)#> Warning: replacing previous import 'graph::addNode' by 'XML::addNode' when#> loading 'CodeDepends'#> Warning: replacing previous import 'graph::plot' by 'graphics::plot' when#> loading 'CodeDepends' This is affecting packages that import from CodeDepends. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#25>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3dsdaGExloN35R-zx0JNK0nNEF0ffrks5teCLygaJpZM4SpVtS> .

-- Gabriel Becker, Ph.D Scientist Bioinformatics and Computational Biology Genentech Research

[joshuaulrich]

... or finding the full set of things for the importFrom is doable but will be annoying.

You could probably modify the bit of R CMD check that finds imports from base packages to find imports from a specific package. That might be globally useful to the entire R community, since it's often said it's better to importFrom(pkg, foo) than import(pkg). I'd be willing to help, if you're interested.

[gmbecker]

This is fixed in 0.6.0, now in Github and submitted to CRAN.

[wlandau]

Thanks, Gabe!