-
Notifications
You must be signed in to change notification settings - Fork 166
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
Assembler missing on smartos 16 machines? #1327
Comments
Re-running playbook on machines, hopefully may fix |
SmartOS rebuild: https://ci.nodejs.org/job/node-test-commit-smartos/18225/ |
Looking good so far. Thanks! |
Rebuild at https://ci.nodejs.org/job/node-test-commit-smartos/18225/ looks to be passing, going to close this out for now. Please reopen if happens again |
It's back.... |
https://ci.nodejs.org/job/node-test-commit-smartos/18227/nodes=smartos16-64/console 12:53:02 ERROR: Did not find a new enough assembler, install one or build with
12:53:02 --openssl-no-asm.
12:53:02 Please refer to BUILDING.md |
Is the right thing to do this?
Or something else? I'm not sure if that's the playbook to rebuild in this case. (Doesn't work for me in any event, but I'll try to figure out why that playbook doesn't work when run from my machine if it's the right one to run.) |
Seems like |
Nope, didn't fix it. |
I did The problem appears to be that |
I was stepping through the Python code and this started working. I hope someone fixed it... because otherwise... this is mysterious, in a bad way. :-D |
Only login besides me is from Cambridge, Massachussetts. Sounds like @refack fixed it? |
Sorry I'm trying to fix it, but getting weird stuff |
As in I have no idea why it sometimes work, and sometimes not. |
Well it is nodejs/node#20394. |
Interesting. Any idea what those situations are? |
Looks like it works with setting |
PR-URL: #21220 Fixes: nodejs/build#1327 Refs: #20394 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
PR-URL: #21220 Fixes: nodejs/build#1327 Refs: #20394 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
node-test-commit-smartos
is consistently failing onsmartos16-64
with this message:The text was updated successfully, but these errors were encountered: