Re: [Rd] R CMD check warning about compiler warning flags

2017-12-20 Thread Duncan Murdoch
On 20/12/2017 5:48 PM, Hadley Wickham wrote: On Wed, Dec 20, 2017 at 4:26 PM, Prof Brian Ripley wrote: On 20/12/2017 17:42, Winston Chang wrote: On recent builds of R-devel, R CMD check gives a WARNING when some compiler warning flags are detected, such as -Werror, because they are non-portab

Re: [Rd] R CMD check warning about compiler warning flags

2017-12-20 Thread Hadley Wickham
On Wed, Dec 20, 2017 at 4:26 PM, Prof Brian Ripley wrote: > On 20/12/2017 17:42, Winston Chang wrote: >> >> On recent builds of R-devel, R CMD check gives a WARNING when some >> compiler warning flags are detected, such as -Werror, because they are >> non-portable. This appears to have been added

Re: [Rd] R CMD check warning about compiler warning flags

2017-12-20 Thread Prof Brian Ripley
On 20/12/2017 17:42, Winston Chang wrote: On recent builds of R-devel, R CMD check gives a WARNING when some compiler warning flags are detected, such as -Werror, because they are non-portable. This appears to have been added in this commit: https://github.com/wch/r-source/commit/2e80059 Tha

[Rd] R_RegisterCCallable

2017-12-20 Thread Therneau, Terry M., Ph.D.
I've just converted all the C symbols in bdsmatrix to use the more modern form, e.g.,    .C("charle",     changed to .C( Ccharlie, along with defining RCMethodDef in my initfile and adding .register=TRUE to the NAMESPACE file.  All due to a request from CRAN to modernize one aspect, w

[Rd] R CMD check warning about compiler warning flags

2017-12-20 Thread Winston Chang
On recent builds of R-devel, R CMD check gives a WARNING when some compiler warning flags are detected, such as -Werror, because they are non-portable. This appears to have been added in this commit: https://github.com/wch/r-source/commit/2e80059 I'm working on a package where these compiler war

[Rd] utils::unzip ignores overwrite argument, effectively

2017-12-20 Thread Gábor Csárdi
It does give a warning, but then it overwrites the files, anyway. Reproducible example below. This is R 3.4.3, but it does not seem to be fixed in R-devel: https://github.com/wch/r-source/blob/4a9ca3e5ac6b19d7faa7c9290374f7604bf0ef64/src/main/dounzip.c#L171-L174 FYI, Gábor dir.create(tmp <- temp