replace require kludge in runRollup.js (#706 - comment)

rollup · Aug 30, 2016 · 3817b74 · 3817b74
1 parent 93b8a60
commit 3817b74
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/bin/src/runRollup.js b/bin/src/runRollup.js
@@ -1,10 +1,9 @@
 import { realpathSync } from 'fs';
+import { rollup } from 'rollup';
 import relative from 'require-relative';
 import handleError from './handleError';
 import SOURCEMAPPING_URL from './sourceMappingUrl.js';
 
-const rollup = require( '../dist/rollup.js' ); // TODO make this an import, somehow
-
 import { install as installSourcemapSupport } from 'source-map-support';
 installSourcemapSupport();
 
@@ -58,7 +57,7 @@ export default function runRollup ( command ) {
 			config = realpathSync( config );
 		}
 
-		rollup.rollup({
+		rollup({
 			entry: config,
 			onwarn: message => {
 				if ( /Treating .+ as external dependency/.test( message ) ) return;
@@ -217,7 +216,7 @@ function bundle ( options ) {
 		handleError({ code: 'MISSING_INPUT_OPTION' });
 	}
 
-	return rollup.rollup( options ).then( bundle => {
+	return rollup( options ).then( bundle => {
 		if ( options.dest ) {
 			return bundle.write( options );
 		}

diff --git a/rollup.config.cli.js b/rollup.config.cli.js
@@ -15,7 +15,7 @@ export default {
 		buble(),
 		commonjs({
 			include: 'node_modules/**',
-			namedExports: { 'chalk': [ 'red', 'cyan', 'grey' ] }
+			namedExports: { chalk: [ 'red', 'cyan', 'grey' ] }
 		}),
 		nodeResolve({
 			main: true
@@ -25,6 +25,10 @@ export default {
 		'fs',
 		'path',
 		'module',
-		'source-map-support'
-	]
+		'source-map-support',
+		'rollup'
+	],
+	paths: {
+		rollup: '../dist/rollup.js'
+	}
 };