git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Set the executable bit on gitMergeCommon.py.

From
Junio C Hamano <junkio@cox.net>
Date
May 23, 2006, 08:34 UTC
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?

Previous: Peter Eriksen
Message 2 of 2 in “Set the executable bit on gitMergeCommon.py.”
  1. Set the executable bit on gitMergeCommon.py.Peter Eriksen, May 23, 2006
  2. Junio C HamanoMay 23, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.