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: update libuv to version 1.7.4 #2817

Closed
wants to merge 1 commit into from
Closed

Conversation

saghul
Copy link
Member

@saghul saghul commented Sep 11, 2015

No description provided.

@saghul
Copy link
Member Author

saghul commented Sep 11, 2015

@mscdex mscdex added the libuv Issues and PRs related to the libuv dependency or the uv binding. label Sep 11, 2015
@cjihrig
Copy link
Contributor

cjihrig commented Sep 11, 2015

Rubber stamp LGTM. So far the CI seems thrilled.

bnoordhuis pushed a commit that referenced this pull request Sep 11, 2015
PR-URL: #2817
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@bnoordhuis
Copy link
Member

Landed in 68dc69a, thanks!

@rvagg
Copy link
Member

rvagg commented Sep 12, 2015

Highlights for Node in this one? I don't see anything obvious here, particularly given we're not using rwlock any more. It'd be useful to know if there's anything we can put in the next release notes about this.

rvagg pushed a commit that referenced this pull request Sep 12, 2015
PR-URL: #2817
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@rvagg rvagg mentioned this pull request Sep 12, 2015
@bnoordhuis
Copy link
Member

@rvagg Better uptime reporting on FreeBSD and a single CR is no longer converted to CRLF on Windows.

@rvagg
Copy link
Member

rvagg commented Sep 12, 2015

a single CR is no longer converted to CRLF on Windows

in what context? sounds like a breaking change if broadly applied.

@bnoordhuis
Copy link
Member

In the context of writing to the console.

@Fishrock123 Fishrock123 mentioned this pull request Sep 13, 2015
7 tasks
Fishrock123 pushed a commit that referenced this pull request Sep 15, 2015
PR-URL: #2817
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
rvagg added a commit that referenced this pull request Sep 15, 2015
* buffer: Fixed a minor errors that was causing crashes
  (Michaël Zasso) #2635,
* child_process: Fix error that was causing crashes (Evan Lucas) #2727
* crypto: Replace use of rwlocks, unsafe on Windows XP / 2003
  (Ben Noordhuis) #2723
* libuv: Upgrade from 1.7.3 to 1.7.4 (Saúl Ibarra Corretgé) #2817
* node: Fix faulty process.release.libUrl on Windows (Rod Vagg) #2699
* node-gyp: Float v3.0.3 which has improved support for Node.js and
  io.js v0.10 to v4+ (Rod Vagg) #2700
* npm: Upgrade to version 2.14.3 from 2.13.3, includes a security
  update, see https://github.com/npm/npm/releases/tag/v2.14.2 for more
  details, (Kat Marchán) #2696.
* timers: Improved timer performance from porting the 0.12
  implementation, plus minor fixes (Jeremiah Senkpiel) #2540,
  (Julien Gilli) nodejs/node-v0.x-archive#8751 nodejs/node-v0.x-archive#8905

PR-URL: #2698
rvagg added a commit that referenced this pull request Sep 15, 2015
* buffer: Fixed a minor errors that was causing crashes
  (Michaël Zasso) #2635,
* child_process: Fix error that was causing crashes (Evan Lucas) #2727
* crypto: Replace use of rwlocks, unsafe on Windows XP / 2003
  (Ben Noordhuis) #2723
* libuv: Upgrade from 1.7.3 to 1.7.4 (Saúl Ibarra Corretgé) #2817
* node: Fix faulty process.release.libUrl on Windows (Rod Vagg) #2699
* node-gyp: Float v3.0.3 which has improved support for Node.js and
  io.js v0.10 to v4+ (Rod Vagg) #2700
* npm: Upgrade to version 2.14.3 from 2.13.3, includes a security
  update, see https://github.com/npm/npm/releases/tag/v2.14.2 for more
  details, (Kat Marchán) #2696.
* timers: Improved timer performance from porting the 0.12
  implementation, plus minor fixes (Jeremiah Senkpiel) #2540,
  (Julien Gilli) nodejs/node-v0.x-archive#8751 nodejs/node-v0.x-archive#8905

PR-URL: #2698
@saghul saghul deleted the uv_upgrade branch September 22, 2015 19:59
@rvagg rvagg mentioned this pull request Sep 22, 2015
@MylesBorins
Copy link
Contributor

landed in lts-v4.x-staging as 8119693

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libuv Issues and PRs related to the libuv dependency or the uv binding.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants