script/plugin install fails doesn't populate new directory in vendor/plugins

Using rails 2.3.4, ruby 1.8.7 (2009-06-12 patchlevel 174), and git 1.6.5.rc2, behold this:

$ script/plugin install git://github.com/thoughtbot/paperclip.git
Initialized empty Git repository in /Users/technicalpickles/code/scratch/upupload/vendor/plugins/paperclip/.git/
$ ls vendor/plugins/paperclip
$

Seems the problem is around railties/lib/commands/plugin.rb ... when system('git pull --depth 1...') is called, this exits with 1 (confirmed in the shell by doing it by hand).

$ mkdir -p vendor/plugins/paperclip
$ cd vendor/plugins/paperclip
$ git init
Initialized empty Git repository in /Users/technicalpickles/code/scratch/upupload/vendor/plugins/paperclip/.git/
$ git pull --depth 1 git://github.com/thoughtbot/paperclip.git -q
$ echo $?
1

I'm not sure why this particular approach of init & pull is used rather than just cloning. I was able to get it working by replacing Commands::Plugin#install_using_git with this:

    def install_using_git(options = {})
      root = rails_env.root
      install_path = mkdir_p "#{root}/vendor/plugins/#{name}"
      base_cmd = "git clone #{uri}"
      base_cmd += " -q" if options[:quiet] and not $verbose
      base_cmd += " #{options[:revision]}" if options[:revision]
      base_cmd += " #{install_path}"
      if system(base_cmd)
        puts "removing: .git .gitignore" if $verbose
        rm_rf %W(#{install_path}/.git #{install_path}/.gitignore)
      else
        rm_rf install_path
      end
    end

... and it seems to work well enough. On aside, the else should probably display an error message or otherwise notifiy the user what happened.