Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nodejs/node
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: ea725eda426add1d684c80f2c2c585cd2ca797c9
Choose a base ref
...
head repository: nodejs/node
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 2554549ab5410745c4312a3b039e31e1620e05ab
Choose a head ref
  • 5 commits
  • 257 files changed
  • 1 contributor

Commits on Jul 27, 2016

  1. deps: update v8_inspector 

    To add a LICENSE file along with the v8_inspector code, we need to
pick up v8_inspector from an intermediate repository:
https://github.com/pavelfeldman/v8_inspector. This repo still tracks
upstream code in Blink.

This roll also picks up the latest v8_inspector from upstream fixing
a few issues.

* Pickup commit id bc60957 from pavelfeldman/v8_inspector
* Update node.gyp to adapt to the new file paths
* Update the DevTools hash for the devtools frontend.

Fixes: #7123
Fixes: #7736
Fixes: #7734
PR-URL: #7796
Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
    @ofrobots
    ofrobots committed Jul 27, 2016
    Configuration menu
    Copy the full SHA
    b9445bf View commit details
    Browse the repository at this point in the history

  2. deps: remove jinja.el from deps/v8_inspector 

    jinja.el is not shipped as part of the distribution, and neither is
it used in the build. While not strictly necessary, removing it
simplifies life from a licensing point of view.

PR-URL: #7796
Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
    @ofrobots
    ofrobots committed Jul 27, 2016
    Configuration menu
    Copy the full SHA
    4be6271 View commit details
    Browse the repository at this point in the history

  3. tools: update license-builder.sh for v8_inspector 

    Start including the license from v8_inspector and its build time
dependencies: jinja2 and markupsafe.

PR-URL: #7796
Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
    @ofrobots
    ofrobots committed Jul 27, 2016
    Configuration menu
    Copy the full SHA
    3534600 View commit details
    Browse the repository at this point in the history

  4. doc: update licenses 

    Created using `tools/license-builder.sh`.

PR-URL: #7796
Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
    @ofrobots
    ofrobots committed Jul 27, 2016
    Configuration menu
    Copy the full SHA
    7db495a View commit details
    Browse the repository at this point in the history

  5. deps: v8_inspector: remove jinja2 tests 

    PR-URL: #7796
Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
    @ofrobots
    ofrobots committed Jul 27, 2016
    Configuration menu
    Copy the full SHA
    2554549 View commit details
    Browse the repository at this point in the history
Loading