-
Notifications
You must be signed in to change notification settings - Fork 80
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
MacOS install compatibility. #5
Comments
@ganezasan Created a proof - https://circleci.com/gh/ganezasan/circleci-sandbox/2627 @iynere Could we discuss how you would like this resolved in a "standardized" way? |
@KyleTryon actually, this was already on my plate for today |
@KyleTryon @ganezasan you can take a look at the (misleadingly named) https://github.com/CircleCI-Public/node-orb/tree/test-against-oracle |
#6 closes this |
@iynere after updating the orb, I'm having the following error when running
|
@JCMais did you try running it with |
@JCMais this is a separate issue so please open a new issue with further information, if you could |
Orb version
What happened
@ganezasan has found that the Yarn install incorrectly installs under MacOS due to including an incorrect PATH (for MacOS).
https://github.com/CircleCI-Public/node-orb/blob/master/src/commands/install-yarn.yml#L21
First reported here: https://discuss.circleci.com/t/having-fork-resource-temporarily-unavailable-when-running-commands-on-macos/30966/4?u=jcm
The incorrect path leads to slow processes on MacOS and errors similar to the following:
^D^D/tmp/.bash_env-5d10d6f32cffde0008c4dc34-0-build: fork: Resource temporarily unavailable
Expected behavior
Detect OS and change install path (@ganezasan has mentioned removing this line would work).
The text was updated successfully, but these errors were encountered: