new hook post-receive dont accept python #544

Closed
opened 2025-11-02 03:27:37 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @tofulm on GitHub (Mar 20, 2017).

hi,
On last release v1.1.0,
the new hook post-receive does not allow we to execute scripts in python.
L6 : sh "$SHELL_FOLDER/post-receive.d/$i"
If we delete sh it's ok

Originally created by @tofulm on GitHub (Mar 20, 2017). hi, On last release v1.1.0, the new hook post-receive does not allow we to execute scripts in python. L6 : `sh "$SHELL_FOLDER/post-receive.d/$i`" If we delete `sh` it's ok
GiteaMirror added the issue/duplicate label 2025-11-02 03:27:37 -06:00
Author
Owner

@strk commented on GitHub (Mar 20, 2017):

+1 to remove the sh call.
Hooks can be in any language, as long as they are executables, they
should be executed.

@strk commented on GitHub (Mar 20, 2017): +1 to remove the sh call. Hooks can be in any language, as long as they are executables, they should be executed.
Author
Owner

@lunny commented on GitHub (Mar 20, 2017):

So anyone could send a PR to fix that?

@lunny commented on GitHub (Mar 20, 2017): So anyone could send a PR to fix that?
Author
Owner

@strk commented on GitHub (Mar 20, 2017):

Please anyone doing a PR: wrap the execution in a function as there
are currently 3 different places where that overcomplex shell script
is executed

@strk commented on GitHub (Mar 20, 2017): Please anyone doing a PR: wrap the execution in a function as there are currently 3 different places where that overcomplex shell script is executed
Author
Owner

@philfry commented on GitHub (Mar 20, 2017):

anything wrong with #1257?
(see also #1250 )

@philfry commented on GitHub (Mar 20, 2017): anything wrong with #1257? (see also #1250 )
Author
Owner

@lunny commented on GitHub (Mar 21, 2017):

Let's move that issue and PR to discuss continue.

@lunny commented on GitHub (Mar 21, 2017): Let's move that issue and PR to discuss continue.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#544