[BUGFIX beta] Adds early Android versions to legacy target list

emberjs · Dec 5, 2018 · 45e7565 · 45e7565
1 parent 6b173a9
commit 45e7565
Show file tree

Hide file tree

Showing 6 changed files with 845 additions and 30 deletions.
diff --git a/bin/run-tests.js b/bin/run-tests.js
@@ -101,16 +101,16 @@ function generateBuiltTests() {
     return run(common + '&enableoptionalfeatures=true&dist=prod&prod=true');
   });
   testFunctions.push(function() {
-    return run(common + '&ie=true&nolint=true');
+    return run(common + '&legacy=true&nolint=true');
   });
   testFunctions.push(function() {
-    return run(common + '&ie=true&dist=min&prod=true');
+    return run(common + '&legacy=true&dist=min&prod=true');
   });
   testFunctions.push(function() {
-    return run(common + '&ie=true&dist=prod&prod=true');
+    return run(common + '&legacy=true&dist=prod&prod=true');
   });
   testFunctions.push(function() {
-    return run(common + '&ie=true&enableoptionalfeatures=true&dist=prod&prod=true');
+    return run(common + '&legacy=true&enableoptionalfeatures=true&dist=prod&prod=true');
   });
 }
 

diff --git a/ember-cli-build.js b/ember-cli-build.js
@@ -124,7 +124,7 @@ module.exports = function() {
       toES5(templateCompilerDependenciesES)
     );
 
-    bundleTrees.push(new Funnel(bundlesES5, { destDir: 'ie' }));
+    bundleTrees.push(new Funnel(bundlesES5, { destDir: 'legacy' }));
   }
 
   let emberTestsEmptyTestem = new Funnel('tests', {

diff --git a/lib/index.js b/lib/index.js
@@ -69,12 +69,21 @@ module.exports = {
 
     let emberSourceDistPath = path.join(__dirname, '..', 'dist');
 
-    if (this.project.targets) {
-      let browsers = browserslist(this.project.targets.browsers);
-
-      if (browsers.find(browser => browser.startsWith('ie'))) {
-        emberSourceDistPath = path.join(emberSourceDistPath, 'ie');
-      }
+    let emberCliBabel = this.addons.find(a => a.name === 'ember-cli-babel');
+    let needsLegacyBuild = [
+      'transform-template-literals',
+      'transform-literals',
+      'transform-arrow-functions',
+      'transform-destructuring',
+      'transform-spread',
+      'transform-parameters',
+      'transform-computed-properties',
+      'transform-shorthand-properties',
+      'transform-block-scoping',
+    ].some(p => emberCliBabel.isPluginRequired(p));
+
+    if (needsLegacyBuild) {
+      emberSourceDistPath = path.join(emberSourceDistPath, 'legacy');
     }
 
     let emberFiles = [

diff --git a/package.json b/package.json
@@ -24,16 +24,16 @@
     "dist/ember.min.map",
     "dist/ember.prod.js",
     "dist/ember.prod.map",
-    "dist/ie/ember-template-compiler.js",
-    "dist/ie/ember-template-compiler.map",
-    "dist/ie/ember-testing.js",
-    "dist/ie/ember-testing.map",
-    "dist/ie/ember.debug.js",
-    "dist/ie/ember.debug.map",
-    "dist/ie/ember.min.js",
-    "dist/ie/ember.min.map",
-    "dist/ie/ember.prod.js",
-    "dist/ie/ember.prod.map",
+    "dist/legacy/ember-template-compiler.js",
+    "dist/legacy/ember-template-compiler.map",
+    "dist/legacy/ember-testing.js",
+    "dist/legacy/ember-testing.map",
+    "dist/legacy/ember.debug.js",
+    "dist/legacy/ember.debug.map",
+    "dist/legacy/ember.min.js",
+    "dist/legacy/ember.min.map",
+    "dist/legacy/ember.prod.js",
+    "dist/legacy/ember.prod.map",
     "docs/data.json",
     "lib/index.js",
     "vendor/ember"
@@ -65,6 +65,7 @@
     "broccoli-merge-trees": "^2.0.0",
     "browserslist": "^4.3.4",
     "chalk": "^2.3.0",
+    "ember-cli-babel": "^7.1.3",
     "ember-cli-get-component-path-option": "^1.0.0",
     "ember-cli-is-package-missing": "^1.0.0",
     "ember-cli-normalize-entity-name": "^1.0.0",

diff --git a/tests/index.html b/tests/index.html
@@ -74,9 +74,9 @@
 
     <script>
       var dist = QUnit.urlParams.dist;
-      var ie = QUnit.urlParams.ie;
+      var legacy = QUnit.urlParams.legacy;
 
-      var relativePath = ie ? '../ie' : '..';
+      var relativePath = legacy ? '../legacy' : '..';
 
       switch (dist) {
         case 'prod':
@@ -99,9 +99,9 @@
 
     <script>
       var dist = QUnit.urlParams.dist;
-      var ie = QUnit.urlParams.ie;
+      var legacy = QUnit.urlParams.legacy;
 
-      var relativePath = ie ? '../ie' : '..';
+      var relativePath = legacy ? '../legacy' : '..';
 
       if (dist === 'es') {
         // do nothing template compiler already included
@@ -113,9 +113,9 @@
     <script>
       var prod = QUnit.urlParams.prod;
       var dist = QUnit.urlParams.dist;
-      var ie = QUnit.urlParams.ie;
+      var legacy = QUnit.urlParams.legacy;
 
-      var relativePath = ie ? '../ie' : '..';
+      var relativePath = legacy ? '../legacy' : '..';
 
       if (dist === 'es') {
         // do nothing, tests are included