-
Notifications
You must be signed in to change notification settings - Fork 47
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
sudo error during travis check #151
Comments
I'm using This was fixed by adding
Note that I've got This all seems to be related to Travis' container infrastructure, on which they don't allow sudo. |
Thanks for the reply. Sorry, 'require' was a typo. |
Maybe they're in the middle of some kind of migration... in any case, it seems like containers are going to be the new normal for Travis: http://docs.travis-ci.com/user/workers/container-based-infrastructure/ |
I wonder how this will impact travis for R packages. Currently it looks like R is freshly installed via apt-get every time a check is run. Ideally the right container should contain a fresh snapshot of R packages. @hlapp might have suggestions there? |
this is interesting -- it does seem to be the case that travis has started opting people into container-based builds. i just filed travis-ci/travis-ci#3521 to hopefully find out more. in the interim, it seems like |
so it looks like this is on the way for everyone. i'm going to keep this open to track updating docs everywhere. |
Good to know. ooks like the apt add-on should do the trick, so sudo should On Tue, Mar 31, 2015 at 5:43 PM, Craig Citro notifications@github.com
|
@thibautjombart actually, it won't quite get us there yet -- we have to pre-specify the list of packages we'd want to be available in that case. i don't know if it would be possible to get a handful of packages approved from a PPA (marutter's PPA), but i should look into it. |
Does look like this is repo-specific, as I still don't need |
@jread-usgs yep, thus far it's only for new repos as of feb 14. definitely a bit crazy-making. ;) |
Hi, |
@jangorecki the R support uses sudo for various things (eg installing R), and doesn't work out of the box with container-based builds. for new repos, they're being opted in to container-based builds by default, which causes trouble. older repos are using the VM infrastructure, which allows |
Thanks for explanation. If it is going to stay like that it would be worth to update all the examples. |
Looking at https://travis-ci.org/bocinsky/FedData/builds/61545774 it seems that you've got the same problem noted here craigcitro/r-travis#151 which should be fixed with this addition.
Travis seems to be failing with the basic file created by devtools::add_travis()
Just as of today, worked a few hours ago. See:
https://github.com/thibautjombart/apex
Apparently config file would need a
or
But neither worked for me.
Here's the log:
The text was updated successfully, but these errors were encountered: