Re: [PATCH] wrap-for-bin.sh: facilitate running Git executables under valgrind

2018-05-09 Thread Antonio Ospite
On Wed, 9 May 2018 08:25:21 -0700 Elijah Newren wrote: > Hi Antonio, > Hi Elijah, > On Wed, May 9, 2018 at 6:28 AM, Antonio Ospite wrote: > > Testing locally built git executables under valgrind is not immediate. > > > > Something like the following does not work: > > > > $ valgrind ./bin-w

Re: [PATCH] wrap-for-bin.sh: facilitate running Git executables under valgrind

2018-05-09 Thread Elijah Newren
Hi Antonio, On Wed, May 9, 2018 at 6:28 AM, Antonio Ospite wrote: > Testing locally built git executables under valgrind is not immediate. > > Something like the following does not work: > > $ valgrind ./bin-wrappers/git > > because the wrapper script forks and execs the command and valgrind do

Re: [PATCH] wrap-for-bin.sh: facilitate running Git executables under valgrind

2018-05-09 Thread Jeff King
On Wed, May 09, 2018 at 03:28:58PM +0200, Antonio Ospite wrote: > Testing locally built git executables under valgrind is not immediate. > > Something like the following does not work: > > $ valgrind ./bin-wrappers/git > > because the wrapper script forks and execs the command and valgrind do

[PATCH] wrap-for-bin.sh: facilitate running Git executables under valgrind

2018-05-09 Thread Antonio Ospite
Testing locally built git executables under valgrind is not immediate. Something like the following does not work: $ valgrind ./bin-wrappers/git because the wrapper script forks and execs the command and valgrind does not track children processes by default. Something like the following may w