captainhookphp / captainhook

CaptainHook is a very flexible git hook manager for software developers that makes sharing git hooks with your team a breeze.
http://captainhook.info
MIT License
1.01k stars 87 forks source link

Version 5.16.2 broke git diff commands #209

Closed shadowhand closed 1 year ago

shadowhand commented 1 year ago

Since upgrading to 5.16.2 we started to get this every time captainhook runs:

post-checkout:                                                                                                          
 - composer install                                                  :                                                  
Command failed:                                                                                                         
  exit-code: 127                                                                                                        
  message:   sh: C: command not found                                                                                   
sh: M: command not found                                                                                                
sh: R: command not found                                                                                                
usage: git diff-tree [--stdin] [-m] [-s] [-v] [--no-commit-id] [--pretty]                                               

I'm not sure if this is because of captainhook itself, or a change in sebastianfeldmann/git.

sebastianfeldmann commented 1 year ago

I will have a look as soon as I'm at home 🙈

sebastianfeldmann commented 1 year ago

I found the problem I will release as fix in roughly one hour

sebastianfeldmann commented 1 year ago

Composer update should fix it. It's only the git dependency that causes the issue I still increased the Captain's version to not require the broken dependency by accident.

Let me know if it fixes your issue.

shadowhand commented 1 year ago

Thank you for the quick response. I was able to update sebastianfeldmann/git and captainhook/captainhook today and the issue has been resolved.

sebastianfeldmann commented 1 year ago

Thanks for the feedback