-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
src: avoid std::make_unique
for now
#23856
Conversation
This broke Travis CI. We have run into this problem with `std::make_unique` in the past, and opted to not use it for now, e.g. in nodejs#20386.
CI: https://ci.nodejs.org/job/node-test-pull-request/18124/ Feel free to 👍 this comment to approve fast-tracking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😭
AFAIR, in the past it was caused by AIX bot using unsupported (by Node.js) compiler. Is there a tracking bug for that issue?
I believe this should be fixed by using the minimal supported compiler, i.e. #23778. |
@refack If that PR is close to landing and addresses the issue, sure. It didn’t look that way from your most recents comments, though? |
#23778 landed. |
Okay, I guess we can close this then. |
This broke Travis CI. We have run into this problem with
std::make_unique
in the past, and opted to not use itfor now, e.g. in #20386.
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes