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 to npm 2.14.15 #4872

Closed
wants to merge 3 commits into from
Closed

Conversation

zkat
Copy link
Contributor

@zkat zkat commented Jan 26, 2016

Hi! This PR includes three releases!

Relevant changes:

  • Lots of doc updates/fixes
  • 6051a69 #10685 Fix npm public registry URL in notices.
  • #10629 Node can be compiled FIPS-compliant now due to MD5 extraction (it was previously used for non-security purposes)
  • nodejs/node-gyp#831 New node-gyp with better *BSD support
  • #10532 Clarify that registry.npmjs.org is the default, but that you're free to use the npm CLI with whatever registry you wish. This goes along with previous licensing-related improvements, too.
  • 333e118 Better AIX support for node-gyp

Those are the highlights! And to repeat: This does include some of the previously-talked-about licensing changes.

r: @Fishrock123
r: @iarna

@zkat zkat added npm Issues and PRs related to the npm client dependency or the npm registry. land-on-v4.x labels Jan 26, 2016
@MylesBorins
Copy link
Contributor

@MylesBorins
Copy link
Contributor

will manually test tomorrow

@zkat
Copy link
Contributor Author

zkat commented Jan 26, 2016

I may be able to slurp a fix into this that includes the relevant CI fixes -- but this still needs that quick pre-test patch to make it stop being a prerelease :(

@MylesBorins
Copy link
Contributor

@zkat the CI failures appear to be unrelated... looks like windows fanned was down last night, and the single failure on arm is also a CI failure not code. I'm going to re run those two systems again

ARM: https://ci.nodejs.org/job/node-test-commit-arm/1931/

WINDOWS: https://ci.nodejs.org/job/node-test-commit-windows-fanned/1115/

@MylesBorins
Copy link
Contributor

Failure on PPC for citgm is expected

@Fishrock123
Copy link
Member

@thealphanerd Keep in mind node-test-commit is irrelevant for npm since it doesn't actually test that code. :)

@MylesBorins
Copy link
Contributor

Oh yeah... Womp.

At least citgm tests a bunch of npm functionality
On Jan 26, 2016 11:53 AM, "Jeremiah Senkpiel" notifications@github.com
wrote:

@thealphanerd https://github.com/TheAlphaNerd Keep in mind
node-test-commit is irrelevant for npm since it doesn't actually test
that code. :)


Reply to this email directly or view it on GitHub
#4872 (comment).

@zkat
Copy link
Contributor Author

zkat commented Jan 26, 2016

Also our test suite still isn't passing on Windows. That'll stay red until we're done with our current project.

@joaocgreis
Copy link
Member

I'm trying to get the Windows CI stable, and noticed this failing.

C:\workspace\node-test-binary-windows\RUN_SUBSET\0\VS_VERSION\vs2013\label\win2008r2>git fetch https://github.com/nodejs/node.git +refs/pull/4872/head:refs/remotes/origin/_t

C:\workspace\node-test-binary-windows\RUN_SUBSET\0\VS_VERSION\vs2013\label\win2008r2>git checkout refs/remotes/origin/_t~1
warning: unable to access 'deps/npm/node_modules/node-gyp/node_modules/npmlog/node_modules/gauge/node_modules/lodash.padright/node_modules/lodash._createpadding/node_modules/lodash.repeat/.gitattributes': Filename too long
Checking out files: 100% (9189/9189), done.
Previous HEAD position was af7313c... added binaries
HEAD is now at 39eeca6... test: add test for debugging one line files

C:\workspace\node-test-binary-windows\RUN_SUBSET\0\VS_VERSION\vs2013\label\win2008r2>git checkout refs/remotes/origin/_t
warning: unable to access 'deps/npm/node_modules/node-gyp/node_modules/npmlog/node_modules/gauge/node_modules/lodash.padright/node_modules/lodash._createpadding/node_modules/lodash.repeat/.gitattributes': Filename too long
fatal: cannot create directory at 'deps/npm/node_modules/npm-install-checks/node_modules/npmlog/node_modules/gauge/node_modules/lodash.pad/node_modules/lodash._createpadding/node_modules/lodash.repeat': Filename too long

So, C:\workspace\node-test-binary-windows\RUN_SUBSET\0\VS_VERSION\vs2013\label\win2008r2\'deps/npm/node_modules/npm-install-checks/node_modules/npmlog/node_modules/gauge/node_modules/lodash.pad/node_modules/lodash._createpadding/node_modules/lodash.repeat is the problem... Is there a way to make this filename smaller? I could try to make the test directory name smaller in CI, but not without breaking the parallel tests in Windows, and that would make CI noticeably slower.

@Fishrock123
Copy link
Member

@joaocgreis Maybe we should run npm3's dedupe on the npm2 source tree for this, that might help?

@joaocgreis
Copy link
Member

@Fishrock123 if that works and makes the tree less deep, it should solve this.

@Fishrock123
Copy link
Member

Usual failures with pre-release builds. None with a "release" build. LGTM.

@zkat
Copy link
Contributor Author

zkat commented Jan 26, 2016

Please don't flatten things -- last we checked, it won't work, and this is all part of the work we're doing right now to get Windows CI stable for npm. It is, in fact, our main focus right now, so you should be seeing a green Windows soon enough.

I've pushed a small patch that -should- get things working on non-windows CI again (it worked locally, so here's hoping! fingers crossed).

@richardlau
Copy link
Member

@zkat If npm's LICENSE has been updated please could you keep Node's LICENSE in step? I think this will have to be done manually as the license-builder tool hasn't been backported to v4.x.

@orangemocha
Copy link
Contributor

@zkat any reason not to include 2.14.16 in this PR? It has a couple of important Windows fixes and we were hoping to get it merged into Node soon.

@iarna
Copy link
Member

iarna commented Jan 27, 2016

@orangemocha Not to speak for @zkat, but 2.14.16 hasn't spent it's week in pre-release testing (as next-2) yet. It'd be eligible tomorrow though.

@jasnell
Copy link
Member

jasnell commented Jan 27, 2016

LGTM on cursory scan. At least nothing significant caught my eye.

@zkat
Copy link
Contributor Author

zkat commented Jan 27, 2016

@richardlau Ahh! Thanks for the heads up. Change is pushed.

@orangemocha As @iarna said -- we have delayed releases. I'll be putting up another PR for 2.14.16 on Friday or Monday, most likely. :)

I'm gonna go ahead and merge this once the LICENSE thing gets a +1

@iarna
Copy link
Member

iarna commented Jan 27, 2016

LGTM

@joaocgreis
Copy link
Member

joaocgreis commented Jan 27, 2016 via email

This was referenced Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

9 participants