Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

chakrashim: fix debug build failure #298

Merged
merged 2 commits into from
Jun 13, 2017
Merged

Conversation

kfarnung
Copy link
Contributor

Fixing debug build failure introduced in
df8e7ca

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

chakrashim

@kfarnung
Copy link
Contributor Author

@kfarnung
Copy link
Contributor Author

CI finished with failures in Ubuntu. Ported the fix from v8.x. That fix was picked up in the macOS leg and verified.

kfarnung added 2 commits June 13, 2017 13:46
Fixing debug build failure introduced in
df8e7ca

PR-URL: nodejs#298
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
* async-hooks/test-callback-error - depends on V8 option
  --abort-on-uncaught-exception

PR-URL: nodejs#298
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
@kfarnung kfarnung merged commit 76c7a22 into nodejs:master Jun 13, 2017
@kfarnung kfarnung deleted the debug_build branch June 13, 2017 21:41
kfarnung added a commit that referenced this pull request Jun 13, 2017
Fixing debug build failure introduced in
df8e7ca

PR-URL: #298
Reviewed-By: Kunal Pathak <kunal.pathak@microsoft.com>
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants