BayesianTools icon indicating copy to clipboard operation
BayesianTools copied to clipboard

TravisCI requires libnlopt-dev

Open TankredO opened this issue 7 years ago • 8 comments

TravisCI in the master branch (devel build) is failing with the warning:

Found file(s) with version 3 serialization:
  ‘build/vignette.rds’
Such files are only readable in R >= 3.5.0.
Recreate them with R < 3.5.0 or save(version = 2) or saveRDS(version = 2) as appropriate

I don't know if something needs to be adjusted in the build process of TravisCI, or if we need to change something on the package level.

TankredO avatar Mar 14 '18 13:03 TankredO

There is some discussion about this in

  • http://r.789695.n4.nabble.com/R-CMD-build-then-check-fails-on-R-devel-due-to-serialization-version-td4747582.html
  • https://github.com/yihui/knitr/issues/1383

can you try to figure out what's the problem?

florianhartig avatar Mar 14 '18 18:03 florianhartig

Seems the issue is only in the old rel - given what I read, that's a bit weird, one would expect a problem in devel rather than in oldrel

florianhartig avatar Mar 14 '18 18:03 florianhartig

The problem is in devel (https://travis-ci.org/florianhartig/BayesianTools/builds/327189876), but there is another problem in the development branch in oldrel (https://github.com/florianhartig/BayesianTools/issues/108). I will try to fix this.

TankredO avatar Mar 15 '18 11:03 TankredO

https://stat.ethz.ch/pipermail/r-devel/2018-January/075359.html

TankredO avatar Mar 16 '18 08:03 TankredO

OK, I merged back develop into master 0.1.4.1 with https://github.com/florianhartig/BayesianTools/commit/8e12df39d01b875f4833e8bc247ed0607d8ce9d8, this should also fix the master problem.

Should we keep this open until the check problems are solved in the packages / CRAN and then revert the changes made here?

florianhartig avatar Mar 18 '18 13:03 florianhartig

I don't understand why this fixed the error. Nothing was changed that influences the build process (except buildignore of the pkgdown .yml). Do you have an idea why it works now?

TankredO avatar Mar 19 '18 08:03 TankredO

ae, it worked in dev, I just merged in dev to master. no idea why dev worked in the first place, but it worked since a few days, right, since 189 to be exact if you look here https://travis-ci.org/florianhartig/BayesianTools/branches

florianhartig avatar Mar 19 '18 09:03 florianhartig

Again, I keep this ticket open as a reminder to check periodically if the TRAVIS / CRAN checks are fixed, so that we can remove this yaml options

florianhartig avatar Mar 21 '18 14:03 florianhartig