lib: end-of-life the deprecated require('sys')

See: nodejs#2405
sam-github · Feb 24, 2019 · f24105e · f24105e
1 parent 7e0ddf6
commit f24105e
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 34 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -137,7 +137,7 @@ module.exports = {
     'no-path-concat': 'error',
     'no-proto': 'error',
     'no-redeclare': 'error',
-    'no-restricted-modules': ['error', 'sys'],
+    'no-restricted-modules': ['error'],
     /* eslint-disable max-len */
     'no-restricted-properties': [
       'error',

diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -551,6 +551,9 @@ The `REPLServer.prototype.convertToContext()` API has been removed.
 ### DEP0025: require('sys')
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: XXX
+    description: End-of-Life.
   - version:
     - v4.8.6
     - v6.12.0
@@ -561,7 +564,7 @@ changes:
     description: Runtime deprecation.
 -->
 
-Type: Runtime
+Type: End-of-Life
 
 The `sys` module is deprecated. Please use the [`util`][] module instead.
 

diff --git a/lib/internal/bootstrap/cache.js b/lib/internal/bootstrap/cache.js
@@ -14,7 +14,6 @@ const { hasTracing, hasInspector } = internalBinding('config');
 // Modules with source code compiled in js2c that
 // cannot be compiled with the code cache.
 const cannotBeRequired = [
-  'sys',  // Deprecated.
   'internal/v8_prof_polyfill',
   'internal/v8_prof_processor',
 

diff --git a/lib/sys.js b/lib/sys.js
diff --git a/node.gyp b/node.gyp
@@ -72,7 +72,6 @@
       'lib/_stream_passthrough.js',
       'lib/_stream_wrap.js',
       'lib/string_decoder.js',
-      'lib/sys.js',
       'lib/timers.js',
       'lib/tls.js',
       'lib/_tls_common.js',