From: Junio C Hamano <junkio@cox.net>
Date: Tue, 23 May 2006 08:34:53 GMT
Subject: Re: [PATCH] Set the executable bit on gitMergeCommon.py.
Message-ID: <7virnx5ec2.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <20060523081238.GB4038@bohr.gbar.dtu.dk>

"Peter Eriksen" <s022018@student.dtu.dk> writes:

> From: Peter Eriksen <s022018@student.dtu.dk>
> Date: Mon, 22 May 2006 15:35:42 +0200
>
> Signed-off-by: Peter Eriksen <s022018@student.dtu.dk>

It makes some sense to make git-merge-recursive.py executable,
just in case somebody wants to run it without preprocessing.

However, gitMergeCommon.py is a module without the main program.
It does not even have the usual "primarily meant to be a library
but you could run it standalone" idiom at the end:

	if __name__ == '__main__': test()

So I am not sure if this patch makes sense -- does it misbehave
on your system when installed without +x bit?