Repository navigation
Issue while trying to commit - "repo.index.commit" #1129
Description
I can see a similar open issue but it relates to post-commit, hence adding this.
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\git\index\fun.py", line 83, in run_commit_hook
creationflags=PROC_CREATIONFLAGS,)
File "C:\Python37\lib\subprocess.py", line 800, in init
restore_signals, start_new_session)
File "C:\Python37\lib\subprocess.py", line 1207, in _execute_child
startupinfo)
OSError: [WinError 193] %1 is not a valid Win32 application
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\XXXX\Desktop\auto_mate\deployment\gitautomate.py", line 42, in
repo1.index.commit("first commit")
File "C:\Python37\lib\site-packages\git\index\base.py", line 944, in commit
run_commit_hook('pre-commit', self)
File "C:\Python37\lib\site-packages\git\index\fun.py", line 85, in run_commit_hook
raise HookExecutionError(hp, ex) from ex
git.exc.HookExecutionError: Hook('C:\Users\XXXX\Desktop\auto_mate\deployment\clonedir.git\hooks\pre-commit') failed due to: OSError('[WinError 193] %1 is not a valid Win32 application')
cmdline: C:\Users\XXXX\Desktop\auto_mate\deployment\clonedir.git\hooks\pre-commit
Activity
ameyapranjale commentedon Feb 21, 2021
---------------pre- commit file --------------
#!/bin/sh
HOOKS_DIR=$(dirname $0)
acn_git_hooks_deploy_id XXXX
export CONFIG_DIRECTORY='C:/Users/XXXX/.acn-security-tools/config'
if uname | grep "MINGW"* > /dev/null;
then
py ${HOOKS_DIR}/detect_remote
py ${HOOKS_DIR}/detect_secret
else
python ${HOOKS_DIR}/detect_remote
python ${HOOKS_DIR}/detect_secret
fi
Byron commentedon Feb 22, 2021
As shown by
File "C:\Python37\lib\subprocess.py", line 1207, in _execute_childsubprocess is used to execute hooks. On windows, this probably doesn't include shellscripts.Fixing this would be possible and a PR is certainly welcome.
Please feel free to keep commenting on the closed issue for follow-ups.