Bug 17179 - improve message "namespace dependency not required"
Summary: improve message "namespace dependency not required"
Status: UNCONFIRMED
Alias: None
Product: R
Classification: Unclassified
Component: Wishlist (show other bugs)
Version: R-devel (trunk)
Hardware: Other Other
: P5 enhancement
Assignee: R-core
URL:
Depends on:
Blocks:
 
Reported: 2016-11-09 10:38 UTC by k_bugzilla
Modified: 2016-11-09 10:38 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description k_bugzilla 2016-11-09 10:38:58 UTC
I tried the following with my package:
I have importFrom("foo", "bar) in NAMESPACE and removed package "foo" from DESCRIPTION (Imports:).

When checking my package, this is correctly detected, but the error message confused me a bit at first:
"Namespace dependency not required: 'foo'"

I tought: "oh, it is required, why do you tell me otherwise". But then I realised it has to be read "a dependency was detected and it is not mentioned as being required in DESCRIPTION".

So maybe we can improve the message (and its plural form) a bit? E.g.
"Namespace dependency not marked as required (in DESCRIPTION): 'foo'"

(src/library/tools/R/QC.R)