Re: [R-pkg-devel] Need help to resolve NOTEs in auto check

2020-01-22 Thread peter dalgaard
(a) Spelling issues: False positive are checked manually by CRAN, so just tell 
them on submission. For software items, Uwe at some point recommended "Single 
quotes around software names such as 'pkg', functions should be written with 
parentheses as in foo()."

(b) The procedure for submitting is to run 'R CMD build' on the (unpackaged) 
source directory, then submit the resulting .tar.gz file. If you don't do that, 
the build time stamp will be absent. Similarly 'R CMD check' should be run on 
the output from 'R CMD build'. Like this (oups, that dir is out of sync on this 
computer):

Peters-iMac:ISWR-2nd-ed pd$ R CMD build ISwR
* checking for file ‘ISwR/DESCRIPTION’ ... OK
* preparing ‘ISwR’:
* checking DESCRIPTION meta-information ... OK
* checking for LF line-endings in source and make files and shell scripts
* checking for empty or unneeded directories
* looking to see if a ‘data/datalist’ file should be added
* re-saving .R files as .rda
  NB: *.R converted to .rda: other files may need to be removed
* re-saving tabular files
* creating default NAMESPACE file
* building ‘ISwR_2.0-5.tar.gz’

Peters-iMac:ISWR-2nd-ed pd$ R CMD check --as-cran ISwR_2.0-5.tar.gz 
* using log directory ‘/Users/pd/BIOSTAT/books/ISWR-2nd-ed/ISwR.Rcheck’
* using R version 3.6.0 (2019-04-26)
* using platform: x86_64-apple-darwin15.6.0 (64-bit)
* using session charset: UTF-8
* using option ‘--as-cran’
* checking for file ‘ISwR/DESCRIPTION’ ... OK
* this is package ‘ISwR’ version ‘2.0-5’
* checking CRAN incoming feasibility ... WARNING
Maintainer: ‘Peter Dalgaard ’

Insufficient package version (submitted: 2.0.5, existing: 2.0.8)

Days since last update: 2

The Date field is over a month old.
* checking package namespace information ... OK
*..


(c) Examples _will_ be run as part of the check procedure. If that produces 
stray files in the check directory, you will tget the message that you see. In 
interactive usage, example(...) will similarly create files in the current 
directory, which is verboten, so write to a temporary directory instead. (See 
help(tempfile) for specifics).

> On 22 Jan 2020, at 11:29 , Ian Walker  wrote:
> 
> Hello,
> 
> I'm trying to repackage an R function so I can update the licence. I'm stuck 
> with the automatic checks. The problem appears to be the following three 
> NOTEs in the checking output:
> 
> * checking CRAN incoming feasibility ... NOTE
> Maintainer: ‘Ian Walker <
> i.wal...@bath.ac.uk
>> ’
> 
> Possibly mis-spelled words in DESCRIPTION:
>  DEMs (7:503)
>  STL (7:47)
>  stereolithography (7:52)
>  stl (7:24)
> 
> The build time stamp is missing.
> 
> It's not clear what is wrong here - is it the spellings (which are fine) or 
> the time stamp? If the latter, how do I resolve this?
> 
> * checking DESCRIPTION meta-information ... NOTE
> Checking should be performed on sources prepared by ‘R CMD build’.
> 
> I can't make any sense of this issue!
> 
> * checking for non-standard things in the check directory ... NOTE
> Found the following files/directories:
>  ‘lovelyfunction.stl’ ‘volcano.stl’
> 
> These two .stl files are simply mentioned in the documentation's example code 
> - they're files that would be created if somebody ran the example code; 
> they're not in the R package I've created.
> 
> Thanks for any help,
> 
> Ian
> --
> Dr Ian Walker FHEA | Department of Psychology, University of Bath
> i.wal...@bath.ac.uk (academic) | m...@drianwalker.com (other matters)
> 
> Website: drianwalker.com | Twitter: twitter.com/ianwalker
> 
> My books:
> Research Methods and Statistics - a new, clear introduction 
> http://tinyurl.com/res-stats
> Research with People - the essential research textbook: http://amzn.to/sRbYxy
>   [[alternative HTML version deleted]]
> 
> __
> R-package-devel@r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-package-devel

-- 
Peter Dalgaard, Professor,
Center for Statistics, Copenhagen Business School
Solbjerg Plads 3, 2000 Frederiksberg, Denmark
Phone: (+45)38153501
Office: A 4.23
Email: pd@cbs.dk  Priv: pda...@gmail.com

__
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel


Re: [R-pkg-devel] Need help to resolve NOTEs in auto check

2020-01-22 Thread Ivan Krylov
Hello Ian,

On Wed, 22 Jan 2020 10:29:51 +
Ian Walker  wrote:

> * checking CRAN incoming feasibility ... NOTE
> Maintainer: ‘Ian Walker <
> i.wal...@bath.ac.uk
> >’  

This NOTE is there for CRAN staff to have another look at the
Maintainer: field in case this is someone's first submission or there
is a change in package ownership.

> Possibly mis-spelled words in DESCRIPTION:
>   DEMs (7:503)
>   STL (7:47)
>   stereolithography (7:52)
>   stl (7:24)

These should be explained in the "Optional comment" field in the CRAN
submission form.

> The build time stamp is missing.

> NOTE
> Checking should be performed on sources prepared by ‘R CMD build’.

How exactly do you build and check your package? It is recommended to
run the command "R CMD build ." in the source directory of the package,
then "R CMD check mypackage_version.tar.gz" (substituting the name of
the newly generated file for mypackage_version.tar.gz).

> These two .stl files are simply mentioned in the documentation's
> example code - they're files that would be created if somebody ran
> the example code; they're not in the R package I've created.

It might be a better idea to use file.temp() for these file paths. CRAN
policy says that packages should not write files anywhere besides R's
session temporary directory unless user explicitly passes the the path.

-- 
Best regards,
Ivan

__
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel