middleman-core 3.0.2 → 3.0.4

data/lib/middleman-core/application.rb

@@ -223,7 +223,7 @@ module Middleman
     # messages, which can take a long time (minutes at full CPU)
     # if the object is huge or has cyclic references, like this.
     def to_s
-      "#<Middleman::Application>"
+      "#<Middleman::Application:0x#{object_id}>"
     end
 
     # Expand a path to include the index file if it's a directory

data/lib/middleman-core/cli/server.rb

@@ -37,6 +37,10 @@ module Middleman::Cli
       :type    => :boolean,
       :default => false,
       :desc    => 'Generate profiling report for server startup'
+    method_option :reload_paths,
+      :type    => :string,
+      :default => false,
+      :desc    => 'Additional paths to auto-reload when files change'
 
     # Start the server
     def server
@@ -56,7 +60,8 @@ module Middleman::Cli
         :environment       => options["environment"],
         :debug             => options["verbose"],
         :instrumenting     => options["instrument"],
-        :"disable-watcher" => options["disable-watcher"]
+        :"disable-watcher" => options["disable-watcher"],
+        :reload_paths      => options["reload_paths"]
       }
 
       puts "== The Middleman is loading"

data/lib/middleman-core/preview_server.rb

@@ -9,43 +9,26 @@ module Middleman
     DEFAULT_PORT = 4567
 
     class << self
-      attr_reader :app
+      attr_reader :app, :port
       delegate :logger, :to => :app
 
       # Start an instance of Middleman::Application
       # @return [void]
-      def start(options={})
-        @app = ::Middleman::Application.server.inst do
-          if options[:environment]
-            set :environment, options[:environment].to_sym
-          end
-
-          logger(options[:debug] ? 0 : 1, options[:instrumenting] || false)
-        end
-
-        port = options[:port] || DEFAULT_PORT
+      def start(opts={})
+        @options = opts
+        @port = @options[:port] || DEFAULT_PORT
 
+        mount_instance
         logger.info "== The Middleman is standing watch on port #{port}"
 
-        @webrick ||= setup_webrick(
-          options[:host]  || "0.0.0.0",
-          port,
-          options[:debug] || false
-        )
-
-        mount_instance(app)
-
-        start_file_watcher unless options[:"disable-watcher"]
-
         @initialized ||= false
         unless @initialized
           @initialized = true
 
           register_signal_handlers unless ::Middleman::WINDOWS
 
-          # Save the last-used options so it may be re-used when
+          # Save the last-used @options so it may be re-used when
           # reloading later on.
-          @last_options = options
           ::Middleman::Profiling.report("server_start")
 
           @webrick.start
@@ -66,8 +49,12 @@ module Middleman
       # Simply stop, then start the server
       # @return [void]
       def reload
-        stop
-        start @last_options
+        logger.info "== The Middleman is reloading"
+        
+        unmount_instance
+        mount_instance
+        
+        logger.info "== The Middleman is standing watch on port #{port}"
       end
 
       # Stop the current instance, exit Webrick
@@ -78,47 +65,47 @@ module Middleman
       end
 
     private
+      def new_app
+        opts = @options
+        @app =::Middleman::Application.server.inst do
+          if opts[:environment]
+            set :environment, opts[:environment].to_sym
+          end
+          
+          logger(opts[:debug] ? 0 : 1, opts[:instrumenting] || false)
+        end
+      end
 
       def start_file_watcher
-        # Watcher Library
-        require "listen"
-
-        return if @listener
-
-        @listener = Listen.to(Dir.pwd, :relative_paths => true)
-
+        return if @options[:"disable-watcher"]
+        
+        first_run = !@listener
+        
+        if first_run
+          # Watcher Library
+          require "listen"
+          @listener = Listen.to(Dir.pwd, :relative_paths => true)
+        end
+        
         @listener.change do |modified, added, removed|
           added_and_modified = (modified + added)
 
-          unless added_and_modified.empty?
-            # See if the changed file is config.rb or lib/*.rb
-            if needs_to_reload?(added_and_modified)
-              reload
-              return
-            end
-
-            # Otherwise forward to Middleman
+          # See if the changed file is config.rb or lib/*.rb
+          if needs_to_reload?(added_and_modified) || needs_to_reload?(removed)
+            reload
+          else
             added_and_modified.each do |path|
-              @app.files.did_change(path)
+              app.files.did_change(path)
             end
-          end
 
-          unless removed.empty?
-            # See if the changed file is config.rb or lib/*.rb
-            if needs_to_reload?(removed)
-              reload
-              return
-            end
-
-            # Otherwise forward to Middleman
             removed.each do |path|
-              @app.files.did_delete(path)
+              app.files.did_delete(path)
             end
           end
         end
 
         # Don't block this thread
-        @listener.start(false)
+        @listener.start(false) if first_run
       end
 
       # Trap the interupt signal and shut down smoothly
@@ -131,13 +118,12 @@ module Middleman
 
       # Initialize webrick
       # @return [void]
-      def setup_webrick(host, port, is_logging)
+      def setup_webrick(host, is_logging)
         @host = host
-        @port = port
 
         http_opts = {
           :BindAddress => @host,
-          :Port        => @port,
+          :Port        => port,
           :AccessLog   => []
         }
 
@@ -147,15 +133,28 @@ module Middleman
           http_opts[:Logger] = ::WEBrick::Log.new(nil, 0)
         end
 
-        ::WEBrick::HTTPServer.new(http_opts)
+        begin
+          ::WEBrick::HTTPServer.new(http_opts)
+        rescue Errno::EADDRINUSE => e
+          logger.error "== Port #{port} is unavailable. Either close the instance of Middleman already running on #{port} or start this Middleman on a new port with: --port=#{port.to_i+1}"
+          exit(1)
+        end
       end
 
       # Attach a new Middleman::Application instance
       # @param [Middleman::Application] app
       # @return [void]
-      def mount_instance(app)
-        @app = app
-        @webrick.mount "/", ::Rack::Handler::WEBrick, @app.class.to_rack_app
+      def mount_instance
+        @app = new_app
+        @webrick ||= setup_webrick(
+          @options[:host]  || "0.0.0.0",
+          @options[:debug] || false
+        )
+        
+        @app = new_app
+        start_file_watcher
+          
+        @webrick.mount "/", ::Rack::Handler::WEBrick, app.class.to_rack_app
       end
 
       # Detach the current Middleman::Application instance
@@ -169,8 +168,22 @@ module Middleman
       # @param [Array<String>] paths Array of paths to check
       # @return [Boolean] Whether the server needs to reload
       def needs_to_reload?(paths)
+        match_against = [
+          %r{^config\.rb},
+          %r{^lib/^[^\.](.*)\.rb$},
+          %r{^helpers/^[^\.](.*)_helper\.rb$}
+        ]
+        
+        if @options[:reload_paths]
+          @options[:reload_paths].split(',').each do |part|
+            match_against << %r{^#{part}}
+          end
+        end
+        
         paths.any? do |path|
-          path.match(%{^config\.rb}) || path.match(%r{^lib/^[^\.](.*)\.rb$}) || path.match(%r{^helpers/^[^\.](.*)_helper\.rb$})
+          match_against.any? do |matcher|
+            path.match(matcher)
+          end
         end
       end
     end

data/lib/middleman-core/version.rb

@@ -1,5 +1,5 @@
 module Middleman
   # Current Version
   # @return [String]
-  VERSION = '3.0.2' unless const_defined?(:VERSION)
+  VERSION = '3.0.4' unless const_defined?(:VERSION)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: middleman-core
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 3.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-09-05 00:00:00.000000000 Z
+date: 2012-09-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -585,7 +585,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1472293550739068315
+      hash: -1143395816558407601
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -594,7 +594,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1472293550739068315
+      hash: -1143395816558407601
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23