Re: [PATCH] mergetools/p4merge: Honor $TMPDIR for the /dev/null placeholder

2012-12-20 Thread Joachim Schmitz
Junio C Hamano wrote: David Aguilar writes: Use mktemp to create the /dev/null placeholder for p4merge. This keeps it out of the current directory. Reported-by: Jeremy Morton Signed-off-by: David Aguilar --- I consider this a final finishing touch on a new 1.8.1 feature, so hopefully we can

Re: [PATCH] mergetools/p4merge: Honor $TMPDIR for the /dev/null placeholder

2012-12-20 Thread David Aguilar
On Wed, Dec 19, 2012 at 10:28 PM, Junio C Hamano wrote: > David Aguilar writes: > >> Use mktemp to create the /dev/null placeholder for p4merge. >> This keeps it out of the current directory. >> >> Reported-by: Jeremy Morton >> Signed-off-by: David Aguilar >> --- >> I consider this a final fini

Re: [PATCH] mergetools/p4merge: Honor $TMPDIR for the /dev/null placeholder

2012-12-19 Thread Junio C Hamano
Junio C Hamano writes: > David Aguilar writes: > >> Use mktemp to create the /dev/null placeholder for p4merge. >> This keeps it out of the current directory. >> >> Reported-by: Jeremy Morton >> Signed-off-by: David Aguilar >> --- >> I consider this a final finishing touch on a new 1.8.1 featu

Re: [PATCH] mergetools/p4merge: Honor $TMPDIR for the /dev/null placeholder

2012-12-19 Thread Junio C Hamano
David Aguilar writes: > Use mktemp to create the /dev/null placeholder for p4merge. > This keeps it out of the current directory. > > Reported-by: Jeremy Morton > Signed-off-by: David Aguilar > --- > I consider this a final finishing touch on a new 1.8.1 feature, > so hopefully we can get this