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

Reduce build time of package #7

Open
sdhutchins opened this issue Oct 2, 2018 · 8 comments
Open

Reduce build time of package #7

sdhutchins opened this issue Oct 2, 2018 · 8 comments

Comments

@sdhutchins
Copy link
Contributor

sdhutchins commented Oct 2, 2018

The travis-ci build job failed.

Could be due to:

  • Too many dependencies for the app.
@sdhutchins sdhutchins self-assigned this Oct 2, 2018
@sdhutchins sdhutchins added the bug label Oct 2, 2018
@sdhutchins
Copy link
Contributor Author

Current build configuration

{
  "r": [
    "oldrel",
    "release",
    "devel"
  ],
  "os": "linux",
  "dist": "trusty",
  "sudo": false,
  "cache": "packages",
  "group": "stable",
  ".result": "configured",
  "language": "r",
  "r_packages": [
    "tidyverse",
    "vegan"
  ],
  "after_success": [
    "R CMD INSTALL $PKG_TARBALL",
    "Rscript -e 'lintr::lint_package()'"
  ],
  "bioc_packages": [
    "BiocInstaller"
  ],
  "r_github_packages": [
    "joey711/phyloseq",
    "grunwaldlab/metacoder",
    "ropensci/taxa",
    "MadsAlbertsen/ampvis2"
  ],
  "warnings_are_errors": false
}

@sdhutchins
Copy link
Contributor Author

Currently, the build is breaking...I'll look into fixing that today, @grabear

7.25s$ Rscript -e 'deps <- devtools::dev_package_deps(dependencies = NA);devtools::install_deps(dependencies = TRUE);if (!all(deps$package %in% installed.packages())) { message("missing: ", paste(setdiff(deps$package, installed.packages()), collapse=", ")); q(status = 1, save = "no")}'
Error: Missing commas separating Remotes: 'github::joey711/phyloseq
    ggthemes'
Execution halted
The command "Rscript -e 'deps <- devtools::dev_package_deps(dependencies = NA);devtools::install_deps(dependencies = TRUE);if (!all(deps$package %in% installed.packages())) { message("missing: ", paste(setdiff(deps$package, installed.packages()), collapse=", ")); q(status = 1, save = "no")}'" failed and exited with 1 during .

@sdhutchins
Copy link
Contributor Author

A great article: http://jtleek.com/protocols/travis_bioc_devel/

@sdhutchins
Copy link
Contributor Author

@grabear Anything against bumping R to version 3.5.0?

Installing package into ‘/home/travis/R/Library’
(as ‘lib’ is unspecified)
ERROR: this R is version 3.4.4, package 'microbiome' requires R >=  3.5.0

@sdhutchins
Copy link
Contributor Author

The build is passing for R 3.5.1 and the development release of R. Just not the old release (which we can remove from .travis.yml).

@grabear
Copy link
Member

grabear commented Dec 18, 2018

3.5 is fine

@grabear
Copy link
Member

grabear commented Dec 20, 2018

microbiome/microbiome#101

Per this PR I'm going to change the R version to >= 3.3.0

@sdhutchins
Copy link
Contributor Author

We need to add oldrel back to travis.yml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants