allow CJS as a ext

maccman · Aug 17, 2012 · a08dfff · a08dfff
1 parent 453cc37
commit a08dfff
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/examples/example.js b/examples/example.js
@@ -58,7 +58,7 @@
 
   return this.require;
 }).call(this);
-this.require.define({"modules/program":function(exports, require, module){module.exports = function(){
+this.require.define({"modules/program.cjs":function(exports, require, module){module.exports = function(){
   alert('Long live the Programs!');
 };
 ;}});

diff --git a/examples/modules/program.module.js → examples/modules/program.cjs.js b/examples/modules/program.module.js → examples/modules/program.cjs.js
diff --git a/lib/sprockets/commonjs.rb b/lib/sprockets/commonjs.rb
@@ -4,11 +4,13 @@
 module Sprockets
   class CommonJS < Tilt::Template
 
-    DEFINE_WRAPPER = '%s.define({%s:' +
+    WRAPPER = '%s.define({%s:' +
                      'function(exports, require, module){' +
                      '%s' +
                      ";}});\n"
 
+    EXTENSIONS = %w{.module .cjs}
+
     class << self
       attr_accessor :default_namespace
     end
@@ -36,7 +38,7 @@ def evaluate(scope, locals, &block)
     attr_reader :namespace
 
     def commonjs_module?(scope)
-      File.extname(scope.logical_path) == '.module'
+      EXTENSIONS.include?(File.extname(scope.logical_path))
     end
 
     def module_name(scope)