node 6.0.0

[xeoneux]

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same formula update/change?
• Have you built your formula locally prior to submission with brew install <formula> (where <formula> is the name of the formula you're submitting)?
• Does your submission pass brew audit --strict --online <formula> (after doing brew install <formula>)?

Description

Update node to v6.0.0

[dunn]

Just remove this block entirely if there's no new development release.

[dunn]

The mirror needs to be updated too.

[xeoneux]

@dunn MACOSX_DEPLOYMENT_TARGET changed to 10.7. Does it affect the version requirement?

[xeoneux]

All tests pass except azure-cli due to changes in Node.js v6 API. Issue over at Azure/azure-xplat-cli.

[dunn]

Thanks for opening the Azure PR! Hopefully they'll merge it quickly.

[chrmoritz]

The ICU 57 upgrade didn't made it into node v6.0.0, so it should be reverted from this PR (as it's not supported).

You also have to add a revision to the azure-cli formula too, so that the native addon part of it can be recompiled against v8 5.0 shiped with node6.
Edit: And of cause we need either a floating patch with the node 6 fix too (or wait for a new release if a quick release with a fix for it is planned).

The MACOSX_DEPLOYMENT_TARGET change was made to allow a libuv upgrade to v2.0 to be semver minor. Until this upgrade lands later in a semver minor v6.x release node 6 will still be compatible with older OS X versions.

[jbergstroem]

How about introducing support for building against a shared c-ares?

[chrmoritz]

@jbergstroem: Is this comment nodejs/node#5775 (comment) still valid? If not adding an option to build against a shared c-ares shouldn't be a problem.

[jbergstroem]

@chrmoritz yes, I've landed shared support against c-ares. Additionally, if you want, building against a shared zlib and http-parser should work fine as well.

[xeoneux]

Azure/azure-xplat-cli#2821 got merged into base branch. Are we waiting for a release?

[xeoneux]

Also, check #638 for my motivation to keep npm packages out of homebrew.

[DomT4]

Azure/azure-xplat-cli#2821 got merged into base branch. Are we waiting for a release?

No need to wait. Can backport the patch with a link to the upstream commit and explanation it can be removed on next stable release, or simply use the commit diff as a patch do if it applies cleanly with the older release.

[LinusU]

@DomT4 Can we merge this now then?

[DomT4]

Seems reasonable. I'll deal with the azure-cli issue shortly after merging.

Merged in b8b57c7. Thank you for your contribution to Homebrew @xeoneux; we appreciate it! 😺

[LinusU]

Awesome 🎉