quanteda / readtext

an R package for reading text files
https://readtext.quanteda.io
118 stars 28 forks source link

Fix broken Travis script #134

Closed kbenoit closed 6 years ago

kbenoit commented 6 years ago

I think this requires an adjustment to the .travis.yml., perhaps based on https://github.com/quanteda/readtext/pull/133.

conjugateprior commented 6 years ago

I'd be interested in a fix for this one too, since I'd like to be able to use have readtext as a dependency. For me it's an ndjson installation failure (below) that stops streamR installation which stops readtext installation. (Paul's two apt-get fixes are already in).

'/home/travis/R-bin/lib/R/bin/R' --no-site-file --no-environ --no-save  \
  --no-restore --quiet CMD INSTALL  \
  '/tmp/Rtmp3woujk/devtools387c2fb58294/ndjson'  \
  --library='/home/travis/R/Library' --install-tests 

* installing *source* package ‘ndjson’ ...
** package ‘ndjson’ successfully unpacked and MD5 sums checked
** libs
g++ -std=gnu++11 -I"/home/travis/R-bin/lib/R/include" -DNDEBUG  -I"/home/travis/R/Library/Rcpp/include" -I/home/travis/R-bin/include   -fpic  -g -O2 -c RcppExports.cpp -o RcppExports.o
g++ -std=gnu++11 -I"/home/travis/R-bin/lib/R/include" -DNDEBUG  -I"/home/travis/R/Library/Rcpp/include" -I/home/travis/R-bin/include   -fpic  -g -O2 -c gzstream.cpp -o gzstream.o
g++ -std=gnu++11 -I"/home/travis/R-bin/lib/R/include" -DNDEBUG  -I"/home/travis/R/Library/Rcpp/include" -I/home/travis/R-bin/include   -fpic  -g -O2 -c ndjson.cpp -o ndjson.o
In file included from ndjson.cpp:8:0:
json.h:127:10: error: #error "unsupported GCC version - see https://github.com/nlohmann/json#supported-compilers"
         #error "unsupported GCC version - see https://github.com/nlohmann/json#supported-compilers"
          ^
make: *** [ndjson.o] Error 1
ERROR: compilation failed for package ‘ndjson’
* removing ‘/home/travis/R/Library/ndjson’
conjugateprior commented 6 years ago

For reference: my .travis

language: R
addons:
  apt:
    packages:
      - libxml2
      - libxml2-dev
      - libapparmor-dev
      - libpoppler-cpp-dev
sudo: false
cache: packages
conjugateprior commented 6 years ago

Some ndjson-specific advice from a while back here: https://stackoverflow.com/questions/39728709/using-an-alternate-compiler-for-travis-ci-r-project-builds (first answer from the ndjson author). Might be a useful place to start?