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

deps: upgrade npm in LTS to 2.14.21 in Node.js 0.12 LTS #5594

Closed
wants to merge 1 commit into from

Conversation

othiym23
Copy link
Contributor

@othiym23 othiym23 commented Mar 8, 2016

Pull Request check-list

Please make sure to review and check all of these items:

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with
    this change (including linting)? (see below)
  • Is the commit message formatted according to [CONTRIBUTING.md][0]?

Affected core subsystem(s)

  • deps

Description of change

Upgrades the v0.12 LTS branch to use npm@lts / npm@2.

Got one repeatable test failure, but am confident it's not related to this change:

/usr/bin/python tools/test.py --mode=release simple message
=== release test-chdir ===
Path: simple/test-chdir
fs.js:747
  return binding.mkdir(pathModule._makeLong(path),
                 ^
Error: EEXIST, file already exists '/Users/ogd/Documents/projects/npm/release-node/test/fixtures/weird 쎤쎫쎯 characters áâã'
    at Error (native)
    at Object.fs.mkdirSync (fs.js:747:18)
    at Object.<anonymous> (/Users/ogd/Documents/projects/npm/release-node/test/simple/test-chdir.js:34:4)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3
Command: out/Release/node /Users/ogd/Documents/projects/npm/release-node/test/simple/test-chdir.js
[00:19|%   5|+  42|-   1]: release test-child-process-send-utf8^C Interrupted
make: *** [test] Error 1

We've also confirmed that npm's tests run when run against Node 0.12 in Travis. It should also get a citgm run.

I tried to be conservative about what tags I applied, and apologize in advance if I misinterpreted the meanings of the ones that I set. This might also need lts-agenda, since I understand this will be discussed at this week's LTS meeting.

[email protected] includes the updated LICENSE which includes the removal of the potentially confusing web site and registry Terms of Use. v0.12 (also) doesn't have the license updating tool, so is there a best practice for updating the top-level LICENSE to reflect the changed npm license?

r: @thealphanerd
r: @Fishrock123

@othiym23 othiym23 added npm Issues and PRs related to the npm client dependency or the npm registry. land-on-v0.12 lts Issues and PRs related to Long Term Support releases. labels Mar 8, 2016
@othiym23 othiym23 changed the title deps: upgrade npm in LTS to 2.14.21 deps: upgrade npm in LTS to 2.14.21 in Node.js 0.12 LTS Mar 8, 2016
@othiym23
Copy link
Contributor Author

othiym23 commented Mar 8, 2016

(The test failure recurs if I manually clear out that file:

/usr/bin/python tools/test.py --mode=release simple message
=== release test-chdir ===
Path: simple/test-chdir
assert.js:86
  throw new assert.AssertionError({
        ^
AssertionError: false == true
    at Object.<anonymous> (/Users/ogd/Documents/projects/npm/release-node/test/simple/test-chdir.js:36:1)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3
Command: out/Release/node /Users/ogd/Documents/projects/npm/release-node/test/simple/test-chdir.js
[03:02|% 100|+ 814|-   1]: Done
make: *** [test] Error 1

)

@rvagg
Copy link
Member

rvagg commented Mar 8, 2016

leave the LICENSE thing with me for now

@MylesBorins
Copy link
Contributor

@MylesBorins
Copy link
Contributor

the results are in from citgm

OSX failed due to jenkins weirdness https://ci.nodejs.org/job/thealphanerd-smoker/108/nodes=osx1010/console

I'm going to test locally

@rvagg
Copy link
Member

rvagg commented Mar 8, 2016

jenkins weirdness should be resolved now

@MylesBorins
Copy link
Contributor

Latest run: https://ci.nodejs.org/job/thealphanerd-smoker/109/

edit: the moment failure we are seeing on ubuntu is being tracked up stream
moment/moment#3016

@mattjohnsonpint
Copy link

WRT the moment.js error, it's DST related, and should go away on its own by this Monday. We have a fix pending in a PR, but see no need to push a hotfix release for such a minor issue at this time.

@ChALkeR
Copy link
Member

ChALkeR commented Apr 11, 2016

Could this be closed now?

@MylesBorins
Copy link
Contributor

We have a newer version of npm in v0.12 now. closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lts Issues and PRs related to Long Term Support releases. npm Issues and PRs related to the npm client dependency or the npm registry.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants