Re: [PATCH] perf: accommodate for MacOSX

Hi Lars, On Tue, 21 Jun 2016, Lars Schneider wrote: > > On 21 Jun 2016, at 13:55, Johannes Schindelin > > wrote: > > > >> ... > >> If we don't run any perf tests by default on Travis CI then I wouldn't > >> take the ".travis.yml" part of the patch just to keep our Travis CI > >> setup as lean

Re: [PATCH] perf: accommodate for MacOSX

> On 21 Jun 2016, at 13:55, Johannes Schindelin > wrote: > >> ... >> I think we definitively should take the "perf-lib.sh" part of the patch >> as this makes the perf test run on OSX and therefore is a strict >> improvement. > > Yes, it was meant as the starting point to get more things to run

Re: [PATCH] perf: accommodate for MacOSX

Hi Lars, On Tue, 21 Jun 2016, Lars Schneider wrote: > > On 20 Jun 2016, at 21:48, Junio C Hamano wrote: > > > > Johannes Schindelin writes: > > > >> On Sun, 19 Jun 2016, Lars Schneider wrote: > >> > On 18 Jun 2016, at 15:03, Johannes Schindelin > wrote: > > As this deve

Re: [PATCH] perf: accommodate for MacOSX

> On 20 Jun 2016, at 21:48, Junio C Hamano wrote: > > Johannes Schindelin writes: > >> On Sun, 19 Jun 2016, Lars Schneider wrote: >> On 18 Jun 2016, at 15:03, Johannes Schindelin wrote: As this developer has no access to MacOSX developer setups anymore, Travis becom

Re: [PATCH] perf: accommodate for MacOSX

Johannes Schindelin writes: > On Sun, 19 Jun 2016, Lars Schneider wrote: > >> > On 18 Jun 2016, at 15:03, Johannes Schindelin >> > wrote: >> > >> > As this developer has no access to MacOSX developer setups anymore, >> > Travis becomes the best bet to run performance tests on that OS. >> >> We

Re: [PATCH] perf: accommodate for MacOSX

Hi Lars, On Sun, 19 Jun 2016, Lars Schneider wrote: > > On 18 Jun 2016, at 15:03, Johannes Schindelin > > wrote: > > > > As this developer has no access to MacOSX developer setups anymore, > > Travis becomes the best bet to run performance tests on that OS. > > We don't run the performance test

Re: [PATCH] perf: accommodate for MacOSX

> On 18 Jun 2016, at 15:03, Johannes Schindelin > wrote: > > As this developer has no access to MacOSX developer setups anymore, > Travis becomes the best bet to run performance tests on that OS. We don't run the performance tests on Travis CI right now. Maybe we should? With your patch below i

[PATCH] perf: accommodate for MacOSX

As this developer has no access to MacOSX developer setups anymore, Travis becomes the best bet to run performance tests on that OS. However, on MacOSX /usr/bin/time is that good old BSD executable that no Linux user cares about, as demonstrated by the perf-lib.sh's use of GNU-ish extensions. And