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

bioconductor: 3.4 -> 3.6 #35023

Merged
merged 2 commits into from Feb 21, 2018
Merged

bioconductor: 3.4 -> 3.6 #35023

merged 2 commits into from Feb 21, 2018

Conversation

jbedo
Copy link
Contributor

@jbedo jbedo commented Feb 15, 2018

Motivation for this change

Bioconductor was quite out of date because the old expression fetched bioconductor with the same version as R. This is not appropriate as bioconductor versions are not synchronised with R versions. I have updated the expression to explicitly fetch the latest bioconductor (3.6).

I have also updated the R packages as many of the bioconductor packages require the latest CRAN versions.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • [x ] NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@jbedo jbedo requested a review from peti as a code owner February 15, 2018 23:52
@grahamc
Copy link
Member

grahamc commented Feb 16, 2018

@GrahamcOfBorg eval

looks like the eval issue was handled.

@peti
Copy link
Member

peti commented Feb 16, 2018

Test builds are running at https://hydra.nixos.org/jobset/nixpkgs/r-updates.

@peti
Copy link
Member

peti commented Feb 16, 2018

Hmm, after applying this patch, https://hydra.nixos.org/jobset/nixpkgs/r-updates no longer evaluates and performs no builds. Something isn't right.

Copy link
Member

@peti peti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jbedo
Copy link
Contributor Author

jbedo commented Feb 17, 2018

Apologies for neglecting the last testing step. It now passes for me.

@peti
Copy link
Member

peti commented Feb 20, 2018

OK, the new version looks good! It's compiling at https://hydra.nixos.org/jobset/nixpkgs/r-updates right now.

@peti
Copy link
Member

peti commented Feb 20, 2018

The number of broken builds increased from 500 to 1400. A good chunk of those failures seem to be caused by this issue: https://hydra.nixos.org/build/69692858. Apparently, GenomicRanges_1.30.1.tar.gz is not available for download?

@jbedo
Copy link
Contributor Author

jbedo commented Feb 20, 2018

Looks like GenomicRanges had a version bump since I made the original pull request. Turns out old packages are not available at the url we use for bioc packages, which makes bioc quite fragile. I've addressed this by adding the archive url to the list of urls searched for packages, which should help prevent breakage whenever a bioc package is update. I've also re-updated all the packages that have changed since my original pull request.

@peti
Copy link
Member

peti commented Feb 21, 2018

I'm restarting the test builds with the new version ...

@peti peti self-assigned this Feb 21, 2018
@peti peti merged commit f457933 into NixOS:master Feb 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants