bundler 1.1.rc.3 → 1.1.rc.5

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+## 1.1.rc.5 (Dec 14, 2011)
+
+Bugfixes:
+
+  - Fix ASCII encoding errors with gem (rerelease with ruby 1.8)
+
+## 1.1.rc.4 (Dec 14, 2011)
+
+Features:
+
+  - `bundle viz` has the option to output a DOT file instead of a PNG (@hirochachacha, #683)
+
+Bugfixes:
+
+  - Ensure binstubs generated when using --standalone point to the standalonde bundle (@cowboyd, #1588)
+  - fix `bundle viz` (@hirochachacha, #1586)
+
 ## 1.1.rc.3 (Dec 8, 2011)
 
 Bugfixes:

data/lib/bundler/cli.rb

@@ -507,16 +507,17 @@ module Bundler
       Viz requires the ruby-graphviz gem (and its dependencies).
       The associated gems must also be installed via 'bundle install'.
     D
-    method_option :file, :type => :string, :default => 'gem_graph.png', :aliases => '-f', :banner => "The name to use for the generated png file."
+    method_option :file, :type => :string, :default => 'gem_graph', :aliases => '-f', :banner => "The name to use for the generated file. see format option"
     method_option :version, :type => :boolean, :default => false, :aliases => '-v', :banner => "Set to show each gem version."
     method_option :requirements, :type => :boolean, :default => false, :aliases => '-r', :banner => "Set to show the version of each required dependency."
+    method_option :format, :type => :string, :default => "png", :aliases => '-F', :banner => "This is output format option. Supported format is png, jpg, svg, dot ..."
     def viz
       output_file = File.expand_path(options[:file])
-      graph = Graph.new( Bundler.load )
+      output_format = options[:format]
+      graph = Graph.new(Bundler.load, output_file, options[:version], options[:requirements], options[:format])
 
       begin
-        graph.viz(output_file, options[:version], options[:requirements])
-        Bundler.ui.info output_file
+        graph.viz
       rescue LoadError => e
         Bundler.ui.error e.inspect
         Bundler.ui.warn "Make sure you have the graphviz ruby gem. You can install it with:"

data/lib/bundler/graph.rb

@@ -1,130 +1,148 @@
+require 'set'
 module Bundler
   class Graph
+    GRAPH_NAME = :Gemfile
 
-    USER_OPTIONS = {:style => 'filled', :fillcolor => '#B9B9D5'}.freeze
+    def initialize(env, output_file, show_version = false, show_requirements = false, output_format = "png")
+      @env               = env
+      @output_file       = output_file
+      @show_version      = show_version
+      @show_requirements = show_requirements
+      @output_format     = output_format
 
-    def initialize(env)
-      @env = env
-    end
-
-    def nodes
-      populate
-      @nodes
-    end
+      @groups            = []
+      @relations         = Hash.new {|h, k| h[k] = Set.new}
+      @node_options      = {}
+      @edge_options      = {}
 
-    def groups
-      populate
-      @groups
+      _populate_relations
     end
 
-    def viz(output_file, show_gem_versions = false, show_dependency_requirements = false)
-      require 'graphviz'
-      populate
+    attr_reader :groups, :relations, :node_options, :edge_options, :output_file, :output_format
 
-      graph_viz = GraphViz::new('Gemfile', {:concentrate => true, :normalize => true, :nodesep => 0.55})
-      graph_viz.edge[:fontname] = graph_viz.node[:fontname] = 'Arial, Helvetica, SansSerif'
-      graph_viz.edge[:fontsize] = 12
-
-      viz_nodes = {}
+    def viz
+      GraphVizClient.new(self).run
+    end
 
-      # populate all of the nodes
-      nodes.each do |name, node|
-        label = name.dup
-        label << "\n#{node.version}" if show_gem_versions
-        options = { :label => label }
-        options.merge!( USER_OPTIONS ) if node.is_user
-        viz_nodes[name] = graph_viz.add_node( name, options )
-      end
+    private
 
-      group_nodes = {}
-      @groups.each do |name, dependencies|
-        group_nodes[name] = graph_viz.add_node(name.to_s, { :shape => 'box3d', :fontsize => 16 }.merge(USER_OPTIONS))
-        dependencies.each do |dependency|
-          options = { :weight => 2 }
-          if show_dependency_requirements && (dependency.requirement.to_s != ">= 0")
-            options[:label] = dependency.requirement.to_s
+    def _populate_relations
+      relations = Hash.new {|h, k| h[k] = Set.new}
+      parent_dependencies = _groups.values.to_set.flatten
+      while true
+        if parent_dependencies.empty?
+          break
+        else
+          tmp = Set.new
+          parent_dependencies.each do |dependency|
+            child_dependencies = dependency.to_spec.runtime_dependencies.to_set
+            relations[dependency.name] += child_dependencies.to_set
+            @relations[dependency.name] += child_dependencies.map(&:name).to_set
+            tmp += child_dependencies
+
+            @node_options[dependency.name] = {:label => _make_label(dependency, :node)}
+            child_dependencies.each do |c_dependency|
+              @edge_options["#{dependency.name}_#{c_dependency.name}"] = {:label => _make_label(c_dependency, :edge)}
+            end
           end
-          graph_viz.add_edge( group_nodes[name], viz_nodes[dependency.name], options )
+          parent_dependencies = tmp
         end
       end
+      @relations
+    end
 
-      @groups.keys.select { |group| group != :default }.each do |group|
-        graph_viz.add_edge( group_nodes[group], group_nodes[:default], { :weight => 2 } )
-      end
-
-      viz_nodes.each do |name, node|
-        nodes[name].dependencies.each do |dependency|
-          options = { }
-          if nodes[dependency.name].is_user
-            options[:constraint] = false
-          end
-          if show_dependency_requirements && (dependency.requirement.to_s != ">= 0")
-            options[:label] = dependency.requirement.to_s
-          end
+    def _groups
+      relations = Hash.new {|h, k| h[k] = Set.new}
+      @env.current_dependencies.each do |dependency|
+        dependency.groups.each do |group|
+          relations[group.to_s].add(dependency)
+          @relations[group.to_s].add(dependency.name)
 
-          graph_viz.add_edge( node, viz_nodes[dependency.name], options )
+          @node_options[group.to_s] ||= {:label => _make_label(group, :node)}
+          @edge_options["#{group}_#{dependency.name}"] = {:label => _make_label(dependency, :edge)}
         end
       end
-
-      graph_viz.output( :png => output_file )
+      @groups = relations.keys
+      relations
     end
 
-    private
-
-    def populate
-      return if @populated
-
-      # hash of name => GraphNode
-      @nodes = {}
-      @groups = {}
-
-      # Populate @nodes
-      @env.specs.each { |spec| @nodes[spec.name] = GraphNode.new(spec.name, spec.version) }
-
-      # For gems in Gemfile, add details
-      @env.current_dependencies.each do |dependency|
-        next unless node = @nodes[dependency.name]
-        node.is_user = true
-
-        dependency.groups.each do |group|
-          if @groups.has_key? group
-            group = @groups[group]
-          else
-            group = @groups[group] = []
-          end
-          group << dependency
+    def _make_label(symbol_or_string_or_dependency, element_type)
+      case element_type.to_sym
+      when :node
+        if symbol_or_string_or_dependency.is_a?(Gem::Dependency)
+          label = symbol_or_string_or_dependency.name.dup
+          label << "\n#{symbol_or_string_or_dependency.to_spec.version.to_s}" if @show_version
+        else
+          label = symbol_or_string_or_dependency.to_s
+        end
+      when :edge
+        label = nil
+        if symbol_or_string_or_dependency.respond_to?(:requirements_list) && @show_requirements
+          tmp = symbol_or_string_or_dependency.requirements_list.join(", ")
+          label = tmp if tmp != ">= 0"
         end
+      else
+        raise ArgumentError, "2nd argument is invalid"
       end
+      label
+    end
 
-      # walk though a final time and add edges
-      @env.specs.each do |spec|
+    class GraphVizClient
+      def initialize(graph_instance)
+        @graph_name    = graph_instance.class::GRAPH_NAME
+        @groups        = graph_instance.groups
+        @relations     = graph_instance.relations
+        @node_options  = graph_instance.node_options
+        @edge_options  = graph_instance.edge_options
+        @output_file   = graph_instance.output_file
+        @output_format = graph_instance.output_format
+      end
 
-        from = @nodes[spec.name]
-        spec.runtime_dependencies.each do |dependency|
-          from.dependencies << dependency
+      def g
+        require 'graphviz'
+        @g ||= ::GraphViz.digraph(@graph_name, {:concentrate => true, :normalize => true, :nodesep => 0.55}) do |g|
+          g.edge[:weight]   = 2
+          g.edge[:fontname] = g.node[:fontname] = 'Arial, Helvetica, SansSerif'
+          g.edge[:fontsize] = 12
         end
-
       end
 
-      @nodes.freeze
-      @groups.freeze
-      @populated = true
-    end
-
-  end
+      def run
+        @groups.each do |group|
+          g.add_node(
+            group,
+            {:style     => 'filled',
+             :fillcolor => '#B9B9D5',
+             :shape     => "box3d",
+             :fontsize  => 16}.merge(@node_options[group])
+          )
+        end
 
-  # Add version info
-  class GraphNode
+        @relations.each do |parent, children|
+          children.each do |child|
+            if @groups.include?(parent)
+              g.add_node(child, {:style => 'filled', :fillcolor => '#B9B9D5'}.merge(@node_options[child]))
+              g.add_edge(parent, child, {:constraint => false}.merge(@edge_options["#{parent}_#{child}"]))
+            else
+              g.add_node(child, @node_options[child])
+              g.add_edge(parent, child, @edge_options["#{parent}_#{child}"])
+            end
+          end
+        end
 
-    def initialize(name, version)
-      @name = name
-      @version = version
-      @is_user = false
-      @dependencies = []
+        if @output_format.to_s == "debug"
+          $stdout.puts g.output :none => String
+          Bundler.ui.info "debugging bundle viz..."
+        else
+          begin
+            g.output @output_format.to_sym => "#{@output_file}.#{@output_format}"
+            Bundler.ui.info "#{@output_file}.#{@output_format}"
+          rescue ArgumentError => e
+            $stderr.puts "Unsupported output format. See Ruby-Graphviz/lib/graphviz/constants.rb"
+            raise e
+          end
+        end
+      end
     end
-
-    attr_reader :name, :dependencies, :version
-    attr_accessor :is_user
-
   end
 end

data/lib/bundler/installer.rb

@@ -53,7 +53,7 @@ module Bundler
       # the gem.
       Installer.post_install_messages = {}
       specs.each do |spec|
-        install_gem_from_spec(spec)
+        install_gem_from_spec(spec, options[:standalone])
       end
 
       lock
@@ -62,7 +62,7 @@ module Bundler
 
   private
 
-    def install_gem_from_spec(spec)
+    def install_gem_from_spec(spec, standalone = false)
       # Download the gem to get the spec, because some specs that are returned
       # by rubygems.org are broken and wrong.
       Bundler::Fetcher.fetch(spec) if spec.source.is_a?(Bundler::Source::Rubygems)
@@ -76,7 +76,10 @@ module Bundler
 
       # newline comes after installing, some gems say "with native extensions"
       Bundler.ui.info ""
-      generate_bundler_executable_stubs(spec) if Bundler.settings[:bin]
+      if Bundler.settings[:bin]
+        standalone ? generate_standalone_bundler_executable_stubs(spec) : generate_bundler_executable_stubs(spec)
+      end
+
       FileUtils.rm_rf(Bundler.tmp)
     rescue Exception => e
       # install hook failed
@@ -105,6 +108,21 @@ module Bundler
       end
     end
 
+    def generate_standalone_bundler_executable_stubs(spec)
+      bin_path = Bundler.bin_path
+      template = File.read(File.expand_path('../templates/Executable.standalone', __FILE__))
+      ruby_command = Thor::Util.ruby_command
+
+      spec.executables.each do |executable|
+        next if executable == "bundle"
+        standalone_path = Pathname(Bundler.settings[:path]).expand_path.relative_path_from(bin_path)
+        executable_path = Pathname(spec.full_gem_path).join(spec.bindir, executable).relative_path_from(bin_path)
+        File.open "#{bin_path}/#{executable}", 'w', 0755 do |f|
+          f.puts ERB.new(template, nil, '-').result(binding)
+        end
+      end
+    end
+
     def generate_standalone(groups)
       standalone_path = Bundler.settings[:path]
       bundler_path = File.join(standalone_path, "bundler")

data/lib/bundler/templates/Executable.standalone

@@ -0,0 +1,12 @@
+#!/usr/bin/env <%= Bundler.settings[:shebang] || RbConfig::CONFIG['ruby_install_name'] %>
+#
+# This file was generated by Bundler.
+#
+# The application '<%= executable %>' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+$:.unshift File.expand_path '../<%= standalone_path %>', __FILE__
+
+require 'bundler/setup'
+load File.expand_path '../<%= executable_path %>', __FILE__

data/lib/bundler/version.rb

@@ -2,5 +2,5 @@ module Bundler
   # We're doing this because we might write tests that deal
   # with other versions of bundler and we are unsure how to
   # handle this better.
-  VERSION = "1.1.rc.3" unless defined?(::Bundler::VERSION)
+  VERSION = "1.1.rc.5" unless defined?(::Bundler::VERSION)
 end

data/spec/install/gems/standalone_spec.rb

@@ -235,4 +235,26 @@ describe "bundle install --standalone" do
       end
     end
   end
+
+  describe "with --binstubs" do
+    before do
+      install_gemfile <<-G, :standalone => true, :binstubs => true
+        source "file://#{gem_repo1}"
+        gem "rails"
+      G
+    end
+
+    it "creates stubs that use the standalone load path" do
+      Dir.chdir(bundled_app) do
+        `bin/rails -v`.chomp.should eql "2.3.2"
+      end
+    end
+
+    it "creates stubs that can be executed from anywhere" do
+      require 'tmpdir'
+      Dir.chdir(Dir.tmpdir) do
+        `#{bundled_app}/bin/rails -v`.chomp.should eql "2.3.2"
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: bundler
 version: !ruby/object:Gem::Version 
-  hash: 15424251
+  hash: 15424247
   prerelease: true
   segments: 
   - 1
   - 1
   - rc
-  - 3
-  version: 1.1.rc.3
+  - 5
+  version: 1.1.rc.5
 platform: ruby
 authors: 
 - "Andr\xC3\xA9 Arko"
@@ -19,7 +19,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-12-08 00:00:00 -08:00
+date: 2011-12-14 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -104,6 +104,7 @@ files:
 - lib/bundler/source.rb
 - lib/bundler/spec_set.rb
 - lib/bundler/templates/Executable
+- lib/bundler/templates/Executable.standalone
 - lib/bundler/templates/Gemfile
 - lib/bundler/templates/newgem/Gemfile.tt
 - lib/bundler/templates/newgem/LICENSE.tt