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

Cherrypicked commits from joyent/node #1163

Closed
wants to merge 4 commits into from
Closed

Cherrypicked commits from joyent/node #1163

wants to merge 4 commits into from

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Mar 16, 2015

See individual commits for more detail

Julien Gilli and others added 4 commits March 16, 2015 15:07
Make PACKAGEMAKER customizable because PackageMaker is not necessarily
installed in /Developer on OSX anymore.

PR-URL: nodejs/node-v0.x-archive#9377
Reviewed-By: Timothy J Fontaine <tjfontaine@gmail.com>
This commit changes the Windows examples in path.markdown to
correctly display '\\'.

PR-URL: nodejs/node-v0.x-archive#9412
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Allows socket.remote* properties to still be accessed even after the
socket is closed.

Fixes: nodejs/node-v0.x-archive#9287
PR-URL: nodejs/node-v0.x-archive#9366
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: nodejs/node-v0.x-archive#8741
Reviewed-By: Fedor Indutny <fedor@indutny.com>
@bnoordhuis
Copy link
Member

LGTM. 095ae0d looks like a semver-minor change.

@cjihrig cjihrig added the semver-minor PRs that contain new features and should be released in the next minor version. label Mar 16, 2015
@cjihrig
Copy link
Contributor Author

cjihrig commented Mar 16, 2015

Right. I'll call it out in #1157

@cjihrig
Copy link
Contributor Author

cjihrig commented Mar 16, 2015

Landed in 89bf6c0, e6e616f, 30666f2, and 68d4bed.

@cjihrig cjihrig closed this Mar 16, 2015
@cjihrig cjihrig deleted the node branch March 16, 2015 20:24
@rvagg
Copy link
Member

rvagg commented Mar 18, 2015

sorry, I would have -1'd the PackageMaker path thing because (1) we're not using it and (2) we're going to ditch PackageMaker entirely as soon as I get my act in to gear with #776, but that's no big deal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-minor PRs that contain new features and should be released in the next minor version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants