From 31885c2a998f5e6924740b4c9e3ea8eb951345d3 Mon Sep 17 00:00:00 2001 From: Max Howell Date: Thu, 4 Aug 2011 13:45:48 +0100 Subject: [PATCH] Constants that are used once are dumb It just means nobody knows what is being done without scrolling up and down, up and down the file like a doofus on a pogo stick. --- Library/Homebrew/cmd/update.rb | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/Library/Homebrew/cmd/update.rb b/Library/Homebrew/cmd/update.rb index 03ff290719..2f40bfb70a 100644 --- a/Library/Homebrew/cmd/update.rb +++ b/Library/Homebrew/cmd/update.rb @@ -12,12 +12,7 @@ module Homebrew extend self end class RefreshBrew - REPOSITORY_URL = "http://github.com/mxcl/homebrew.git" - CHECKOUT_COMMAND = "git checkout -q master" - UPDATE_COMMAND = "git pull #{REPOSITORY_URL} master" - REVISION_COMMAND = "git rev-parse HEAD" - DIFF_COMMAND = "git diff-tree -r --name-status -z %s %s" - + REPOSITORY_URL = "http://github.com/mxcl/homebrew.git" FORMULA_DIR = 'Library/Formula/' EXAMPLE_DIR = 'Library/Contributions/examples/' @@ -36,7 +31,7 @@ class RefreshBrew def update_from_masterbrew! HOMEBREW_REPOSITORY.cd do if git_repo? - safe_system CHECKOUT_COMMAND + safe_system "git checkout -q master" @initial_revision = read_revision else begin @@ -44,11 +39,11 @@ class RefreshBrew safe_system "git fetch #{REPOSITORY_URL}" safe_system "git reset --hard FETCH_HEAD" rescue Exception - safe_system "rm -rf .git" + safe_system "/bin/rm -rf .git" raise end end - execute(UPDATE_COMMAND) + execute "git pull #{REPOSITORY_URL} master" @current_revision = read_revision end @@ -58,7 +53,7 @@ class RefreshBrew @changes_map = Hash.new {|h,k| h[k] = [] } changes = HOMEBREW_REPOSITORY.cd do - execute(DIFF_COMMAND % [initial_revision, current_revision]).split("\0") + execute("git diff-tree -r --name-status -z #{initial_revision} #{current_revision}").split("\0") end while status = changes.shift @@ -150,7 +145,7 @@ class RefreshBrew private def read_revision - execute(REVISION_COMMAND).chomp + execute("git rev-parse HEAD").chomp end def filter_by_directory(files, dir)