[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] add shebang line to build/run_tests.py

From: Madan U Sreenivasan <madan_at_collab.net>
Date: 2006-05-30 15:45:36 CEST

On Tue, 30 May 2006 16:34:05 +0530, Malcolm Rowe
<malcolm-svn-dev@farside.org.uk> wrote:

> On Tue, May 30, 2006 at 04:12:32PM +0530, Madan U Sreenivasan wrote:
>> Pl. find a patch that adds the shebang line (#!/usr/bin/env python)
>> to
>> build/run_tests.py. As a consequence, the svn:executable property had to
>> be added to build/run_tests.py and the Makefile.in modified to run
>> run_tests.py directly (instead of 'python run_tests.py').
>>
>
> Why is this a good idea?

Because it is easier to invoke run_tests.py. Also the test scripts in
subversion/tests/cmdline use the shebang line. This change would make
runnable python scripts more uniform.

OTOH, I was wondering if there is any specific reason we have a python
script without the shebang line.

Regards,
Madan.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 30 15:15:05 2006

This is an archived mail posted to the Subversion Dev mailing list.