From: Ralf Jung Date: Mon, 27 Mar 2017 17:13:59 +0000 (+0200) Subject: try to reproduce the quirky behavior of bash/execve for running scrips X-Git-Url: https://git.ralfj.de/git-mirror.git/commitdiff_plain/a34e9c02d6fa063656dd224c9ba3f9f713e1e776?hp=2b00eaa147818ccfd68a1466d28c41995accef7d try to reproduce the quirky behavior of bash/execve for running scrips --- diff --git a/git_mirror.py b/git_mirror.py index 12075bf..6b72775 100644 --- a/git_mirror.py +++ b/git_mirror.py @@ -29,6 +29,17 @@ import email.mime.text, email.utils, smtplib mail_sender = "null@localhost" config_file = os.path.join(os.path.dirname(__file__), 'git-mirror.conf') +def Popen_quirky(cmd, **args): + ''' + Runs cmd via subprocess.Popen; and if that fails, puts it into the shell (/bin/sh). + It seems that's what executing things in bash does, and even execve. Also, + all so-far released versions of Gitolite get the shebang line wrong. + ''' + try: + return subprocess.Popen(cmd, **args) + except OSError as e: + return subprocess.Popen(['/bin/sh'] + cmd, **args) + class GitCommand: def __getattr__(self, name): def call(*args, capture_stderr = False, check = True): @@ -177,7 +188,7 @@ class Repo: # Now run the post-receive hooks. This will *also* push the changes to all mirrors, as we # are one of these hooks! os.putenv("GIT_MIRROR_SOURCE", mirror) # tell ourselves which repo we do *not* have to update - with subprocess.Popen(['hooks/post-receive'], stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.STDOUT) as p: + with Popen_quirky(['hooks/post-receive'], stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.STDOUT) as p: (stdout, stderr) = p.communicate("{} {} {}\n".format(oldsha, newsha, ref).encode('utf-8')) stdout = stdout.decode('utf-8') if p.returncode: