Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-24 Thread Thomas Gummerer
On 04/21, Christian Couder wrote: > On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer > wrote: > > On 04/20, Christian Couder wrote: > >> > >> Could you try with the following patch: > >> > >> http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ > > > > Yeah, I tried with a

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Jeff King
On Fri, Apr 21, 2017 at 05:23:51PM +0200, Christian Couder wrote: > > I just tried on "pu" and only the first test > > (t7009-filter-branch-null-sha1.sh) fails there. > > I bisected this test's failure (when using > GIT_TEST_SPLIT_INDEX=YesPlease) to e6a1dd77e1 (read-cache: regenerate > shared in

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Jeff King
On Fri, Apr 21, 2017 at 07:25:21PM +0700, Duy Nguyen wrote: > >> Yeah, you are right. > >> It looks like we have GIT_TEST_OPTS to pass options like --debug, > >> --valgrind, --verbose, but we don't have an environment variable to > >> set config options. > > > > Or maybe GIT_CONFIG_PARAMETERS work

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Christian Couder
On Fri, Apr 21, 2017 at 4:25 PM, Christian Couder wrote: > On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer > wrote: >> On 04/20, Christian Couder wrote: >>> >>> Could you try with the following patch: >>> >>> http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ >> >> Yea

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Christian Couder
On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer wrote: > On 04/20, Christian Couder wrote: >> >> Could you try with the following patch: >> >> http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ > > Yeah, I tried with and without that patch with the same result. > Unless

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Duy Nguyen
On Fri, Apr 21, 2017 at 6:57 PM, Christian Couder wrote: > On Fri, Apr 21, 2017 at 1:46 PM, Christian Couder > wrote: >> On Fri, Apr 21, 2017 at 11:53 AM, Duy Nguyen wrote: >>> On Fri, Apr 21, 2017 at 2:10 PM, Christian Couder >>> wrote: On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Christian Couder
On Fri, Apr 21, 2017 at 1:46 PM, Christian Couder wrote: > On Fri, Apr 21, 2017 at 11:53 AM, Duy Nguyen wrote: >> On Fri, Apr 21, 2017 at 2:10 PM, Christian Couder >> wrote: >>> On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer >>> wrote: On 04/20, Christian Couder wrote: > > Coul

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Christian Couder
On Fri, Apr 21, 2017 at 11:53 AM, Duy Nguyen wrote: > On Fri, Apr 21, 2017 at 2:10 PM, Christian Couder > wrote: >> On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer >> wrote: >>> On 04/20, Christian Couder wrote: Could you try with the following patch: http://public-inbox.o

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Duy Nguyen
On Fri, Apr 21, 2017 at 2:10 PM, Christian Couder wrote: > On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer > wrote: >> On 04/20, Christian Couder wrote: >>> >>> Could you try with the following patch: >>> >>> http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ >> >> Yea

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Christian Couder
On Thu, Apr 20, 2017 at 11:24 PM, Thomas Gummerer wrote: > On 04/20, Christian Couder wrote: >> >> Could you try with the following patch: >> >> http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ > > Yeah, I tried with and without that patch with the same result. > Unless

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-21 Thread Christian Couder
On Fri, Apr 21, 2017 at 6:01 AM, Junio C Hamano wrote: > Christian Couder writes: > >> Could you try with the following patch: >> >> http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ > > Ah, this reminds me. The patch has been in the stalled state for > quite some time

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-20 Thread Junio C Hamano
Christian Couder writes: > Could you try with the following patch: > > http://public-inbox.org/git/20170330210354.20018-1-chrisc...@tuxfamily.org/ Ah, this reminds me. The patch has been in the stalled state for quite some time due to confusing description. How about explaining it like so and

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-20 Thread Thomas Gummerer
On 04/20, Christian Couder wrote: > Hi, > > On Thu, Apr 20, 2017 at 10:52 PM, Thomas Gummerer > wrote: > > Hi, > > > > I just tried to run the test suite with GIT_TEST_SPLIT_INDEX=YesPlease > > and noticed that a few tests are broken both in pu and master. > > > > Below the test failures on mast

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-20 Thread Jeff King
On Thu, Apr 20, 2017 at 09:52:14PM +0100, Thomas Gummerer wrote: > I just tried to run the test suite with GIT_TEST_SPLIT_INDEX=YesPlease > and noticed that a few tests are broken both in pu and master. > [...] > Bisecting between master and v2.10.0 leads me to the merge commit > 94c9b5af70 ("Merg

Re: [BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-20 Thread Christian Couder
Hi, On Thu, Apr 20, 2017 at 10:52 PM, Thomas Gummerer wrote: > Hi, > > I just tried to run the test suite with GIT_TEST_SPLIT_INDEX=YesPlease > and noticed that a few tests are broken both in pu and master. > > Below the test failures on master: > > Test Summary Report > --- > t70

[BUG] test suite broken with GIT_TEST_SPLIT_INDEX

2017-04-20 Thread Thomas Gummerer
Hi, I just tried to run the test suite with GIT_TEST_SPLIT_INDEX=YesPlease and noticed that a few tests are broken both in pu and master. Below the test failures on master: Test Summary Report --- t7009-filter-branch-null-sha1.sh (Wstat: 256 Tests: 5 Failed: 2)