Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-28 Thread Felipe Contreras
On Wed, Nov 28, 2012 at 8:33 PM, Junio C Hamano wrote: > Felipe Contreras writes: > >> On Wed, Nov 28, 2012 at 5:57 PM, Junio C Hamano wrote: >> ... >>> You need a fix for that; didn't I already say "you need a bit more >>> than that"? >> >> I disagree. Most of the contrib scripts are expected t

Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-28 Thread Junio C Hamano
Felipe Contreras writes: > On Wed, Nov 28, 2012 at 5:57 PM, Junio C Hamano wrote: > ... >> You need a fix for that; didn't I already say "you need a bit more >> than that"? > > I disagree. Most of the contrib scripts are expected to be used as > they are. You are only looking at one of the uses

Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-28 Thread Felipe Contreras
On Wed, Nov 28, 2012 at 5:57 PM, Junio C Hamano wrote: > Christian Couder writes: > >> On Wed, Nov 28, 2012 at 9:03 AM, Felipe Contreras >> wrote: >>> That works if somebody managed to export PYTHON_PATH, which very very >>> often is not the case for me. >> >> Yeah, and even if PYTHON_PATH is u

Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-28 Thread Junio C Hamano
Christian Couder writes: > On Wed, Nov 28, 2012 at 9:03 AM, Felipe Contreras > wrote: >> On Wed, Nov 28, 2012 at 8:36 AM, Junio C Hamano wrote: >>> Christian Couder writes: >>> All the python scripts except contrib/svn-fe/svnrdump_sim.py start with "#!/usr/bin/env python".

Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-28 Thread Christian Couder
On Wed, Nov 28, 2012 at 9:03 AM, Felipe Contreras wrote: > On Wed, Nov 28, 2012 at 8:36 AM, Junio C Hamano wrote: >> Christian Couder writes: >> >>> All the python scripts except contrib/svn-fe/svnrdump_sim.py >>> start with "#!/usr/bin/env python". >>> >>> This patch fix contrib/svn-fe/svnrdump

Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-28 Thread Felipe Contreras
On Wed, Nov 28, 2012 at 8:36 AM, Junio C Hamano wrote: > Christian Couder writes: > >> All the python scripts except contrib/svn-fe/svnrdump_sim.py >> start with "#!/usr/bin/env python". >> >> This patch fix contrib/svn-fe/svnrdump_sim.py to do the same. > > I suspect you need a bit more than tha

Re: [PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-27 Thread Junio C Hamano
Christian Couder writes: > All the python scripts except contrib/svn-fe/svnrdump_sim.py > start with "#!/usr/bin/env python". > > This patch fix contrib/svn-fe/svnrdump_sim.py to do the same. I suspect you need a bit more than that. $ make git-p4 $ diff -u git-p4.py git-p4 shows you ho

[PATCH] svnrdump_sim: start the script with /usr/bin/env python

2012-11-27 Thread Christian Couder
All the python scripts except contrib/svn-fe/svnrdump_sim.py start with "#!/usr/bin/env python". This patch fix contrib/svn-fe/svnrdump_sim.py to do the same. Signed-off-by: Christian Couder --- contrib/svn-fe/svnrdump_sim.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/c