-
Notifications
You must be signed in to change notification settings - Fork 723
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
RVM claims it can't find curl when using node-which; Node.js builds contain ./node_modules/bin
in $PATH
#5092
Comments
The error has mysteriously stopped occurring. |
It's still happening: https://travis-ci.org/twbs/bootstrap/jobs/90976058 |
@BanzaiMan Help? |
This is peculiar. My current guess is that the instance used in the failing builds were somehow lacking |
Okay, it gets even weirder. curl is installed on the system, despite what RVM claims.
|
Still repros even after upgrading to rvm 1.26.11: https://travis-ci.org/twbs/bootstrap/jobs/91533567 |
Filed rvm/rvm#3559 since the evidence points towards a possible bug in RVM. |
This still happens. If I clear the cache it works though. |
Example, failing https://travis-ci.org/mpc-hc/mpc-hc.org/builds/93137684 Same build, it's just the cache is cleared. |
@XhmikosR Does the build reliably fail if you keep the cache? |
Yeah it would seem so.
|
I noticed the same thing with bootstrap too BTW.
|
What bootstrap? |
The original post is for Bootstrap but I notice the same failures in my other projects too. |
Until travis-ci/travis-ci#5092 gets resolved. A slow build is better than a broken build. Refs travis-ci/travis-ci#5092 (comment) [skip sauce]
Judging by https://travis-ci.org/twbs/bootstrap/builds/93292414 , it seems that the problem occurs even if the caching is only of |
So that the build will stop failing due to travis-ci/travis-ci#5092
Actually, this occurs whether caching is on or not. |
Some npm modules define commands with widely used bash functions (and probably others). Putting this component in `$PATH` causes subtle issues, such as travis-ci/travis-ci#5092. We are removing it, so that this is no longer the case. This will most likely break some builds, so we add a warning.
Hello. Our attempt to remove This issue is going to cause pain in non-negligible number of users either way, unfortunately. |
@BanzaiMan It broke many existing builds (based on this bug too) on the other side too, so at least assume the correct default and let people fix it (I personally lost various hours due to unpredicted behavior). What about providing at least a configuration option to disable this behavior? That would be enough, instead of having to hack around with `, |
See also: #4862. |
This issue is really long, the tl;dr; is:
There's a way to solve it for you: before_cache:
- rm -f ./node_modules/.bin/which # <-- workaround for https://github.com/travis-ci/travis- |
Wouldn't it be possible to add a setting like:
(or something similar 😉) that would cause There's even |
That would be much appreciated indeed! |
Thanks for contributing to this issue. As it has been 90 days since the last activity, we are automatically closing the issue. This is often because the request was already solved in some way and it just wasn't updated or it's no longer applicable. If that's not the case, please do feel free to either reopen this issue or open a new one. We'll gladly take a look again! You can read more here: https://blog.travis-ci.com/2018-03-09-closing-old-issues |
This problem still exists, and is still as annoying as ever -- please keep it open :) |
Thanks for contributing to this issue. As it has been 90 days since the last activity, we are automatically closing the issue in 24 hours. This is often because the request was already solved in some way and it just wasn't updated or it's no longer applicable. If that's not the case, please do feel free to either reopen this issue or open a new one. We'll gladly take a look again! You can read more here: https://blog.travis-ci.com/2018-03-09-closing-old-issues |
Bot: Can you leave this open and linked to travis-ci/travis-build#1263 please? :) |
./node_modules/bin
in $PATH
As workaround for this bug I'm doing this in the .travis.yml :
|
This literally stopped working just a few minutes ago.
Obviously, falling back to compiling Ruby from source should be avoided if at all possible.
The text was updated successfully, but these errors were encountered: