git-multirepo 1.0.0.beta40 → 1.0.0.beta42

data/lib/multirepo/commands/open-command.rb

@@ -1,58 +1,56 @@
-require "os"
-
-require "multirepo/utility/console"
-require "multirepo/utility/utils"
-
-module MultiRepo
-  class OpenCommand < Command
-    self.command = "open"
-    self.summary = "Opens repositories in the OS's file explorer."
-
-    def self.options
-      [
-        ['[--main]', 'Open the main repository.'],
-        ['[--all]', 'Open the main repository and all dependencies.'],
-      ].concat(super)
-    end
-    
-    def initialize(argv)
-      @main_only = argv.flag?("main")
-      @all = argv.flag?("all")
-      super
-    end
-
-    def validate!
-      super
-      unless validate_only_one_flag(@main_only, @all)
-        help! "You can't provide more than one operation modifier (--deps, --main, etc.)"
-      end
-    end
-    
-    def run
-      super
-      ensure_in_work_tree
-      ensure_multirepo_enabled
-      
-      if @main_only
-        open_main
-      elsif @all
-        open_dependencies
-        open_main
-      else
-        open_dependencies
-      end
-    rescue MultiRepoException => e
-      Console.log_error(e.message)
-    end
-
-    def open_main
-      Utils.reveal_in_default_file_browser(".")
-    end
-
-    def open_dependencies
-      ConfigFile.new(".").load_entries.each do |entry|
-        Utils.reveal_in_default_file_browser(entry.repo.path)
-      end
-    end
-  end
+require "os"
+
+require "multirepo/utility/console"
+require "multirepo/utility/utils"
+
+module MultiRepo
+  class OpenCommand < Command
+    self.command = "open"
+    self.summary = "Opens repositories in the OS's file explorer."
+
+    def self.options
+      [
+        ['[--main]', 'Open the main repository.'],
+        ['[--all]', 'Open the main repository and all dependencies.'],
+      ].concat(super)
+    end
+    
+    def initialize(argv)
+      @all = argv.flag?("all")
+      @main_only = argv.flag?("main")
+      @deps_only = argv.flag?("deps")
+      super
+    end
+
+    def validate!
+      super
+      unless validate_only_one_flag(@all, @main_only, @deps_only)
+        help! "You can't provide more than one operation modifier (--deps, --main, etc.)"
+      end
+    end
+    
+    def run
+      ensure_in_work_tree
+      ensure_multirepo_enabled
+      
+      if @all
+        open_dependencies
+        open_main
+      elsif @main_only
+        open_main
+      else
+        open_dependencies
+      end
+    end
+
+    def open_main
+      Utils.reveal_in_default_file_browser(".")
+    end
+
+    def open_dependencies
+      ConfigFile.new(".").load_entries.each do |entry|
+        Utils.reveal_in_default_file_browser(entry.repo.path)
+      end
+    end
+  end
 end

data/lib/multirepo/commands/remove-command.rb

@@ -1,51 +1,48 @@
-require "multirepo/utility/console"
-require "multirepo/files/config-file"
-
-module MultiRepo
-  class RemoveCommand < Command
-    self.command = "remove"
-    self.summary = "Removes the specified dependency from multirepo."
-    
-    def self.options
-      [
-        ['<path>', 'The relative path to the dependency to remove (e.g. ../MyOldDependency).'],
-        ['[--delete]', 'Delete the dependency on disk in addition to removing it from the multirepo config.']
-      ].concat(super)
-    end
-    
-    def initialize(argv)
-      @path = argv.shift_argument
-      @delete = argv.flag?("delete")
-      super
-    end
-    
-    def validate!
-      super
-      help! "You must specify a dependency repository to remove" unless @path
-    end
-    
-    def run
-      super
-      ensure_in_work_tree
-      ensure_multirepo_enabled
-      
-      repo = Repo.new(@path)
-      entry = ConfigEntry.new(repo)
-      
-      config_file = ConfigFile.new(".")
-      if config_file.entry_exists?(entry)
-        config_file.remove_entry(entry)
-        Console.log_step("Removed '#{@path}' from the .multirepo file")
-        
-        if @delete
-          FileUtils.rm_rf(@path)
-          Console.log_step("Deleted '#{@path}' from disk")
-        end
-      else
-        raise MultiRepoException, "'#{@path}' isn't tracked by multirepo"
-      end
-    rescue MultiRepoException => e
-      Console.log_error(e.message)
-    end
-  end
+require "multirepo/utility/console"
+require "multirepo/files/config-file"
+
+module MultiRepo
+  class RemoveCommand < Command
+    self.command = "remove"
+    self.summary = "Removes the specified dependency from multirepo."
+    
+    def self.options
+      [
+        ['<path>', 'The relative path to the dependency to remove (e.g. ../MyOldDependency).'],
+        ['[--delete]', 'Delete the dependency on disk in addition to removing it from the multirepo config.']
+      ].concat(super)
+    end
+    
+    def initialize(argv)
+      @path = argv.shift_argument
+      @delete = argv.flag?("delete")
+      super
+    end
+    
+    def validate!
+      super
+      help! "You must specify a dependency repository to remove" unless @path
+    end
+    
+    def run
+      ensure_in_work_tree
+      ensure_multirepo_enabled
+      
+      repo = Repo.new(@path)
+      entry = ConfigEntry.new(repo)
+      
+      config_file = ConfigFile.new(".")
+      if config_file.entry_exists?(entry)
+        config_file.remove_entry(entry)
+        Console.log_step("Removed '#{@path}' from the .multirepo file")
+        
+        if @delete
+          FileUtils.rm_rf(@path)
+          Console.log_step("Deleted '#{@path}' from disk")
+        end
+      else
+        raise MultiRepoException, "'#{@path}' isn't tracked by multirepo"
+      end
+    end
+  end
 end

data/lib/multirepo/commands/uninit-command.rb

@@ -1,21 +1,18 @@
-require "multirepo/utility/console"
-
-module MultiRepo
-  class UninitCommand < Command
-    self.command = "uninit"
-    self.summary = "Removes all traces of multirepo in the current multirepo repository."
-    
-    def run
-      super
-      ensure_in_work_tree
-      
-      FileUtils.rm_f(".multirepo")
-      TrackingFiles.new(".").delete
-      uninstall_hooks
-      
-      Console.log_step("All traces of multirepo have been removed from this repository")
-    rescue MultiRepoException => e
-      Console.log_error(e.message)
-    end
-  end
+require "multirepo/utility/console"
+
+module MultiRepo
+  class UninitCommand < Command
+    self.command = "uninit"
+    self.summary = "Removes all traces of multirepo in the current multirepo repository."
+    
+    def run
+      ensure_in_work_tree
+      
+      FileUtils.rm_f(".multirepo")
+      TrackingFiles.new(".").delete
+      uninstall_hooks
+      
+      Console.log_step("All traces of multirepo have been removed from this repository")
+    end
+  end
 end

data/lib/multirepo/commands/update-command.rb

@@ -1,61 +1,56 @@
-require "multirepo/utility/console"
-require "multirepo/files/tracking-files"
-
-module MultiRepo
-  class UpdateCommand < Command
-    self.command = "update"
-    self.summary = "Force-updates the multirepo tracking files."
-    
-    def self.options
-      [
-        ['[--force]', 'Update the tracking files even if dependencies contain uncommitted changes.'],
-        ['[--commit]', 'Commit the tracking files after updating them.']
-      ].concat(super)
-    end
-    
-    def initialize(argv)
-      @commit = argv.flag?("commit")
-      @force = argv.flag?("force")
-      super
-    end
-
-    def run
-      super
-      ensure_in_work_tree
-      ensure_multirepo_enabled
-      
-      Console.log_step("Updating...")
-      
-      dependencies_clean = Utils.ensure_dependencies_clean(ConfigFile.new(".").load_entries)
-      if dependencies_clean
-        update_lock_file_step("Updated tracking files")
-      elsif !dependencies_clean && @force
-        update_lock_file_step("Force-updated tracking files (ignoring uncommitted changes)")
-      else
-        raise MultiRepoException, "Can't update because not all dependencies are clean"
-      end
-      
-      Console.log_step("Done!")
-    rescue MultiRepoException => e
-      Console.log_error(e.message)
-    end
-    
-    def update_lock_file_step(log_message)
-      tracking_files = TrackingFiles.new(".")
-      changed = tracking_files.update
-      
-      if changed
-        Console.log_substep("Updated tracking files")
-      else
-        Console.log_substep("Tracking files are already up-to-date")
-      end
-      
-      if @commit
-        committed = tracking_files.commit("[multirepo] Updated tracking files manually")
-        Console.log_substep("Committed tracking files") if committed
-      elsif changed
-        Console.log_substep(log_message)
-      end
-    end
-  end
+require "multirepo/utility/console"
+require "multirepo/files/tracking-files"
+
+module MultiRepo
+  class UpdateCommand < Command
+    self.command = "update"
+    self.summary = "Force-updates the multirepo tracking files."
+    
+    def self.options
+      [
+        ['[--force]', 'Update the tracking files even if dependencies contain uncommitted changes.'],
+        ['[--commit]', 'Commit the tracking files after updating them.']
+      ].concat(super)
+    end
+    
+    def initialize(argv)
+      @commit = argv.flag?("commit")
+      @force = argv.flag?("force")
+      super
+    end
+
+    def run
+      ensure_in_work_tree
+      ensure_multirepo_enabled
+      
+      Console.log_step("Updating...")
+      
+      dependencies_clean = Utils.dependencies_clean?(ConfigFile.new(".").load_entries)
+      if dependencies_clean
+        update_lock_file_step("Updated tracking files")
+      elsif !dependencies_clean && @force
+        update_lock_file_step("Force-updated tracking files (ignoring uncommitted changes)")
+      else
+        raise MultiRepoException, "Can't update because not all dependencies are clean"
+      end
+      
+      Console.log_step("Done!")
+    end
+    
+    def update_lock_file_step(log_message)
+      tracking_files = TrackingFiles.new(".")
+      changed = tracking_files.update
+      
+      if changed
+        Console.log_substep("Updated tracking files")
+      else
+        Console.log_substep("Tracking files are already up-to-date")
+      end
+      
+      if @commit
+        committed = tracking_files.commit("[multirepo] Updated tracking files manually")
+        Console.log_substep("Committed tracking files") if committed
+      end
+    end
+  end
 end

data/lib/multirepo/config.rb

@@ -1,16 +1,16 @@
-require "singleton"
-
-module MultiRepo
-  class Config
-    include Singleton
-    
-    attr_accessor :verbose
-    @verbose = false
-    
-    attr_accessor :running_git_hook
-    @running_git_hook = false
-
-    attr_accessor :git_executable
-    @git_executable = nil
-  end
+require "singleton"
+
+module MultiRepo
+  class Config
+    include Singleton
+    
+    attr_accessor :verbose
+    @verbose = false
+    
+    attr_accessor :running_git_hook
+    @running_git_hook = false
+
+    attr_accessor :git_executable
+    @git_executable = nil
+  end
 end

data/lib/multirepo/files/config-entry.rb

@@ -1,39 +1,39 @@
-require "securerandom"
-
-require "multirepo/utility/console"
-require "multirepo/git/repo"
-
-module MultiRepo
-  class ConfigEntry
-    attr_accessor :id
-    attr_accessor :path
-    attr_accessor :url
-    attr_accessor :repo
-    
-    def encode_with(coder)
-      coder["id"] = @id
-      coder["path"] = @path
-      coder["url"] = @url
-    end
-    
-    def initialize(repo)
-      @id = SecureRandom.uuid
-      @path = repo.path
-      @url = repo.exists? ? repo.remote('origin').url : nil
-    end
-    
-    def ==(entry)
-      entry_path = Pathname.new(entry.path)
-      self_path = Pathname.new(self.path)
-      entry_path.exist? && self_path.exist? && entry_path.realpath == self_path.realpath
-    end
-    
-    def repo
-      Repo.new(path)
-    end
-    
-    def name
-      repo.basename
-    end
-  end
+require "securerandom"
+
+require "multirepo/utility/console"
+require "multirepo/git/repo"
+
+module MultiRepo
+  class ConfigEntry
+    attr_accessor :id
+    attr_accessor :path
+    attr_accessor :url
+    attr_accessor :repo
+    
+    def encode_with(coder)
+      coder["id"] = @id
+      coder["path"] = @path
+      coder["url"] = @url
+    end
+    
+    def initialize(repo)
+      @id = SecureRandom.uuid
+      @path = repo.path
+      @url = repo.exists? ? repo.remote('origin').url : nil
+    end
+    
+    def ==(entry)
+      entry_path = Pathname.new(entry.path)
+      self_path = Pathname.new(self.path)
+      entry_path.exist? && self_path.exist? && entry_path.realpath == self_path.realpath
+    end
+    
+    def repo
+      Repo.new(path)
+    end
+    
+    def name
+      repo.basename
+    end
+  end
 end