Re: [PATCH] perf: accommodate for MacOSX

[Lars Schneider <larsxschneider@gmail.com>]

> On 20 Jun 2016, at 21:48, Junio C Hamano <gitster@pobox.com> wrote:
> 
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
>> On Sun, 19 Jun 2016, Lars Schneider wrote:
>> 
>>>> On 18 Jun 2016, at 15:03, Johannes Schindelin
>>>> <johannes.schindelin@gmx.de> 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 it should work, right?
>> ...
>> 
>> Yeah, well, I should have been clearer in my commit message: this patch
>> allows the perf tests to *run*, not to *pass*... ;-)
> 
> OK, Lars, do we still want to take this patch?  I am leaning towards
> taking it, if only to motivate interested others with OS X to look
> into making the perf tests to actually run.

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.

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 as possible. Running perf tests on Travis CI is probably
bogus anyways because we never know on what hardware our jobs run
and what other jobs run in parallel on that hardware.

- Lars