Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts

2017-03-03 Thread Sebastian Schuberth
On Fri, Mar 3, 2017 at 8:10 PM, Junio C Hamano wrote: >> Just a niggle: This change moves the warning message from stderr to stdout. > > Right. Here is what I'll queue. Indeed, thanks for the note, and also Junio for fixing while queuing. -- Sebastian Schuberth

Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts

2017-03-03 Thread Junio C Hamano
Marc Branchaud writes: > On 2017-03-03 05:57 AM, Sebastian Schuberth wrote: >> It does not make sense for these placeholder scripts to depend on Python >> just because the real scripts do. At the example of Git for Windows, we >> would not even be able to see those warnings as it does not ship wi

Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts

2017-03-03 Thread Junio C Hamano
Jeff King writes: > On Fri, Mar 03, 2017 at 10:57:46AM +, Sebastian Schuberth wrote: > >> It does not make sense for these placeholder scripts to depend on Python >> just because the real scripts do. At the example of Git for Windows, we >> would not even be able to see those warnings as it d

Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts

2017-03-03 Thread Marc Branchaud
On 2017-03-03 05:57 AM, Sebastian Schuberth wrote: It does not make sense for these placeholder scripts to depend on Python just because the real scripts do. At the example of Git for Windows, we would not even be able to see those warnings as it does not ship with Python. So just use plain shell

Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts

2017-03-03 Thread Jeff King
On Fri, Mar 03, 2017 at 10:57:46AM +, Sebastian Schuberth wrote: > It does not make sense for these placeholder scripts to depend on Python > just because the real scripts do. At the example of Git for Windows, we > would not even be able to see those warnings as it does not ship with > Python

[PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts

2017-03-03 Thread Sebastian Schuberth
It does not make sense for these placeholder scripts to depend on Python just because the real scripts do. At the example of Git for Windows, we would not even be able to see those warnings as it does not ship with Python. So just use plain shell scripts instead. Signed-off-by: Sebastian Schuberth