-
Notifications
You must be signed in to change notification settings - Fork 66
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
inspector: bring in inspector_protocol as a direct dependency #76
inspector: bring in inspector_protocol as a direct dependency #76
Commits on Jul 11, 2018
-
Configuration menu - View commit details
-
Copy full SHA for d57325b - Browse repository at this point
Copy the full SHA d57325bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 456581b - Browse repository at this point
Copy the full SHA 456581bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8fcc375 - Browse repository at this point
Copy the full SHA 8fcc375View commit details -
Enhancements to --build-v8-with-gn
* Split "gn gen" and "ninja" invocations into two separate actions * Make the ninja action use console pool so that progress is shown * Set DEPOT_TOOLS_WIN_TOOLCHAIN=0 by default (to avoid non-googlers being asked for credentials) * Propagate depot_tools path to "gn gen" step (it runs vs_toolchain.py) * Add flags for clang-cl, setting gn args, sub-ninja -j and -l parameters
Configuration menu - View commit details
-
Copy full SHA for 8076a08 - Browse repository at this point
Copy the full SHA 8076a08View commit details -
* Fixed several hardcoded paths to use <(STATIC_LIB_PREFIX) & <(STATIC_LIB_SUFFIX) * Added GENERATOR=='ninja' conditionals to /WHOLEARCHIVE flags (msbuild puts .lib files under separate directories). * Add missing link_settings for v8_monolith * Set use_sysroot=false only for non-windows builds. To build using clang-cl: python configure --build-v8-with-gn --use-clang-cl --ninja
Configuration menu - View commit details
-
Copy full SHA for 49c10f5 - Browse repository at this point
Copy the full SHA 49c10f5View commit details -
Properly convert .gypi -> .json in js2c.py
It was breaking when .gypi strings had quotes in them. e.g.: 'foo': 'bar="baz"'
Configuration menu - View commit details
-
Copy full SHA for 124650a - Browse repository at this point
Copy the full SHA 124650aView commit details -
It needed to be passed via "make_global_settings" rather than environment variable.
Configuration menu - View commit details
-
Copy full SHA for fd66b85 - Browse repository at this point
Copy the full SHA fd66b85View commit details -
Configuration menu - View commit details
-
Copy full SHA for 05d782b - Browse repository at this point
Copy the full SHA 05d782bView commit details -
tools: properly convert .gypi in install.py
It was breaking during install when .gypi strings had quotes in them. e.g.: 'foo': 'bar="baz"'
Configuration menu - View commit details
-
Copy full SHA for 570fd7e - Browse repository at this point
Copy the full SHA 570fd7eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 427afa4 - Browse repository at this point
Copy the full SHA 427afa4View commit details -
Configuration menu - View commit details
-
Copy full SHA for a2f9a4c - Browse repository at this point
Copy the full SHA a2f9a4cView commit details -
src: use modern v8::Platform worker threads APIs
Precursor to removing deprecated APIs on the v8 side @ https://chromium-review.googlesource.com/c/v8/v8/+/1045310
Configuration menu - View commit details
-
Copy full SHA for 0c5c2c0 - Browse repository at this point
Copy the full SHA 0c5c2c0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 625d843 - Browse repository at this point
Copy the full SHA 625d843View commit details -
v8: Prepare for swallowed rejection hook.
This is done in preparation for landing https://chromium-review.googlesource.com/c/v8/v8/+/1126099 on the V8 side, which extends the existing PromiseRejectEvent mechanism with new hooks for reject/resolve after a Promise was previously resolved already. Refs: nodejs/promises-debugging#8 Design: https://goo.gl/2stLUY
Configuration menu - View commit details
-
Copy full SHA for d11dafa - Browse repository at this point
Copy the full SHA d11dafaView commit details -
Configuration menu - View commit details
-
Copy full SHA for eab77c7 - Browse repository at this point
Copy the full SHA eab77c7View commit details -
Configuration menu - View commit details
-
Copy full SHA for a2d3495 - Browse repository at this point
Copy the full SHA a2d3495View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0316fdf - Browse repository at this point
Copy the full SHA 0316fdfView commit details -
test: fix scriptParsed event expectations
As per Node.js docs, vm.Script instance is not bound to any context. However, this test was expecting otherwise and depended on implementation details which are going to change. Refs: https://chromium-review.googlesource.com/c/v8/v8/+/1013581
Configuration menu - View commit details
-
Copy full SHA for e3636fb - Browse repository at this point
Copy the full SHA e3636fbView commit details -
src: update NODE_MODULE_VERSION to 66
Major V8 updates are usually API/ABI incompatible with previous versions. This commit adapts NODE_MODULE_VERSION for V8 6.9. Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
Configuration menu - View commit details
-
Copy full SHA for c4f0452 - Browse repository at this point
Copy the full SHA c4f0452View commit details -
test: fix inspector tests after V8 upgrade
V8 improved break locations for the node --inspect-brk -e case.
Configuration menu - View commit details
-
Copy full SHA for d5a76af - Browse repository at this point
Copy the full SHA d5a76afView commit details -
Configuration menu - View commit details
-
Copy full SHA for 820faa8 - Browse repository at this point
Copy the full SHA 820faa8View commit details -
Configuration menu - View commit details
-
Copy full SHA for c59d247 - Browse repository at this point
Copy the full SHA c59d247View commit details
Commits on Jul 12, 2018
-
tools: Remove lldbinit file from install script
In v8 we want to remove the lldbinit file (https://crrev.com/c/1127892). In order to not brake node.js, we need to remove it here first.
Configuration menu - View commit details
-
Copy full SHA for 43906ca - Browse repository at this point
Copy the full SHA 43906caView commit details
Commits on Jul 13, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 7571231 - Browse repository at this point
Copy the full SHA 7571231View commit details
Commits on Jul 17, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 14e8565 - Browse repository at this point
Copy the full SHA 14e8565View commit details
Commits on Jul 18, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 94831af - Browse repository at this point
Copy the full SHA 94831afView commit details -
Configuration menu - View commit details
-
Copy full SHA for b450796 - Browse repository at this point
Copy the full SHA b450796View commit details
Commits on Jul 25, 2018
-
Bring in inspector_protocol as a direct dependency
This patch adds [inspector_protocol](https://www.google.com/url?q=https://chromium.googlesource.com/deps/inspector_protocol/) as a direct dependency to node.js. Inspector Protocol is used in node.js tracing, and this is required to be able to roll protocol independently from v8. inspector_protocol dependency comes with two more libs: - jinja2 - markupsafe
Configuration menu - View commit details
-
Copy full SHA for c9b5b9e - Browse repository at this point
Copy the full SHA c9b5b9eView commit details