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

Should we replace the word "OS X" with "macOS"? #12086

Closed
ghost opened this issue Mar 28, 2017 · 6 comments
Closed

Should we replace the word "OS X" with "macOS"? #12086

ghost opened this issue Mar 28, 2017 · 6 comments
Labels
doc Issues and PRs related to the documentations. good first issue Issues that are suitable for first-time contributors. macos Issues and PRs related to the macOS platform / OSX. question Issues that look for answers.

Comments

@ghost
Copy link

ghost commented Mar 28, 2017

Last year,Apple update MacOS X 10.11 to macOS Sierra(10.12),so "OSX" "OS X" and "MACOSX" more and less are old.Should we update "OSX" "OS X" and "MACOSX" into "macOS" in our code?

@mscdex mscdex added macos Issues and PRs related to the macOS platform / OSX. question Issues that look for answers. labels Mar 28, 2017
@aqrln
Copy link
Contributor

aqrln commented Mar 28, 2017

I don't think it is a good idea to change that in the source code, but the documentation could be updated.

BUILDING.md
77:### Unix / OS X
86:On OS X, you will also need:
240:##### Unix / OS X:
257:##### Unix / OS X:
269:#### Use existing installed ICU (Unix / OS X only):
285:##### Unix / OS X
CONTRIBUTING.md
160:To run the tests on Unix / OS X:
doc/api/child_process.md
436:*Note: Certain platforms (OS X, Linux) will use the value of `argv[0]` for the
doc/api/documentation.md
76:example, that `fs.lchown()` only works on Mac OS X and other BSD-derived systems,
80:relative to Linux and OS X. For an example of the subtle ways in which it's
doc/api/errors.md
526:  parallel, especially on systems (in particular, OS X) where there is a low
doc/api/fs.md
1102:Only available on Mac OS X.
1377:opening a directory on OS X and Linux with the `'a+'` flag - see example below -
1382:// OS X and Linux
1946:The recursive option is only supported on OS X and Windows.
1957:* On OS X, this uses [`kqueue`] for files and [`FSEvents`] for directories.
1975:On Linux and OS X systems, `fs.watch()` resolves the path to an [inode][] and
1985:lifetime of a file, that way though this is different from Linux / OS X,
doc/api/net.md
28:The typical values are 107 on Linux and 103 on OS X. The path is
doc/api/os.md
366:as returned by uname(3). For example `'Linux'` on Linux, `'Darwin'` on OS X and
doc/api/process.md
1628:limited. For instance, on Linux and OS X, `process.title` is limited to the size
doc/releases.md
191:This is particularly recommended if there has been recent work relating to the OS X or Windows installers as they are not tested in any way by CI.
197:Use this opportunity particularly to test OS X and Windows installers if there are any concerns.
207:It is only necessary to have the main Linux (x64 and x86), OS X .pkg and .tar.gz, Windows (x64 and x86) .msi and .exe, source, headers and docs (both produced currently by an OS X slave).

Note: it might be a good opportunity to change the word "slave" there to something else too.

test/README.md
295:Platform check for OS X.
doc/api/child_process.md
78:systems (Unix, Linux, OSX) [`child_process.execFile()`][] can be more efficient
doc/STYLE_GUIDE.md
42:* When using dashes, use emdashes ("—", Ctrl+Alt+"-" on OSX) surrounded by

@richardlau richardlau added good first issue Issues that are suitable for first-time contributors. doc Issues and PRs related to the documentations. labels Mar 28, 2017
@ghost
Copy link
Author

ghost commented Mar 28, 2017

I think source code comment could be updated too

@aqrln
Copy link
Contributor

aqrln commented Mar 28, 2017

@andmeics IMO, there's no point in littering the Git history (i.e., breaking git blame a little bit, potentially introducing conflicts etc.) for a kinda meaningless change that does not impact functionality, nor fixes a problem. Everyone understands what OS X means despite Mac OS X, OS X, macOS or whichever Apple may choose to rename it again in the future being the official name. Others may have different opinions, though. On the other hand, it seems a good change for the docs to me.

@ghost
Copy link
Author

ghost commented Mar 28, 2017

I know,do it can help docs unified

@aqrln
Copy link
Contributor

aqrln commented Mar 28, 2017

@andmeics would you be interested in opening a PR for that? :)

@Fishrock123
Copy link
Contributor

Fishrock123 commented Mar 28, 2017

Yeah let's just leave the source code. People will know what it means for years. IMO we should definitely update the docs though.

@jasnell jasnell closed this as completed in 7a5d07c Apr 4, 2017
jasnell pushed a commit that referenced this issue Apr 4, 2017
This commit updates two paragraphs that exceeded the 80 line standard
after updating to macOS.

PR-URL: #12106
Fixes: #12086
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
jasnell pushed a commit that referenced this issue Apr 4, 2017
This commit updates 3 additional references to Mac OS X in
releases.md to macOS.

PR-URL: #12106
Fixes: #12086
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
italoacasas pushed a commit to italoacasas/node that referenced this issue Apr 10, 2017
This commit updates 3 additional references to Mac OS X in
releases.md to macOS.

PR-URL: nodejs#12106
Fixes: nodejs#12086
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue May 16, 2017
This update changes references to "Mac OS X", "OS X", and "OSX" in
markdown files to "macOS".

Fixes: #12086
PR-URL: #12106
Backport-PR-URL: #12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue May 16, 2017
This commit updates two paragraphs that exceeded the 80 line standard
after updating to macOS.

Fixes: #12086
PR-URL: #12106
Backport-PR-URL: #12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue May 16, 2017
This commit updates 3 additional references to Mac OS X in
releases.md to macOS.

Fixes: #12086
PR-URL: #12106
Backport-PR-URL: #12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue May 18, 2017
This update changes references to "Mac OS X", "OS X", and "OSX" in
markdown files to "macOS".

Fixes: #12086
PR-URL: #12106
Backport-PR-URL: #12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue May 18, 2017
This commit updates two paragraphs that exceeded the 80 line standard
after updating to macOS.

Fixes: #12086
PR-URL: #12106
Backport-PR-URL: #12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue May 18, 2017
This commit updates 3 additional references to Mac OS X in
releases.md to macOS.

Fixes: #12086
PR-URL: #12106
Backport-PR-URL: #12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
andrew749 pushed a commit to michielbaird/node that referenced this issue Jul 19, 2017
This update changes references to "Mac OS X", "OS X", and "OSX" in
markdown files to "macOS".

Fixes: nodejs/node#12086
PR-URL: nodejs/node#12106
Backport-PR-URL: nodejs/node#12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
andrew749 pushed a commit to michielbaird/node that referenced this issue Jul 19, 2017
This commit updates two paragraphs that exceeded the 80 line standard
after updating to macOS.

Fixes: nodejs/node#12086
PR-URL: nodejs/node#12106
Backport-PR-URL: nodejs/node#12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
andrew749 pushed a commit to michielbaird/node that referenced this issue Jul 19, 2017
This commit updates 3 additional references to Mac OS X in
releases.md to macOS.

Fixes: nodejs/node#12086
PR-URL: nodejs/node#12106
Backport-PR-URL: nodejs/node#12650
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. good first issue Issues that are suitable for first-time contributors. macos Issues and PRs related to the macOS platform / OSX. question Issues that look for answers.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants