No more free lunch

actionpack/lib/action_controller.rb

@@ -30,6 +30,7 @@ rescue LoadError
     require 'active_support'
   end
 end
+require 'active_support/core/all'
 
 require File.join(File.dirname(__FILE__), "action_pack")
 

actionpack/lib/action_dispatch.rb

@@ -30,6 +30,7 @@ rescue LoadError
     require 'active_support'
   end
 end
+require 'active_support/core/all'
 
 $:.unshift "#{File.dirname(__FILE__)}/action_dispatch/vendor/rack-1.0"
 begin

actionpack/lib/action_view.rb

@@ -30,6 +30,7 @@ rescue LoadError
     require 'active_support'
   end
 end
+require 'active_support/core/all'
 
 require File.join(File.dirname(__FILE__), "action_pack")
 

activemodel/lib/active_model/core.rb

@@ -1,7 +1,12 @@
-# This file is required by each major ActiveModel component for the core requirements.  This allows you to
-# load individual pieces of ActiveModel as needed.
-$LOAD_PATH << File.join(File.dirname(__FILE__), '..', '..', '..', 'activesupport', 'lib')
+begin
+  require 'active_support'
+rescue LoadError
+  activesupport_path = "#{File.dirname(__FILE__)}/../../activesupport/lib"
+  if File.directory?(activesupport_path)
+    $:.unshift activesupport_path
+    require 'active_support'
+  end
+end
 
-# premature optimization?
 # So far, we only need the string inflections and not the rest of ActiveSupport.
-require 'active_support/inflector'
+require 'active_support/inflector'

activerecord/lib/active_record.rb

@@ -30,6 +30,7 @@ rescue LoadError
     require 'active_support'
   end
 end
+require 'active_support/core/all'
 
 module ActiveRecord
   # TODO: Review explicit loads to see if they will automatically be handled by the initilizer.

activeresource/lib/active_resource.rb

@@ -30,6 +30,7 @@ rescue LoadError
     require 'active_support'
   end
 end
+require 'active_support/core/all'
 
 require 'active_resource/formats'
 require 'active_resource/base'

activesupport/CHANGELOG

@@ -1,5 +1,7 @@
 *Edge*
 
+* require 'active_support' no longer orders the whole menu of core extensions. Ask for just what you need: e.g. require 'active_support/core/time' to use timezones, durations, and stdlib date/time extensions. [Jeremy Kemper]
+
 * Removed rarely-used DRb cache store.  [Jeremy Kemper]
 
 * TimeWithZone.name returns 'Time', to further thwart type checking [Geoff Buesing]

activesupport/lib/active_support.rb

@@ -50,10 +50,7 @@ module ActiveSupport
   autoload :XmlMini, 'active_support/xml_mini'
 end
 
-require 'active_support/core/all'
-
 require 'active_support/vendor'
-require 'active_support/core_ext'
 require 'active_support/dependencies'
 require 'active_support/json'
 

activesupport/lib/active_support/core/all.rb

@@ -1,3 +1,4 @@
+require 'active_support/core_ext'
 require 'active_support/core'
 Dir["#{File.dirname(__FILE__)}/*.rb"].sort.each do |path|
   require "active_support/core/#{File.basename(path, '.rb')}"

activesupport/lib/active_support/core_ext/kernel/requires.rb

@@ -1,3 +1,5 @@
+require 'active_support/core_ext/kernel/reporting'
+
 module Kernel
   # Require a library with fallback to RubyGems.  Warnings during library
   # loading are silenced to increase signal/noise for application warnings.
@@ -21,4 +23,4 @@ module Kernel
       end
     end
   end
-end
+end

railties/Rakefile

@@ -29,12 +29,12 @@ task :default => :test
 task :test do 
   Dir['test/**/*_test.rb'].all? do |file|
     ruby = File.join(*RbConfig::CONFIG.values_at('bindir', 'RUBY_INSTALL_NAME'))
-    system(ruby, '-Itest', file)
+    system(ruby, '-Itest', "-I#{File.dirname(__FILE__)}/../activesupport/lib", file)
   end or raise "Failures"
 end
 
 Rake::TestTask.new("regular_test") do |t|
-  t.libs << 'test'
+  t.libs << 'test' << "#{File.dirname(__FILE__)}/../activesupport/lib"
   t.pattern = 'test/**/*_test.rb'
   t.warning = true
   t.verbose = true

railties/lib/rails/plugin.rb

@@ -1,3 +1,5 @@
+require 'active_support/core_ext/kernel/reporting'
+
 module Rails
   # The Plugin class should be an object which provides the following methods:
   #
@@ -35,10 +37,10 @@ module Rails
     def load_paths
       report_nonexistant_or_empty_plugin! unless valid?
       
-      returning [] do |load_paths|
-        load_paths << lib_path  if has_lib_directory?
-        load_paths << app_paths if has_app_directory?
-      end.flatten
+      load_paths = []
+      load_paths << lib_path  if has_lib_directory?
+      load_paths << app_paths if has_app_directory?
+      load_paths.flatten
     end
     
     # Evaluates a plugin's init.rb file.

railties/lib/rails/rack/metal.rb

@@ -1,4 +1,5 @@
 require 'active_support/ordered_hash'
+require 'active_support/core_ext/class/attribute_accessors'
 
 module Rails
   module Rack

railties/lib/rails_generator.rb

@@ -21,16 +21,18 @@
 # WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #++
 
-$:.unshift(File.dirname(__FILE__))
-$:.unshift(File.dirname(__FILE__) + "/../../activesupport/lib")
-
 begin
-  require 'active_support'  
+  require 'active_support'
 rescue LoadError
-  require 'rubygems'
-  gem 'activesupport'
+  activesupport_path = "#{File.dirname(__FILE__)}/../../activesupport/lib"
+  if File.directory?(activesupport_path)
+    $:.unshift activesupport_path
+    require 'active_support'
+  end
 end
+require 'active_support/core/all'
 
+$:.unshift(File.dirname(__FILE__))
 require 'rails_generator/base'
 require 'rails_generator/lookup'
 require 'rails_generator/commands'