[Bioc-devel] Nightly builds

[Leonardo Collado Torres]

Hi,

I try to follow all the best practices but sometimes I end up relying
on the Bioc nightly build.

Anyhow, I personally use r-travis** for quick checks to detect broken
commits instead of running R CMD check/build on my laptop. This way I
can submit a new commit that fixes a broken commit before the Bioc
nightly build in about the same time as using my laptop. r-travis
doesn't always work as Bioc might have newer versions of dependencies
or other issues (cache, timing, installation of deps), but it helps
most of the time. A scenario where I definitely rely on the Bioc
nightly build is when I can't reproduce the error/warning on my
machine (with the latest setup). For example, Windows issues***.

Note that I do run a quick local test of whatever bug I'm solving, but
I might miss how my changes affected another unit test.

Best,
Leonardo

** Example setup
https://github.com/leekgroup/derfinderPlot/blob/master/.travis.yml
*** You can also use r-hub for testing in other OS including Windows
but maybe not with the R version that bioc-devel needs and/or without
being able to set useDevel(). Example
https://github.com/LieberInstitute/jaffelab/blob/master/.travis.yml#L17-L18
I've only used it for that non-bioc package.

On Wed, Feb 7, 2018 at 9:51 AM, Martin Morgan
<martin.morgan at roswellpark.org> wrote:
>
>
> Bioc developers!
>
> I've been exploring the Bioconductor nightly builds
>
>   http://bioconductor.org/checkResults/
>
> a bit using this in-development package.
>
>   https://github.com/mtmorgan/BiocBuildReports
>
> This
>
>   rpt <- report()
>   filter_recent(rpt) %>% print(n = n())
>
> summarizes the packages changed in the 24 hours before the current build snapshot, and their fate on the nightly builds
>
> > filter_recent(rpt) %>% print(n=nrow(.))
> # A tibble: 18 x 5
>    package           buildsrc checksrc push     txt
>    <chr>             <fct>    <fct>    <fct>    <chr>
>  1 AllelicImbalance  ERROR    skipped  skipped  skipped
>  2 ChIPpeakAnno      ERROR    skipped  skipped  skipped
>  3 GenomicScores     ERROR    skipped  skipped  skipped
>  4 genphen           ERROR    skipped  skipped  skipped
>  5 metavizr          ERROR    skipped  skipped  skipped
>  6 QUBIC             ERROR    skipped  skipped  skipped
>  7 Repitools         OK       ERROR    skipped  skipped
>  8 rtracklayer       OK       WARNINGS UNNEEDED same version exists in interna…
>  9 genoset           OK       WARNINGS YES      new version is higher than in …
> 10 IRanges           OK       WARNINGS YES      new version is higher than in …
> 11 LOLA              OK       WARNINGS YES      new version is higher than in …
> 12 DAPAR             OK       OK       UNNEEDED same version exists in interna…
> 13 AnnotationHubData OK       OK       YES      new version is higher than in …
> 14 BEARscc           OK       OK       YES      new version is higher than in …
> 15 ChemmineR         OK       OK       YES      new version is higher than in …
> 16 DESeq2            OK       OK       YES      new version is higher than in …
> 17 Onassis           OK       OK       YES      new version is higher than in …
> 18 synergyfinder     OK       OK       YES      new version is higher than in …
>
> A couple of things strike me about this.
>
>   - All the packages with 'ERROR' in the buildsrc or checksrc column failed to propagate because the commit, or a previous version, does not build or check. It is not a very productive development model to push broken commits to git.bioconductor.org, especially if the developer doesn't realize that the commits are broken. The best practice is to build and check packages locally, and then commit.
>
>   R CMD build <YourPackage>
>   R CMD check <YourPackage_<version>.tar.gz
>
> An even more robust approach is to clone the repository locally, so that the build and check are against only the committed changes and not miscellaneous files or versions hanging out in your source directory
>
>   cd /tmp
>   git clone /path/to/<YourPackage git>
>   R CMD build <YourPackage>
>   ...
>
> I would not suggest using devtools during this final stage; the build system doesn't use devtools, as devtools has its own ideas about options etc with which to build packages. The very careful would also use the build and check options found on the build report of each package, e.g., from
>
>
> http://bioconductor.org/checkResults/3.7/bioc-LATEST/rtracklayer/malbec2-buildsrc.html
>
> one might
>
>   R CMD build --keep-empty-dirs --no-resave-data rtracklayer
>
> A common problem is that packages are built and checked locally using the wrong version of R and / or Bioconductor -- commits to the devel branch should be against the devel version of Bioconductor and (for the current release cycle) the devel version of R.
>
>   http://bioconductor.org/developers/how-to/useDevel/
>
>   - two packages (rtracklayer, DAPAR) built and checked successfully, but did not propagate to the public repository. This is because they did not bump their version number according to policy
>
>   http://bioconductor.org/developers/how-to/version-numbering/
>
> While there might be reasons for pushing commits to Biocoductor without version bumps, this usually seems to be an oversight on the developer's part. Basically, every public version of a package should have a different version -- there is no shortage of version numbers.
>
> Martin
>
>
> This email message may contain legally privileged and/or...
>
> _______________________________________________
> Bioc-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/bioc-devel
>