Re: [PATCH v5 24/24] t1406: new tests for submodule ref store

From: Duy Nguyen <pclouds@gmail.com>
Date: Mar 2, 2017, 06:13 UTC
Message-ID: <CACsJy8DyjWC5WUE7=dWK-mi_piGZJf7L1B0t5RUROvqDS+aP5g@mail.gmail.com>
In-Reply-To: <8f4ec174-9060-0896-5135-bb50229fe149@alum.mit.edu>

On Wed, Mar 1, 2017 at 10:11 PM, Michael Haggerty <mhagger@alum.mit.edu> wrote:

Show 10 quoted lines

> By trial and error, I found that the test succeeds if I comment out the
> "for_each_reflog()" test. By having that test write its results to
> `/tmp` where they won't be deleted, I found that the problem is that the
> `actual` results are not sorted correctly:
>
>     refs/heads/new-master 0x0
>     refs/heads/master 0x0
>     HEAD 0x1
>
> I don't know why it's so Heisenbergish.

It happens consistently on my other laptop. And yes it looks like sorting order problem, probably because of the underlying file system. I did wonder about that at some point but never asked. We do not guarantee any sorting order in the for-each api, do we?

-- 
Duy