FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

LICENSE update from dependencies by rvagg · Pull Request #9351 · nodejs/node · GitHub

/ node Public

LICENSE update from dependencies - #9351

Closed
rvagg wants to merge 2 commits into
nodejs:masterfrom
rvagg:license-update
ConversationCommits (2)ChecksFiles changed
Closed

LICENSE update from dependencies#9351
rvagg wants to merge 2 commits into
nodejs:masterfrom
rvagg:license-update

Conversation

rvagg commented Oct 29, 2016

Copy link
Copy Markdown
Member

Changes in license-builder.sh and LICENSE resulting from that.

  • Update libuv license, they changed it upstream
  • Update v8_inspector license, their license file was removed from tree when it was reorganised, pulling from file header now
  • Reorder LICENSE file to put v8_inspector along with the runtime components and its build components down with the rest of the build components, this is how it should have been in the first place

nodejs-github-bot added meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. labels Oct 29, 2016

richardlau commented Oct 29, 2016

Copy link
Copy Markdown
Member

The v8_inspector license file was added to address #7123; Was its removal intended? @eugeneo

richardlau mentioned this pull request Feb 11, 2017
Closed
2 tasks

thefourtheye commented Feb 11, 2017

Copy link
Copy Markdown
Contributor

This needs a rebase now.

jasnell commented Mar 24, 2017

Copy link
Copy Markdown
Member

ping @rvagg

jasnell added the stalled Issues and PRs that are stalled. label Mar 24, 2017

fhinkel commented May 26, 2017

Copy link
Copy Markdown
Contributor

There hasn't been any activity here. I'm closing this. Feel free to reopen if I closed this in error.

fhinkel closed this May 26, 2017
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

meta Issues and PRs related to the general management of the project. stalled Issues and PRs that are stalled. tools Issues and PRs related to the tools directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

Back | FazBrowse Home | New Git URL