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

Travis CI: migration to Bionic with GDAL 3 + PROJ 7 #93

Closed
wants to merge 8 commits into from

Conversation

florisvdh
Copy link
Member

@florisvdh florisvdh commented Oct 8, 2020

This builds further on #89 (comment). Roughly the following steps were made:

  • use Ubuntu Bionic (18.04) instead of Xenial (16.04)
  • add the ubuntugis-unstable PPA to get the more recent versions of GDAL, PROJ and GEOS. This is the main reason for migrating to at least Bionic: it is recommended that rgdal, sp and sf use GDAL 3 and PROJ >= 6. If r.rb: use ubuntugis-unstable PPA travis-ci/travis-build#1962 would be accepted, then the 'unstable' PPA would become the default (hence not needed anymore in the yml file here).
    • update: the referred PR has not been included in Travis, so adding unstable PPA remains necessary.
  • add debian package libfribidi-dev, necessary to compile the package dependency textshaping of pkgdown

Default on Travis is Ubuntu 16.04 (Xenial). Given the somewhat flaky support of rgdal for GDAL<3 & PROJ <6 (see ), an easy fix is to upgrade GDAL & PROJ. They are at version 3 and 6 on the Ubuntu Focal repositories, so that should suffice. If upgrading to 20.04 works for r-language on travis!!
It seemed Focal was not supported; automatically uses Xenial. Let's try bionic!
This is a more logical way of adding PPA and deb packages.
It is done before cloning the github repo and before
loading the build cache.

Its effect can be seen in https://travis-ci.com/github/inbo/inborutils/jobs/396969884#L789-L808.

This was done with help of Jeroen Ooms, see travis-ci/travis-build#1962.
It seems this package was already included in the Travis
image.
I drew wrong conclusion from it being missing in my local
Focal installation.
@florisvdh
Copy link
Member Author

florisvdh commented Feb 5, 2021

Was implemented in aa75371.

@florisvdh florisvdh closed this Feb 5, 2021
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

Successfully merging this pull request may close these issues.

1 participant