fix: remove 'use strict' from rollup because vttjs isn't strict (#4551)

videojs · Aug 14, 2017 · db55bbd · db55bbd
1 parent 6233d14
commit db55bbd
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/build/rollup.js b/build/rollup.js
@@ -71,6 +71,7 @@ const es = {
     legacy: true
   },
   banner: compiledLicense(Object.assign({includesVtt: true}, bannerData)),
+  useStrict: false,
   format: 'es',
   dest: 'dist/video.es.js'
 };
@@ -94,6 +95,7 @@ const umd = {
     legacy: true
   },
   banner: compiledLicense(Object.assign({includesVtt: true}, bannerData)),
+  useStrict: false,
   format: 'umd',
   dest: 'dist/video.js'
 };
@@ -134,10 +136,11 @@ const minifiedNovttUmd = Object.assign({}, _.cloneDeep(minifiedUmd), {
 
 minifiedNovttUmd.options.plugins.unshift(ignore(['videojs-vtt.js']));
 
-function runRollup({options, format, dest, banner}) {
+function runRollup({options, useStrict, format, dest, banner}) {
   rollup(options)
   .then(function(bundle) {
     bundle.write({
+      useStrict,
       format,
       dest,
       banner,
@@ -161,7 +164,7 @@ if (!args.watch) {
     runRollup(novttUmd);
   }
 } else {
-  const props = ['format', 'dest', 'banner'];
+  const props = ['format', 'dest', 'banner', 'useStrict'];
   const watchers = [
     ['es', watch({rollup},
                  Object.assign({},