moll / mina-rsync

Deploy with Rsync from any local (or remote) repository when using Mina.
Other
50 stars 25 forks source link

Update rsync.rb #9

Closed ghost closed 10 years ago

ghost commented 10 years ago

Problem on OS X that seems to be related to cwd when doing git reset --hard master/origin. By wrapping the commands in a Dir.chdir block the git error message goes away.

Error message I keep getting is of the following form:

git --git-dir tmp/deploy/.git --work-tree tmp/deploy reset --hard origin/master error: packfile tmp/deploy/.git/objects/pack/pack-ac917f6f3e4ecd079fc654cb5acd6586840f2f97.pack cannot be accessed HEAD is now at ae0366c testing mina-rsync

moll commented 10 years ago

Hey! First, thank you and second, I apologize for seriously long radio silence. :)

You seemed to have closed this. Was the problem not in the current directory?

ghost commented 10 years ago

Hi, This was my first pull request and I realized I should have sent you just my patch and not the commented out original code. I wanted to do this a little cleaner so I closed it. I will send you the cleaned up code patch. This definitely is a repeatable failure on OS X, which seems to me like a Git bug but the Dir.chdir fixed it. This is a great piece of code , I've learned a lot from it. thanks Gary

On Fri, May 23, 2014 at 11:23 AM, Andri Möll notifications@github.comwrote:

Hey! First, thank you and second, I apologize for seriously long radio silence. :)

You seemed to have closed this. Was the problem not in the current directory?

— Reply to this email directly or view it on GitHubhttps://github.com/moll/mina-rsync/pull/9#issuecomment-44025068 .