Merge pull request opal#1543 from opal/elia/extract-sprockets

extract sprockets to opal-sprockets gem

Author: elia

.travis.yml

@@ -24,8 +24,8 @@ matrix:
     - rvm: 2.3.1
       env: RUN=mspec_ruby_nodejs TZ="/usr/share/zoneinfo/Pacific/Fiji"
 
-    - rvm: 2.3.1
-      env: RUN=mspec_ruby_sprockets_phantomjs
+    # - rvm: 2.3.1
+    #   env: RUN=mspec_ruby_sprockets_phantomjs
 
     - rvm: 2.3.1
       env: RUN=minitest

CHANGELOG.md

@@ -61,9 +61,15 @@ Whitespace conventions:
     * `Number#positive?` (to return false for 0)
 
 
+### Deprecated
+
+- `require 'opal/server` and `Opal::Server` are deprecated in favor of `require 'opal/sprockets/server'` and `Opal::Sprockets::Server` (now part of the opal-sprockets gem).
+
+
 ### Removed
 
 - Removed `yaml` from stdlib, the older implementation was only available for NodeJS and not tested. Replace with `require 'nodejs/yaml'`
+- Extracted sprockets support to `opal-sprockets` which should allow for wider support and less coupling (e.g. the `opal` gem will now be able to improve the compiler without worrying about `sprockets` updates). All the old behavior is preserved except for `Opal::Server` that has become `Opal::Sprockets::Server` (see Deprecated section above).
 
 
 ### Fixed

lib/opal.rb

@@ -1,12 +1,14 @@
+require 'opal/config'
 require 'opal/compiler'
 require 'opal/builder'
 require 'opal/erb'
-require 'opal/sprockets'
 require 'opal/paths'
 require 'opal/version'
 require 'opal/errors'
+require 'opal/source_map'
 
 # Opal is a ruby to javascript compiler, with a runtime for running
 # in any JavaScript environment.
 module Opal
+  autoload :Server, 'opal/server'
 end

lib/opal/server.rb

@@ -0,0 +1,3 @@
+warn "DEPRECATED: `require 'opal/server` and `Opal::Server` are deprecated in favor of `require 'opal/sprockets/server'` and `Opal::Sprockets::Server` (now part of the opal-sprockets gem)."
+require 'opal/sprockets/server'
+Opal::Server = Opal::Sprockets::Server