diff --git a/Makefile b/Makefile index c718da0b9d7ee6..f179fcbca79733 100644 --- a/Makefile +++ b/Makefile @@ -1003,7 +1003,7 @@ else BINARYNAME=$(TARNAME)-$(PLATFORM)-$(ARCH) endif BINARYTAR=$(BINARYNAME).tar -# OSX doesn't have xz installed by default, http://macpkg.sourceforge.net/ +# macOS doesn't have xz installed by default, http://macpkg.sourceforge.net/ HAS_XZ ?= $(shell command -v xz > /dev/null 2>&1; [ $$? -eq 0 ] && echo 1 || echo 0) # Supply SKIP_XZ=1 to explicitly skip .tar.xz creation SKIP_XZ ?= 0 diff --git a/configure.py b/configure.py index 8b311a0d702eeb..379fb5bb06c413 100755 --- a/configure.py +++ b/configure.py @@ -164,14 +164,14 @@ dest="fully_static", default=None, help="Generate an executable without external dynamic libraries. This " - "will not work on OSX when using the default compilation environment") + "will not work on macOS when using the default compilation environment") parser.add_argument("--partly-static", action="store_true", dest="partly_static", default=None, help="Generate an executable with libgcc and libstdc++ libraries. This " - "will not work on OSX when using the default compilation environment") + "will not work on macOS when using the default compilation environment") parser.add_argument("--enable-vtune-profiling", action="store_true", @@ -1745,7 +1745,7 @@ def without_ssl_error(option): def configure_static(o): if options.fully_static or options.partly_static: if flavor == 'mac': - warn("Generation of static executable will not work on OSX " + warn("Generation of static executable will not work on macOS " "when using the default compilation environment") return diff --git a/node.gypi b/node.gypi index 33764e0b6a6e98..fc1a87fc0b6aae 100644 --- a/node.gypi +++ b/node.gypi @@ -225,7 +225,7 @@ }], [ 'OS=="mac"', { - # linking Corefoundation is needed since certain OSX debugging tools + # linking Corefoundation is needed since certain macOS debugging tools # like Instruments require it for some features 'libraries': [ '-framework CoreFoundation' ], 'defines!': [ diff --git a/src/large_pages/node_large_page.cc b/src/large_pages/node_large_page.cc index 1dddcb9f6c3560..43ebb7641a627c 100644 --- a/src/large_pages/node_large_page.cc +++ b/src/large_pages/node_large_page.cc @@ -48,7 +48,7 @@ // To move the .text section, perform the following steps: // * Map a new, temporary area and copy the original code there. // * Use mmap using the start address with MAP_FIXED so we get exactly the -// same virtual address (except on OSX). On platforms other than Linux, +// same virtual address (except on macOS). On platforms other than Linux, // use mmap flags to request hugepages. // * On Linux use madvise with MADV_HUGEPAGE to use anonymous 2MB pages. // * If successful copy the code to the newly mapped area and protect it to diff --git a/src/node_internals.h b/src/node_internals.h index e62122379c9820..a0d08fd81fefed 100644 --- a/src/node_internals.h +++ b/src/node_internals.h @@ -362,7 +362,7 @@ bool HasSignalJSHandler(int signum); #ifdef _WIN32 typedef SYSTEMTIME TIME_TYPE; -#else // UNIX, OSX +#else // UNIX, macOS typedef struct tm TIME_TYPE; #endif diff --git a/src/node_report.cc b/src/node_report.cc index ff6e2027dda0d7..69b8646d40f3f0 100644 --- a/src/node_report.cc +++ b/src/node_report.cc @@ -133,7 +133,7 @@ static void WriteNodeReport(Isolate* isolate, tm_struct.wMinute, tm_struct.wSecond); writer.json_keyvalue("dumpEventTime", timebuf); -#else // UNIX, OSX +#else // UNIX, macOS snprintf(timebuf, sizeof(timebuf), "%4d-%02d-%02dT%02d:%02d:%02dZ", diff --git a/src/util.cc b/src/util.cc index 305d66f2f02a56..f9aad9ef5a6213 100644 --- a/src/util.cc +++ b/src/util.cc @@ -320,7 +320,7 @@ std::vector ReadFileSync(FILE* fp) { void DiagnosticFilename::LocalTime(TIME_TYPE* tm_struct) { #ifdef _WIN32 GetLocalTime(tm_struct); -#else // UNIX, OSX +#else // UNIX, macOS struct timeval time_val; gettimeofday(&time_val, nullptr); localtime_r(&time_val.tv_sec, tm_struct); @@ -343,7 +343,7 @@ std::string DiagnosticFilename::MakeFilename( oss << "." << std::setfill('0') << std::setw(2) << tm_struct.wHour; oss << std::setfill('0') << std::setw(2) << tm_struct.wMinute; oss << std::setfill('0') << std::setw(2) << tm_struct.wSecond; -#else // UNIX, OSX +#else // UNIX, macOS oss << "." << std::setfill('0') << std::setw(4) diff --git a/test/common/README.md b/test/common/README.md index fca13871ecb540..a322242138740c 100644 --- a/test/common/README.md +++ b/test/common/README.md @@ -282,7 +282,7 @@ Platform check for Linux. Platform check for Linux on PowerPC. -### `isOSX` +### `isMacOS` * [\][] diff --git a/test/common/index.js b/test/common/index.js index 02ca35c5c55659..8bf8136e5025f0 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -141,7 +141,7 @@ const isSunOS = process.platform === 'sunos'; const isFreeBSD = process.platform === 'freebsd'; const isOpenBSD = process.platform === 'openbsd'; const isLinux = process.platform === 'linux'; -const isOSX = process.platform === 'darwin'; +const isMacOS = process.platform === 'darwin'; const isASan = process.config.variables.asan === 1; const isPi = (() => { try { @@ -1003,7 +1003,7 @@ const common = { isLinux, isMainThread, isOpenBSD, - isOSX, + isMacOS, isPi, isSunOS, isWindows, diff --git a/test/common/index.mjs b/test/common/index.mjs index 430527faf8f0ea..007ce233fbea0d 100644 --- a/test/common/index.mjs +++ b/test/common/index.mjs @@ -30,7 +30,7 @@ const { isLinuxPPCBE, isMainThread, isOpenBSD, - isOSX, + isMacOS, isSunOS, isWindows, localIPv6Hosts, @@ -85,7 +85,7 @@ export { isLinuxPPCBE, isMainThread, isOpenBSD, - isOSX, + isMacOS, isSunOS, isWindows, localIPv6Hosts, diff --git a/test/common/shared-lib-util.js b/test/common/shared-lib-util.js index 3ecd38791cf481..b5d947a266cfea 100644 --- a/test/common/shared-lib-util.js +++ b/test/common/shared-lib-util.js @@ -22,7 +22,7 @@ function addLibraryPath(env) { env.LIBPATH = (env.LIBPATH ? env.LIBPATH + path.delimiter : '') + kExecPath; - // For Mac OSX. + // For macOS. env.DYLD_LIBRARY_PATH = (env.DYLD_LIBRARY_PATH ? env.DYLD_LIBRARY_PATH + path.delimiter : '') + kExecPath; diff --git a/test/fixtures/permission/fs-write.js b/test/fixtures/permission/fs-write.js index 3afbdb4fbbf197..31e96860972a9f 100644 --- a/test/fixtures/permission/fs-write.js +++ b/test/fixtures/permission/fs-write.js @@ -338,7 +338,7 @@ const relativeProtectedFolder = process.env.RELATIVEBLOCKEDFOLDER; // fs.lchmod { - if (common.isOSX) { + if (common.isMacOS) { fs.lchmod(blockedFile, 0o755, common.expectsError({ code: 'ERR_ACCESS_DENIED', permission: 'FileSystemWrite', diff --git a/test/parallel/test-child-process-exec-timeout-expire.js b/test/parallel/test-child-process-exec-timeout-expire.js index 08e54544836d98..6b62d131cbede5 100644 --- a/test/parallel/test-child-process-exec-timeout-expire.js +++ b/test/parallel/test-child-process-exec-timeout-expire.js @@ -38,7 +38,7 @@ cp.exec(cmd, { // At least starting with Darwin Kernel Version 16.4.0, sending a SIGTERM to a // process that is still starting up kills it with SIGKILL instead of SIGTERM. // See: https://github.com/libuv/libuv/issues/1226 - if (common.isOSX) + if (common.isMacOS) assert.ok(err.signal === 'SIGTERM' || err.signal === 'SIGKILL'); else assert.strictEqual(err.signal, sigterm); diff --git a/test/parallel/test-cluster-bind-privileged-port.js b/test/parallel/test-cluster-bind-privileged-port.js index c971b7656f1221..3ac36543a27ba8 100644 --- a/test/parallel/test-cluster-bind-privileged-port.js +++ b/test/parallel/test-cluster-bind-privileged-port.js @@ -38,8 +38,8 @@ if (common.isLinux) { } } -// Skip on OS X Mojave. https://github.com/nodejs/node/issues/21679 -if (common.isOSX) +// Skip on macOS Mojave. https://github.com/nodejs/node/issues/21679 +if (common.isMacOS) common.skip('macOS may allow ordinary processes to use any port'); if (common.isIBMi) diff --git a/test/parallel/test-cluster-concurrent-disconnect.js b/test/parallel/test-cluster-concurrent-disconnect.js index c2ba7b43447733..b754fa221a0dcd 100644 --- a/test/parallel/test-cluster-concurrent-disconnect.js +++ b/test/parallel/test-cluster-concurrent-disconnect.js @@ -26,7 +26,7 @@ if (cluster.isPrimary) { // to send messages when the worker is disconnecting. worker.on('error', (err) => { assert.strictEqual(err.syscall, 'write'); - if (common.isOSX) { + if (common.isMacOS) { assert(['EPIPE', 'ENOTCONN'].includes(err.code), err); } else { assert(['EPIPE', 'ECONNRESET'].includes(err.code), err); diff --git a/test/parallel/test-cluster-shared-handle-bind-privileged-port.js b/test/parallel/test-cluster-shared-handle-bind-privileged-port.js index 775487ad7df034..8bdde0a3320e44 100644 --- a/test/parallel/test-cluster-shared-handle-bind-privileged-port.js +++ b/test/parallel/test-cluster-shared-handle-bind-privileged-port.js @@ -22,8 +22,8 @@ 'use strict'; const common = require('../common'); -// Skip on OS X Mojave. https://github.com/nodejs/node/issues/21679 -if (common.isOSX) +// Skip on macOS Mojave. https://github.com/nodejs/node/issues/21679 +if (common.isMacOS) common.skip('macOS may allow ordinary processes to use any port'); if (common.isIBMi) diff --git a/test/parallel/test-fs-lchmod.js b/test/parallel/test-fs-lchmod.js index 877aea3376554d..d4397102914944 100644 --- a/test/parallel/test-fs-lchmod.js +++ b/test/parallel/test-fs-lchmod.js @@ -9,7 +9,7 @@ const f = __filename; // This test ensures that input for lchmod is valid, testing for valid // inputs for path, mode and callback -if (!common.isOSX) { +if (!common.isMacOS) { common.skip('lchmod is only available on macOS'); } diff --git a/test/parallel/test-fs-mkdir.js b/test/parallel/test-fs-mkdir.js index eac13f3cac5d4a..89b8b436d5c9f4 100644 --- a/test/parallel/test-fs-mkdir.js +++ b/test/parallel/test-fs-mkdir.js @@ -217,7 +217,7 @@ function nextdir() { // mkdirpSync dirname loop // XXX: windows and smartos have issues removing a directory that you're in. -if (common.isMainThread && (common.isLinux || common.isOSX)) { +if (common.isMainThread && (common.isLinux || common.isMacOS)) { const pathname = tmpdir.resolve(nextdir()); fs.mkdirSync(pathname); process.chdir(pathname); diff --git a/test/parallel/test-fs-open-flags.js b/test/parallel/test-fs-open-flags.js index bf00ec3308268a..624bfb207b51cf 100644 --- a/test/parallel/test-fs-open-flags.js +++ b/test/parallel/test-fs-open-flags.js @@ -82,7 +82,7 @@ assert.throws( { code: 'ERR_INVALID_ARG_VALUE', name: 'TypeError' } ); -if (common.isLinux || common.isOSX) { +if (common.isLinux || common.isMacOS) { const tmpdir = require('../common/tmpdir'); tmpdir.refresh(); const file = tmpdir.resolve('a.js'); diff --git a/test/parallel/test-fs-promises-file-handle-aggregate-errors.js b/test/parallel/test-fs-promises-file-handle-aggregate-errors.js index 8e71ac98e6e8ae..f53ce1eeaf0df3 100644 --- a/test/parallel/test-fs-promises-file-handle-aggregate-errors.js +++ b/test/parallel/test-fs-promises-file-handle-aggregate-errors.js @@ -65,7 +65,7 @@ async function checkAggregateError(op) { await checkAggregateError((filePath) => truncate(filePath)); await checkAggregateError((filePath) => readFile(filePath)); await checkAggregateError((filePath) => writeFile(filePath, '123')); - if (common.isOSX) { + if (common.isMacOS) { await checkAggregateError((filePath) => lchmod(filePath, 0o777)); } })().then(common.mustCall()); diff --git a/test/parallel/test-fs-promises-file-handle-close-errors.js b/test/parallel/test-fs-promises-file-handle-close-errors.js index 89bf53300560f6..8d0a1bad460552 100644 --- a/test/parallel/test-fs-promises-file-handle-close-errors.js +++ b/test/parallel/test-fs-promises-file-handle-close-errors.js @@ -60,7 +60,7 @@ async function checkCloseError(op) { await checkCloseError((filePath) => truncate(filePath)); await checkCloseError((filePath) => readFile(filePath)); await checkCloseError((filePath) => writeFile(filePath, '123')); - if (common.isOSX) { + if (common.isMacOS) { await checkCloseError((filePath) => lchmod(filePath, 0o777)); } })().then(common.mustCall()); diff --git a/test/parallel/test-fs-promises-file-handle-op-errors.js b/test/parallel/test-fs-promises-file-handle-op-errors.js index c869351985b6fa..4f86b9a9f83f33 100644 --- a/test/parallel/test-fs-promises-file-handle-op-errors.js +++ b/test/parallel/test-fs-promises-file-handle-op-errors.js @@ -54,7 +54,7 @@ async function checkOperationError(op) { await checkOperationError((filePath) => truncate(filePath)); await checkOperationError((filePath) => readFile(filePath)); await checkOperationError((filePath) => writeFile(filePath, '123')); - if (common.isOSX) { + if (common.isMacOS) { await checkOperationError((filePath) => lchmod(filePath, 0o777)); } })().then(common.mustCall()); diff --git a/test/parallel/test-fs-promises-watch.js b/test/parallel/test-fs-promises-watch.js index 94ef7fcb2abe30..2d417fd2886929 100644 --- a/test/parallel/test-fs-promises-watch.js +++ b/test/parallel/test-fs-promises-watch.js @@ -24,14 +24,14 @@ class WatchTestCase { const kCases = [ // Watch on a directory should callback with a filename on supported systems new WatchTestCase( - common.isLinux || common.isOSX || common.isWindows || common.isAIX, + common.isLinux || common.isMacOS || common.isWindows || common.isAIX, 'watch1', 'foo', 'filePath' ), // Watch on a file should callback with a filename on supported systems new WatchTestCase( - common.isLinux || common.isOSX || common.isWindows, + common.isLinux || common.isMacOS || common.isWindows, 'watch2', 'bar', 'dirPath' diff --git a/test/parallel/test-fs-promises.js b/test/parallel/test-fs-promises.js index 2dc0ae8cb44f6d..d28af0f4833c49 100644 --- a/test/parallel/test-fs-promises.js +++ b/test/parallel/test-fs-promises.js @@ -314,7 +314,7 @@ async function executeOnHandle(dest, func) { (await readlink(newLink)).toLowerCase()); const newMode = 0o666; - if (common.isOSX) { + if (common.isMacOS) { // `lchmod` is only available on macOS. await lchmod(newLink, newMode); stats = await lstat(newLink); diff --git a/test/parallel/test-fs-readdir-buffer.js b/test/parallel/test-fs-readdir-buffer.js index a679aad01bf447..54b7353cb1b3b2 100644 --- a/test/parallel/test-fs-readdir-buffer.js +++ b/test/parallel/test-fs-readdir-buffer.js @@ -2,7 +2,7 @@ const common = require('../common'); const fs = require('fs'); -if (!common.isOSX) { +if (!common.isMacOS) { common.skip('this tests works only on MacOS'); } diff --git a/test/parallel/test-fs-watch-enoent.js b/test/parallel/test-fs-watch-enoent.js index 43bcc647749f07..282171e2d22f61 100644 --- a/test/parallel/test-fs-watch-enoent.js +++ b/test/parallel/test-fs-watch-enoent.js @@ -46,7 +46,7 @@ tmpdir.refresh(); } { - if (common.isOSX || common.isWindows) { + if (common.isMacOS || common.isWindows) { const file = tmpdir.resolve('file-to-watch'); fs.writeFileSync(file, 'test'); const watcher = fs.watch(file, common.mustNotCall()); diff --git a/test/parallel/test-fs-watch-recursive-validate.js b/test/parallel/test-fs-watch-recursive-validate.js index cfebc64818283e..09eccc2d7e5c4c 100644 --- a/test/parallel/test-fs-watch-recursive-validate.js +++ b/test/parallel/test-fs-watch-recursive-validate.js @@ -23,7 +23,7 @@ tmpdir.refresh(); (async () => { // Handle non-boolean values for options.recursive - if (!common.isWindows && !common.isOSX) { + if (!common.isWindows && !common.isMacOS) { assert.throws(() => { const testsubdir = fs.mkdtempSync(testDir + path.sep); fs.watch(testsubdir, { recursive: '1' }); diff --git a/test/parallel/test-fs-watch.js b/test/parallel/test-fs-watch.js index 2c6386a2d118b6..a494b9f8df6473 100644 --- a/test/parallel/test-fs-watch.js +++ b/test/parallel/test-fs-watch.js @@ -24,14 +24,14 @@ class WatchTestCase { const cases = [ // Watch on a file should callback with a filename on supported systems new WatchTestCase( - common.isLinux || common.isOSX || common.isWindows || common.isAIX, + common.isLinux || common.isMacOS || common.isWindows || common.isAIX, 'watch1', 'foo', 'filePath' ), // Watch on a directory should callback with a filename on supported systems new WatchTestCase( - common.isLinux || common.isOSX || common.isWindows, + common.isLinux || common.isMacOS || common.isWindows, 'watch2', 'bar', 'dirPath' @@ -66,7 +66,7 @@ for (const testCase of cases) { clearInterval(interval); interval = null; } - if (common.isOSX) + if (common.isMacOS) assert.strictEqual(['rename', 'change'].includes(eventType), true); else assert.strictEqual(eventType, 'change'); diff --git a/test/parallel/test-fs-watchfile.js b/test/parallel/test-fs-watchfile.js index 966876b6787046..5519848e026401 100644 --- a/test/parallel/test-fs-watchfile.js +++ b/test/parallel/test-fs-watchfile.js @@ -84,7 +84,7 @@ watcher.on('stop', common.mustCall()); // Watch events should callback with a filename on supported systems. // Omitting AIX. It works but not reliably. -if (common.isLinux || common.isOSX || common.isWindows) { +if (common.isLinux || common.isMacOS || common.isWindows) { const dir = tmpdir.resolve('watch'); fs.mkdir(dir, common.mustCall(function(err) { diff --git a/test/parallel/test-process-getgroups.js b/test/parallel/test-process-getgroups.js index 3479cf7028b28b..28df13205f44fc 100644 --- a/test/parallel/test-process-getgroups.js +++ b/test/parallel/test-process-getgroups.js @@ -24,7 +24,7 @@ const common = require('../common'); // Check `id -G` and `process.getgroups()` return same groups. -if (common.isOSX) +if (common.isMacOS) common.skip('Output of `id -G` is unreliable on Darwin.'); const assert = require('assert'); diff --git a/test/parallel/test-stdin-from-file-spawn.js b/test/parallel/test-stdin-from-file-spawn.js index 12c235fcfeb081..3830ac124a1d8a 100644 --- a/test/parallel/test-stdin-from-file-spawn.js +++ b/test/parallel/test-stdin-from-file-spawn.js @@ -8,7 +8,7 @@ if (process.platform === 'linux' || process.platform === 'darwin') { } else if (process.platform === 'win32') { defaultShell = 'cmd.exe'; } else { - common.skip('This is test exists only on Linux/Win32/OSX'); + common.skip('This is test exists only on Linux/Win32/macOS'); } const { execSync } = require('child_process'); diff --git a/test/parallel/test-tls-honorcipherorder.js b/test/parallel/test-tls-honorcipherorder.js index 8e7c4badcb269d..5f123cd739a4c0 100644 --- a/test/parallel/test-tls-honorcipherorder.js +++ b/test/parallel/test-tls-honorcipherorder.js @@ -29,7 +29,7 @@ function test(honorCipherOrder, clientCipher, expectedCipher, defaultCiphers) { const server = tls.createServer(soptions, mustCall(function(clearTextStream) { // End socket to send CLOSE_NOTIFY and TCP FIN packet, otherwise - // it may hang for ~30 seconds in FIN_WAIT_1 state (at least on OSX). + // it may hang for ~30 seconds in FIN_WAIT_1 state (at least on macOS). clearTextStream.end(); })); server.listen(0, localhost, mustCall(function() { diff --git a/test/parallel/test-watch-mode-files_watcher.mjs b/test/parallel/test-watch-mode-files_watcher.mjs index fb358371750d48..2192a198d9cbbb 100644 --- a/test/parallel/test-watch-mode-files_watcher.mjs +++ b/test/parallel/test-watch-mode-files_watcher.mjs @@ -15,7 +15,7 @@ import watcher from 'internal/watch_mode/files_watcher'; if (common.isIBMi) common.skip('IBMi does not support `fs.watch()`'); -const supportsRecursiveWatching = common.isOSX || common.isWindows; +const supportsRecursiveWatching = common.isMacOS || common.isWindows; const { FilesWatcher } = watcher; tmpdir.refresh(); diff --git a/test/sequential/test-cluster-send-handle-large-payload.js b/test/sequential/test-cluster-send-handle-large-payload.js index b227a7970ae6ba..c5226ceb40688a 100644 --- a/test/sequential/test-cluster-send-handle-large-payload.js +++ b/test/sequential/test-cluster-send-handle-large-payload.js @@ -43,7 +43,7 @@ if (cluster.isPrimary) { // Send a second message after a delay on macOS. // // Refs: https://github.com/nodejs/node/issues/14747 - if (common.isOSX) + if (common.isMacOS) setTimeout(() => { process.send({ payload }, handle); }, 1000); else process.send({ payload }, handle); diff --git a/test/sequential/test-fs-watch.js b/test/sequential/test-fs-watch.js index d94021072af498..b7b60abaa5cb7a 100644 --- a/test/sequential/test-fs-watch.js +++ b/test/sequential/test-fs-watch.js @@ -35,7 +35,7 @@ if (!common.isMainThread) const expectFilePath = common.isWindows || common.isLinux || - common.isOSX || + common.isMacOS || common.isAIX; const testDir = tmpdir.path; @@ -124,7 +124,7 @@ function repeat(fn) { // Whitebox test to ensure that wrapped FSEvent is safe // https://github.com/joyent/node/issues/6690 { - if (common.isOSX || common.isWindows) { + if (common.isMacOS || common.isWindows) { let oldhandle; assert.throws( () => { @@ -144,7 +144,7 @@ function repeat(fn) { } { - if (common.isOSX || common.isWindows) { + if (common.isMacOS || common.isWindows) { let oldhandle; assert.throws( () => { diff --git a/test/sequential/test-watch-mode.mjs b/test/sequential/test-watch-mode.mjs index d1dbd75323ddc5..300e4f4e8dc472 100644 --- a/test/sequential/test-watch-mode.mjs +++ b/test/sequential/test-watch-mode.mjs @@ -14,7 +14,7 @@ import { createInterface } from 'node:readline'; if (common.isIBMi) common.skip('IBMi does not support `fs.watch()`'); -const supportsRecursive = common.isOSX || common.isWindows; +const supportsRecursive = common.isMacOS || common.isWindows; function restart(file, content = readFileSync(file)) { // To avoid flakiness, we save the file repeatedly until test is done