Re: [PATCH v3 2/4] rev-list: add an option to mark fewer edges as uninteresting

2014-12-24 Thread brian m. carlson
On Tue, Dec 23, 2014 at 01:51:42PM -0500, Jeff King wrote: On Tue, Dec 23, 2014 at 12:55:48PM -0500, Michael Blume wrote: This patch causes an error on my mac, test 5500 fetch-pack errors on part 44 - fetch creating new shallow root. It looks for remote: Total 1 in the fetch output and

[PATCH v3 2/4] rev-list: add an option to mark fewer edges as uninteresting

2014-12-23 Thread brian m. carlson
In commit fbd4a70 (list-objects: mark more commits as edges in mark_edges_uninteresting - 2013-08-16), we marked an increasing number of edges uninteresting. This change, and the subsequent change to make this conditional on --objects-edge, are used by --thin to make much smaller packs for

Re: [PATCH v3 2/4] rev-list: add an option to mark fewer edges as uninteresting

2014-12-23 Thread Michael Blume
This patch causes an error on my mac, test 5500 fetch-pack errors on part 44 - fetch creating new shallow root. It looks for remote: Total 1 in the fetch output and gets 3 instead. On Tue, Dec 23, 2014 at 7:01 AM, brian m. carlson sand...@crustytoothpaste.net wrote: In commit fbd4a70

Re: [PATCH v3 2/4] rev-list: add an option to mark fewer edges as uninteresting

2014-12-23 Thread Jeff King
On Tue, Dec 23, 2014 at 12:55:48PM -0500, Michael Blume wrote: This patch causes an error on my mac, test 5500 fetch-pack errors on part 44 - fetch creating new shallow root. It looks for remote: Total 1 in the fetch output and gets 3 instead. It fails for me on Linux, too. Interestingly the

Re: [PATCH v3 2/4] rev-list: add an option to mark fewer edges as uninteresting

2014-12-23 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Tue, Dec 23, 2014 at 12:55:48PM -0500, Michael Blume wrote: This patch causes an error on my mac, test 5500 fetch-pack errors on part 44 - fetch creating new shallow root. It looks for remote: Total 1 in the fetch output and gets 3 instead. It fails for