System.import() is deprecated and will be removed soon. Use import() instead.

[wangshuqing]

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch systemjs@6.8.3 for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/systemjs/dist/system.js b/node_modules/systemjs/dist/system.js
index 1cbf2ba..7f0bb75 100644
--- a/node_modules/systemjs/dist/system.js
+++ b/node_modules/systemjs/dist/system.js
@@ -529,7 +529,7 @@
         script.sp = true;
         if (!script.src)
           return;
-        System.import(script.src.slice(0, 7) === 'import:' ? script.src.slice(7) : resolveUrl(script.src, baseUrl)).catch(function (e) {
+          envGlobal.System.import(script.src.slice(0, 7) === 'import:' ? script.src.slice(7) : resolveUrl(script.src, baseUrl)).catch(function (e) {
           // if there is a script load error, dispatch an "error" event
           // on the script tag.
           if (e.message.indexOf('https://git.io/JvFET#3') > -1) {

This issue body was partially generated by patch-package.

[tbrannam]

what were you building that emitted that warning?