[Bug 1504825] Review Request: R-RColorBrewer - Provides color schemes for maps (and other graphics)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1504825



--- Comment #6 from Bill Crawford <billcrawford1970@xxxxxxxxxxxxxx> ---

SPEC:
https://copr-be.cloud.fedoraproject.org/results/willc/r-packages/fedora-26-x86_64/00684361-R-RColorBrewer/R-RColorBrewer.spec
SRPM:
https://copr-be.cloud.fedoraproject.org/results/willc/r-packages/fedora-26-x86_64/00684361-R-RColorBrewer/R-RColorBrewer-1.1.2-2.fc26.src.rpm

The R "check" gives two NOTEs, as follows:

* checking DESCRIPTION meta-information ... NOTE
Malformed Description field: should contain one or more complete sentences.

- This seems slightly unreasonable, since it looks like a good enough sentence
to me, but since it ends in a URL, adding a period might break it :o)

* checking R code for possible problems ... NOTE
brewer.pal: no visible global function definition for 'rgb'
display.brewer.all: no visible global function definition for 'par'
display.brewer.all: no visible global function definition for 'plot'
display.brewer.all: no visible global function definition for 'rect'
display.brewer.all: no visible global function definition for 'text'
display.brewer.pal: no visible global function definition for 'image'
Undefined global functions or variables:
  image par plot rect rgb text
Consider adding
  importFrom("grDevices", "rgb")
  importFrom("graphics", "image", "par", "plot", "rect", "text")
to your NAMESPACE file.

- the package appears to work well despite these alarming warnings; running {{{
    library("RColorBrewer");
    display.brewer.all();
}}} works with no visible errors ...

Is it worth pestering upstream about these? Seems like it might be impolitic to
say the least...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux