bundler 1.10.6 → 1.11.0.pre.1

data/lib/bundler/cli/binstubs.rb

@@ -20,7 +20,7 @@ module Bundler
       end
 
       gems.each do |gem_name|
-        spec = installer.specs.find{|s| s.name == gem_name }
+        spec = installer.specs.find {|s| s.name == gem_name }
         unless spec
           raise GemNotFound, Bundler::CLI::Common.gem_not_found_message(
             gem_name, Bundler.definition.specs)
@@ -33,6 +33,5 @@ module Bundler
         end
       end
     end
-
   end
 end

data/lib/bundler/cli/cache.rb

@@ -30,6 +30,5 @@ module Bundler
           "on Bundler 2.0."
       end
     end
-
   end
 end

data/lib/bundler/cli/check.rb

@@ -6,30 +6,32 @@ module Bundler
     end
 
     def run
-      Bundler.settings[:path] = File.expand_path(options[:path]) if options[:path]
+      if options[:path]
+        Bundler.settings[:path] = File.expand_path(options[:path])
+        Bundler.settings[:disable_shared_gems] = "1"
+      end
       begin
         definition = Bundler.definition
         definition.validate_ruby!
         not_installed = definition.missing_specs
       rescue GemNotFound, VersionConflict
         Bundler.ui.error "Bundler can't satisfy your Gemfile's dependencies."
-        Bundler.ui.warn  "Install missing gems with `bundle install`."
+        Bundler.ui.warn "Install missing gems with `bundle install`."
         exit 1
       end
 
       if not_installed.any?
         Bundler.ui.error "The following gems are missing"
-        not_installed.each { |s| Bundler.ui.error " * #{s.name} (#{s.version})" }
+        not_installed.each {|s| Bundler.ui.error " * #{s.name} (#{s.version})" }
         Bundler.ui.warn "Install missing gems with `bundle install`"
         exit 1
       elsif !Bundler.default_lockfile.exist? && Bundler.settings[:frozen]
-        Bundler.ui.error "This bundle has been frozen, but there is no Gemfile.lock present"
+        Bundler.ui.error "This bundle has been frozen, but there is no #{Bundler.default_lockfile.relative_path_from(SharedHelpers.pwd)} present"
         exit 1
       else
         Bundler.load.lock(:preserve_bundled_with => true) unless options[:"dry-run"]
         Bundler.ui.info "The Gemfile's dependencies are satisfied"
       end
     end
-
   end
 end

data/lib/bundler/cli/clean.rb

@@ -21,6 +21,5 @@ module Bundler
         exit 1
       end
     end
-
   end
 end

data/lib/bundler/cli/common.rb

@@ -3,7 +3,7 @@ module Bundler
     def self.without_groups_message
       groups = Bundler.settings.without
       group_list = [groups[0...-1].join(", "), groups[-1..-1]].
-        reject{|s| s.to_s.empty? }.join(" and ")
+        reject {|s| s.to_s.empty? }.join(" and ")
       group_str = (groups.size == 1) ? "group" : "groups"
       "Gems in the #{group_str} #{group_list} were not installed."
     end
@@ -30,27 +30,26 @@ module Bundler
     end
 
     def self.ask_for_spec_from(specs)
-      if !$stdout.tty? && ENV['BUNDLE_SPEC_RUN'].nil?
+      if !$stdout.tty? && ENV["BUNDLE_SPEC_RUN"].nil?
         raise GemNotFound, gem_not_found_message(name, Bundler.definition.dependencies)
       end
 
       specs.each_with_index do |spec, index|
         Bundler.ui.info "#{index.succ} : #{spec.name}", true
       end
-      Bundler.ui.info '0 : - exit -', true
+      Bundler.ui.info "0 : - exit -", true
 
-      num = Bundler.ui.ask('> ').to_i
+      num = Bundler.ui.ask("> ").to_i
       num > 0 ? specs[num - 1] : nil
     end
 
     def self.gem_not_found_message(missing_gem_name, alternatives)
-      require 'bundler/similarity_detector'
+      require "bundler/similarity_detector"
       message = "Could not find gem '#{missing_gem_name}'."
-      alternate_names = alternatives.map { |a| a.respond_to?(:name) ? a.name : a }
+      alternate_names = alternatives.map {|a| a.respond_to?(:name) ? a.name : a }
       suggestions = SimilarityDetector.new(alternate_names).similar_word_list(missing_gem_name)
       message += "\nDid you mean #{suggestions}?" if suggestions
       message
     end
-
   end
 end

data/lib/bundler/cli/config.rb

@@ -1,87 +1,100 @@
 module Bundler
   class CLI::Config
-    attr_reader :options, :thor
+    attr_reader :name, :options, :scope, :thor
     attr_accessor :args
 
     def initialize(options, args, thor)
       @options = options
       @args = args
       @thor = thor
+      @name = peek = args.shift
+      @scope = "global"
+      if peek && peek =~ /^\-\-/
+        @name = args.shift
+        @scope = $'
+      end
     end
 
     def run
-      peek = args.shift
+      unless name
+        confirm_all
+        return
+      end
 
-      if peek && peek =~ /^\-\-/
-        name, scope = args.shift, $'
-      else
-        name, scope = peek, "global"
+      unless valid_scope?(scope)
+        Bundler.ui.error "Invalid scope --#{scope} given. Please use --local or --global."
+        exit 1
       end
 
-      unless name
-        Bundler.ui.confirm "Settings are listed in order of priority. The top value will be used.\n"
+      if scope == "delete"
+        Bundler.settings.set_local(name, nil)
+        Bundler.settings.set_global(name, nil)
+        return
+      end
 
-        Bundler.settings.all.each do |setting|
-          Bundler.ui.confirm "#{setting}"
-          thor.with_padding do
-            Bundler.settings.pretty_values_for(setting).each do |line|
-              Bundler.ui.info line
-            end
-          end
-          Bundler.ui.confirm ""
-        end
+      if args.empty?
+        confirm(name)
         return
       end
 
-      case scope
-      when "delete"
-        Bundler.settings.set_local(name, nil)
-        Bundler.settings.set_global(name, nil)
-      when "local", "global"
-        if args.empty?
-          Bundler.ui.confirm "Settings for `#{name}` in order of priority. The top value will be used"
-          thor.with_padding do
-            Bundler.settings.pretty_values_for(name).each { |line| Bundler.ui.info line }
-          end
-          return
-        end
+      Bundler.ui.info(message) if message
+      Bundler.settings.send("set_#{scope}", name, new_value)
+    end
 
-        new_value = args.join(" ")
-        locations = Bundler.settings.locations(name)
+  private
 
-        if scope == "global"
-          if locations[:local]
-            Bundler.ui.info "Your application has set #{name} to #{locations[:local].inspect}. " \
-              "This will override the global value you are currently setting"
-          end
+    def confirm_all
+      Bundler.ui.confirm "Settings are listed in order of priority. The top value will be used.\n"
+      Bundler.settings.all.each do |setting|
+        Bundler.ui.confirm "#{setting}"
+        show_pretty_values_for(setting)
+        Bundler.ui.confirm ""
+      end
+    end
 
-          if locations[:env]
-            Bundler.ui.info "You have a bundler environment variable for #{name} set to " \
-              "#{locations[:env].inspect}. This will take precedence over the global value you are setting"
-          end
+    def confirm(name)
+      Bundler.ui.confirm "Settings for `#{name}` in order of priority. The top value will be used"
+      show_pretty_values_for(name)
+    end
 
-          if locations[:global] && locations[:global] != new_value
-            Bundler.ui.info "You are replacing the current global value of #{name}, which is currently " \
-              "#{locations[:global].inspect}"
-          end
-        end
+    def new_value
+      pathname = Pathname.new(args.join(" "))
+      if name.match(/\Alocal\./) && pathname.directory?
+        pathname.expand_path.to_s
+      else
+        args.join(" ")
+      end
+    end
 
-        if scope == "local" && locations[:local] != new_value
-          Bundler.ui.info "You are replacing the current local value of #{name}, which is currently " \
-            "#{locations[:local].inspect}"
+    def message
+      locations = Bundler.settings.locations(name)
+      if scope == "global"
+        if locations[:local]
+          "Your application has set #{name} to #{locations[:local].inspect}. " \
+            "This will override the global value you are currently setting"
+        elsif locations[:env]
+          "You have a bundler environment variable for #{name} set to " \
+            "#{locations[:env].inspect}. This will take precedence over the global value you are setting"
+        elsif locations[:global] && locations[:global] != args.join(" ")
+          "You are replacing the current global value of #{name}, which is currently " \
+            "#{locations[:global].inspect}"
         end
+      elsif scope == "local" && locations[:local] != args.join(" ")
+        "You are replacing the current local value of #{name}, which is currently " \
+          "#{locations[:local].inspect}"
+      end
+    end
 
-        if name.match(/\Alocal\./)
-          pathname = Pathname.new(args.join(" "))
-          new_value = pathname.expand_path.to_s if pathname.directory?
+    def show_pretty_values_for(setting)
+      thor.with_padding do
+        Bundler.settings.pretty_values_for(setting).each do |line|
+          Bundler.ui.info line
         end
-
-        Bundler.settings.send("set_#{scope}", name, new_value)
-      else
-        Bundler.ui.error "Invalid scope --#{scope} given. Please use --local or --global."
-        exit 1
       end
     end
 
+    def valid_scope?(scope)
+      %w(delete local global).include?(scope)
+    end
   end
 end

data/lib/bundler/cli/console.rb

@@ -7,10 +7,10 @@ module Bundler
     end
 
     def run
-      group ? Bundler.require(:default, *(group.split.map! {|g| g.to_sym })) : Bundler.require
+      group ? Bundler.require(:default, *(group.split.map!(&:to_sym))) : Bundler.require
       ARGV.clear
 
-      console = get_console(Bundler.settings[:console] || 'irb')
+      console = get_console(Bundler.settings[:console] || "irb")
       console.start
     end
 
@@ -18,21 +18,21 @@ module Bundler
       require name
       get_constant(name)
     rescue LoadError
-      Bundler.ui.error "Couldn't load console #{name}"
-      get_constant('irb')
+      Bundler.ui.error "Couldn't load console #{name}, falling back to irb"
+      require "irb"
+      get_constant("irb")
     end
 
     def get_constant(name)
       const_name = {
-        'pry'  => :Pry,
-        'ripl' => :Ripl,
-        'irb'  => :IRB,
+        "pry"  => :Pry,
+        "ripl" => :Ripl,
+        "irb"  => :IRB,
       }[name]
       Object.const_get(const_name)
     rescue NameError
       Bundler.ui.error "Could not find constant #{const_name}"
       exit 1
     end
-
   end
 end

data/lib/bundler/cli/exec.rb

@@ -1,3 +1,5 @@
+require "bundler/current_ruby"
+
 module Bundler
   class CLI::Exec
     attr_reader :options, :args, :cmd
@@ -7,7 +9,7 @@ module Bundler
       @cmd = args.shift
       @args = args
 
-      if RUBY_VERSION >= "2.0"
+      if Bundler.current_ruby.ruby_2? && !Bundler.current_ruby.jruby?
         @args << { :close_others => !options.keep_file_descriptors? }
       elsif options.keep_file_descriptors?
         Bundler.ui.warn "Ruby version #{RUBY_VERSION} defaults to keeping non-standard file descriptors on Kernel#exec."
@@ -38,13 +40,12 @@ module Bundler
     rescue Errno::ENOENT
       Bundler.ui = ui
       Bundler.ui.error "bundler: command not found: #{cmd}"
-      Bundler.ui.warn  "Install missing gem executables with `bundle install`"
+      Bundler.ui.warn "Install missing gem executables with `bundle install`"
       exit 127
     rescue ArgumentError
       Bundler.ui = ui
       Bundler.ui.error "bundler: exec needs a command to run"
       exit 128
     end
-
   end
 end

data/lib/bundler/cli/gem.rb

@@ -1,7 +1,12 @@
-require 'pathname'
+require "pathname"
 
 module Bundler
   class CLI::Gem
+    TEST_FRAMEWORK_VERSIONS = {
+      "rspec" => "3.0",
+      "minitest" => "5.0"
+    }
+
     attr_reader :options, :gem_name, :thor, :name, :target
 
     def initialize(options, gem_name, thor)
@@ -10,7 +15,7 @@ module Bundler
       @thor = thor
 
       @name = @gem_name
-      @target = Pathname.pwd.join(gem_name)
+      @target = SharedHelpers.pwd.join(gem_name)
 
       validate_ext_name if options[:ext]
     end
@@ -18,10 +23,11 @@ module Bundler
     def run
       Bundler.ui.confirm "Creating gem '#{name}'..."
 
-      underscored_name = name.tr('-', '_')
-      namespaced_path = name.tr('-', '/')
-      constant_name = name.gsub(/-[_-]*(?![_-]|$)/){ '::' }.gsub(/([_-]+|(::)|^)(.|$)/){ $2.to_s + $3.upcase }
-      constant_array = constant_name.split('::')
+      underscored_name = name.tr("-", "_")
+      namespaced_path = name.tr("-", "/")
+      constant_name = name.gsub(/-[_-]*(?![_-]|$)/) { "::" }.gsub(/([_-]+|(::)|^)(.|$)/) { $2.to_s + $3.upcase }
+      constant_array = constant_name.split("::")
+      test_task = options[:test] == "minitest" ? "test" : "spec"
 
       git_user_name = `git config user.name`.chomp
       git_user_email = `git config user.email`.chomp
@@ -36,6 +42,7 @@ module Bundler
         :author           => git_user_name.empty? ? "TODO: Write your name" : git_user_name,
         :email            => git_user_email.empty? ? "TODO: Write your email address" : git_user_email,
         :test             => options[:test],
+        :test_task        => test_task,
         :ext              => options[:ext],
         :bin              => options[:bin],
         :bundler_version  => bundler_dependency_version
@@ -54,45 +61,25 @@ module Bundler
         "bin/setup.tt" => "bin/setup"
       }
 
-      executables = %w[
+      executables = %w(
         bin/console
         bin/setup
-      ]
-
-      if ask_and_set(:coc, "Do you want to include a code of conduct in gems you generate?",
-          "Codes of conduct can increase contributions to your project by contributors who " \
-          "prefer collaborative, safe spaces. You can read more about the code of conduct at " \
-          "contributor-covenant.org. Having a code of conduct means agreeing to the responsibility " \
-          "of enforcing it, so be sure that you are prepared to do that. For suggestions about " \
-          "how to enforce codes of conduct, see bit.ly/coc-enforcement."
-        )
-        config[:coc] = true
-        Bundler.ui.info "Code of conduct enabled in config"
-        templates.merge!("CODE_OF_CONDUCT.md.tt" => "CODE_OF_CONDUCT.md")
-      end
-
-      if ask_and_set(:mit, "Do you want to license your code permissively under the MIT license?",
-          "This means that any other developer or company will be legally allowed to use your code " \
-          "for free as long as they admit you created it. You can read more about the MIT license " \
-          "at choosealicense.com/licenses/mit."
-        )
-        config[:mit] = true
-        Bundler.ui.info "MIT License enabled in config"
-        templates.merge!("LICENSE.txt.tt" => "LICENSE.txt")
-      end
+      )
 
       if test_framework = ask_and_set_test_framework
         config[:test] = test_framework
+        config[:test_framework_version] = TEST_FRAMEWORK_VERSIONS[test_framework]
+
         templates.merge!(".travis.yml.tt" => ".travis.yml")
 
         case test_framework
-        when 'rspec'
+        when "rspec"
           templates.merge!(
             "rspec.tt" => ".rspec",
             "spec/spec_helper.rb.tt" => "spec/spec_helper.rb",
             "spec/newgem_spec.rb.tt" => "spec/#{namespaced_path}_spec.rb"
           )
-        when 'minitest'
+        when "minitest"
           templates.merge!(
             "test/test_helper.rb.tt" => "test/test_helper.rb",
             "test/newgem_test.rb.tt" => "test/#{namespaced_path}_test.rb"
@@ -100,6 +87,28 @@ module Bundler
         end
       end
 
+      if ask_and_set(:mit, "Do you want to license your code permissively under the MIT license?",
+        "This means that any other developer or company will be legally allowed to use your code " \
+        "for free as long as they admit you created it. You can read more about the MIT license " \
+        "at http://choosealicense.com/licenses/mit.")
+        config[:mit] = true
+        Bundler.ui.info "MIT License enabled in config"
+        templates.merge!("LICENSE.txt.tt" => "LICENSE.txt")
+      end
+
+      if ask_and_set(:coc, "Do you want to include a code of conduct in gems you generate?",
+        "Codes of conduct can increase contributions to your project by contributors who " \
+        "prefer collaborative, safe spaces. You can read more about the code of conduct at " \
+        "contributor-covenant.org. Having a code of conduct means agreeing to the responsibility " \
+        "of enforcing it, so be sure that you are prepared to do that. Be sure that your email " \
+        "address is specified as a contact in the generated code of conduct so that people know " \
+        "who to contact in case of a violation. For suggestions about " \
+        "how to enforce codes of conduct, see http://bit.ly/coc-enforcement.")
+        config[:coc] = true
+        Bundler.ui.info "Code of conduct enabled in config"
+        templates.merge!("CODE_OF_CONDUCT.md.tt" => "CODE_OF_CONDUCT.md")
+      end
+
       templates.merge!("exe/newgem.tt" => "exe/#{name}") if options[:bin]
 
       if options[:ext]
@@ -121,7 +130,10 @@ module Bundler
       end
 
       Bundler.ui.info "Initializing git repo in #{target}"
-      Dir.chdir(target) { `git init`; `git add .` }
+      Dir.chdir(target) do
+        `git init`
+        `git add .`
+      end
 
       if options[:edit]
         # Open gemspec in editor
@@ -129,10 +141,10 @@ module Bundler
       end
     end
 
-    private
+  private
 
     def resolve_name(name)
-      Pathname.pwd.join(name).basename.to_s
+      SharedHelpers.pwd.join(name).basename.to_s
     end
 
     def ask_and_set(key, header, message)
@@ -148,7 +160,7 @@ module Bundler
     end
 
     def validate_ext_name
-      return unless gem_name.index('-')
+      return unless gem_name.index("-")
 
       Bundler.ui.error "You have specified a gem name which does not conform to the \n" \
                        "naming guidelines for C extensions. For more information, \n" \
@@ -181,11 +193,11 @@ module Bundler
     def bundler_dependency_version
       v = Gem::Version.new(Bundler::VERSION)
       req = v.segments[0..1]
-      req << 'a' if v.prerelease?
+      req << "a" if v.prerelease?
       req.join(".")
     end
 
-    def ensure_safe_gem_name name, constant_array
+    def ensure_safe_gem_name(name, constant_array)
       if name =~ /^\d/
         Bundler.ui.error "Invalid gem name #{name} Please give a name which does not start with numbers."
         exit 1
@@ -194,6 +206,5 @@ module Bundler
         exit 1
       end
     end
-
   end
 end