On Sun, Feb 10, 2019 at 10:17:12PM +0000, Thomas Gummerer wrote: > On 02/08, SZEDER Gábor wrote: > > On Thu, Dec 20, 2018 at 09:44:34PM +0200, Paul-Sebastian Ungureanu wrote: > > > Add stash push to the helper. > > > > > > Signed-off-by: Paul-Sebastian Ungureanu <ungureanupaulsebastian@xxxxxxxxx> > > > > This patch causes rare failures in 't3903-stash.sh', I've seen it > > break for the first time today in a Travis CI build: > > Thanks for reporting this. I was going to take a look at it, but > unfortunately I can't seem to reproduce the issue even with --stress > (I let it run for ~1000 repetitions and then aborted it). > > Which platform did you see/test this on, and which compile options did > you use? I went through a failures on > https://travis-ci.org/git/git/builds, but couldn't find this > particular one. Could you point me at the failed run? It was in a Linux build job in one of my custom CI builds [1]: https://travis-ci.org/szeder/git-cooking-topics-for-travis-ci/jobs/490420713#L3401 and I tested it locally on Linux as well. I don't think there are any unusual compiler options. I run it with --stress on Travis CI's macOS as well, but couldn't trigger a failure there. [1] That build log doesn't include the trash dir of the failed test in the log, most likely because of a faulty merge conflict resolution on my part, but you can find a failed trash dir embedded here: https://travis-ci.org/szeder/git/jobs/491401882#L2309