build: Bump MACOSX_DEPLOYMENT_TARGET to 10.7 by ChALkeR · Pull Request #6402 · nodejs/node (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation14 Commits1 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

ChALkeR

Ref: #5731.

We currently target 10.5.
Libuv is going to drop 10.6 in v2: libuv/libuv#758.

Note that OS X versions 10.5-10.8 are not supported by Apple anymore and do not receive security patches.

Note that we don't state supported OS X versions anywhere, and the only version tested at CI is 10.10. Ref: nodejs/build#367.

/cc @bnoordhuis, @ofrobots, @jbergstroem, @nodejs/ctc

@ofrobots

@evanlucas

@bnoordhuis

Awesome-looking patch. LGTM!

@ChALkeR

@cjihrig

@ChALkeR

Ref: nodejs/build#367

libuv is going to drop 10.6 in v2. Ref: libuv/libuv#758

OS X versions below 10.9 are not supported by Apple anymore and do not receive security patches.

PR-URL: nodejs#6402 Reviewed-By: Ali Ijaz Sheikh ofrobots@google.com Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Colin Ihrig cjihrig@gmail.com

@jasnell

Sigh... lol... yes, can get this one in

@indutny

1 similar comment

@MylesBorins

@jasnell

Landing in master and will pull it back

@ChALkeR ChALkeR added semver-major

PRs that contain breaking changes and should be released in the next major version.

build

Issues and PRs related to build files or the CI.

macos

Issues and PRs related to the macOS platform / OSX.

labels

Apr 26, 2016

jasnell pushed a commit that referenced this pull request

Apr 26, 2016

@ChALkeR @jasnell

Refs: nodejs/build#367

libuv is going to drop 10.6 in v2. Ref: libuv/libuv#758

OS X versions below 10.9 are not supported by Apple anymore and do not receive security patches.

PR-URL: #6402 Reviewed-By: Ali Ijaz Sheikh ofrobots@google.com Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Fedor Indutny fedor.indutny@gmail.com Reviewed-By: Myles Borins myles.borins@gmail.com

@jasnell

@ChALkeR

@jasnell Argh, I made a mistake in the commit message. The first ref should have been #5731. It's not critical, though, you can ignore that.

Sorry for that.

jasnell pushed a commit that referenced this pull request

Apr 26, 2016

@ChALkeR @jasnell

Refs: nodejs/build#367

libuv is going to drop 10.6 in v2. Ref: libuv/libuv#758

OS X versions below 10.9 are not supported by Apple anymore and do not receive security patches.

PR-URL: #6402 Reviewed-By: Ali Ijaz Sheikh ofrobots@google.com Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Fedor Indutny fedor.indutny@gmail.com Reviewed-By: Myles Borins myles.borins@gmail.com

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

@Trott

Looks like this was landed without a CI run. :-(

@ChALkeR

@Trott The CI run was made for the whole 6.0.0 release, and this was the last commit before the release.

@jasnell

@ChALkeR ... the CI run for v6 didn't include this because it came in just after I kicked that off. I ran the tests locally but I already had the work around in place that @bnoordhuis identified (which explains why I haven't been able to recreate this locally... I just switched that back off and low and behold, getting the error).

@rvagg rvagg mentioned this pull request

Apr 17, 2019

Labels

build

Issues and PRs related to build files or the CI.

macos

Issues and PRs related to the macOS platform / OSX.

semver-major

PRs that contain breaking changes and should be released in the next major version.