Skip to content
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

Build failure on SmartOS (6.2) #16

Closed
targos opened this issue Aug 16, 2017 · 7 comments
Closed

Build failure on SmartOS (6.2) #16

targos opened this issue Aug 16, 2017 · 7 comments

Comments

@targos
Copy link
Member

targos commented Aug 16, 2017

CI run: https://ci.nodejs.org/job/node-test-commit-smartos/10884/nodes=smartos15-64/console

Error:

  touch /home/iojs/build/workspace/node-test-commit-smartos/nodes/smartos15-64/out/Release/obj.target/node_dtrace_provider.stamp
dtrace: failed to compile script src/v8ustack.d: line 293: failed to resolve V8DBG_FRAMETYPE_ENTRYCONSTRUCTFRAME: Unknown variable name
node_dtrace_ustack.target.mk:26: recipe for target '/home/iojs/build/workspace/node-test-commit-smartos/nodes/smartos15-64/out/Release/obj.target/node/src/node_dtrace_ustack.o' failed
make[2]: *** [/home/iojs/build/workspace/node-test-commit-smartos/nodes/smartos15-64/out/Release/obj.target/node/src/node_dtrace_ustack.o] Error 1

@nodejs/platform-smartos

@cjihrig
Copy link

cjihrig commented Aug 16, 2017

I'll take a look.

@targos
Copy link
Member Author

targos commented Aug 28, 2017

@geek
Copy link
Member

geek commented Aug 28, 2017

@geek geek closed this as completed Aug 28, 2017
@targos
Copy link
Member Author

targos commented Aug 29, 2017

I'm reopening because the build is still failing with this commit and with the same error message: https://ci.nodejs.org/job/node-test-commit-smartos/11145/

@targos
Copy link
Member Author

targos commented Sep 1, 2017

Fixed by nodejs/node@50cc6c7

@targos targos closed this as completed Sep 1, 2017
@misterdjules
Copy link

@targos Just to make sure we're on the same page, some of the commits that fix the build issue on SmartOS are temporary fixes (like f6bd6af), and some of them may not have been through a standard review/testing process (like nodejs/node@50cc6c7).

@cjihrig, myself and others have started to take a closer look at fixing the DTrace and post-mortem debugging issues with recent V8 versions, and we should be able to come up with less temporary fixes soon.

@targos
Copy link
Member Author

targos commented Sep 1, 2017

@misterdjules all commits that are on canary-base but not on master can be considered temporary fixes. This branch only exists to collect fixes that the bot can cherry-pick to this repository for the canary updates/tests/nightly.

They will go through a standard review process when a PR is opened in the node repository to update V8 on master or they will be removed once the changes are upstreamed to V8 (if applicable).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants