Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

R CMD check fails with R-devel_2019-03-11 #27

Closed
ressy opened this issue Mar 12, 2019 · 3 comments
Closed

R CMD check fails with R-devel_2019-03-11 #27

ressy opened this issue Mar 12, 2019 · 3 comments

Comments

@ressy
Copy link
Member

ressy commented Mar 12, 2019

Running R CMD check chiimp_0.2.2.tar.gz fails on recent development versions of R:

...
* checking for portable file names ... OK
* checking for sufficient/correct file permissions ... OK
* checking whether package ‘chiimp’ can be installed ... ERROR
Installation failed.
See ‘/data/home/jesse/dev/chiimp-r-ver/chiimp.Rcheck/00install.out’ for details.
* DONE

Status: 1 ERROR
See
  ‘/data/home/jesse/dev/chiimp-r-ver/chiimp.Rcheck/00check.log’
for details.

The file chiimp.Rcheck/00install.out shows:

* installing *source* package ‘chiimp’ ...
** R
** inst
** byte-compile and prepare package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded from temporary location
** testing if installed package can be loaded from final location
** testing if installed package keeps a record of temporary installation path
ERROR: hard-coded installation path: please report to the package maintainer and use ‘--no-staged-install’
* removing ‘/data/home/jesse/dev/chiimp-r-ver/chiimp.Rcheck/chiimp’

This is causing CI via Travis to fail for the "devel" and "bioc-devel" jobs. it's fine on the latest stable release, though. Is this a bug in the development version or a more stringent check that didn't previously exist?

@ressy
Copy link
Member Author

ressy commented Mar 13, 2019

It's a more stringent check. It now looks for evidence of absolute paths hardcoded into the byte-compiled objects, and that actually does exist in the form of test_data$f.locus_attrs. That isn't even necessary to keep once the data is loaded, so a simple fix may be to just remove the path from the list entirely.

@ressy
Copy link
Member Author

ressy commented Mar 13, 2019

That looks like it'll work fine, but there are other problems with that same object, notably that sample.int no longer produces the same output given the same set.seed(0) call beforehand, and affects the RNG state for anything that follows. A simple fix for now may be to use RNGversion to control the behavior, like RNGversion(3.5.3).

@ressy
Copy link
Member Author

ressy commented Mar 13, 2019

Fixed in 515f8ed.

@ressy ressy closed this as completed Mar 13, 2019
@ressy ressy mentioned this issue Mar 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant