search

mina-deploy / mina

Blazing fast deployer and server automation tool
https://rubygems.org/gems/mina
Other
4.36k stars 492 forks source link

warning: --depth is ignored in local clones; use file:// instead. #56

Closed jpascal closed 12 years ago

jpascal commented 12 years ago 
$ mina deploy
-----> Using git branch 'master'
       warning: --depth is ignored in local clones; use file:// instead.
       Cloning into ....
       done.
rstacruz commented 12 years ago

Ohh, silly me. I suppose that ought to be (lib/mina/git.rb):

- #{echo_cmd %[git clone "#{deploy_to}/scm" . --depth 1 --recursive --branch "#{branch}"]} &&
+ #{echo_cmd %[git clone "file://#{deploy_to}/scm" . --depth 1 --recursive --branch "#{branch}"]} &&
rstacruz commented 12 years ago

Wait, I'm not sure that's a good idea, using file:// will be slower (because it copies and doesn't use hardlinks). Looks like --depth is trully redundant afterall.

Ref: http://git-scm.com/book/en/Git-on-the-Server-The-Protocols

rstacruz commented 12 years ago

Scheduled for v0.2.1.