On 9/22/2017 8:39 PM, Jonathan Tan wrote:
On Fri, 22 Sep 2017 20:26:19 +0000 Jeff Hostetler <[email protected]> wrote:This draft contains filters to: () omit all blobs () omit blobs larger than some size () omit blobs using a sparse-checkout specification In addition to specifying the filter criteria, the rev-list command was updated to include options to: () print a list of the omitted objects (due to the current filtering criteria) () print a list of missing objects (probably from a prior partial clone/fetch).Thanks, this last point seems useful. I tried applying your patches and it doesn't apply cleanly on master. Could you try rebasing? In particular, the code references get_sha1_with_context(), which no longer exists (it was replaced with get_oid_with_context()).
I usually build and submit patches relative to the latest available tag (v2.14.1) rather than a moving target, but yeah I'll look thru the other comments and rebase it forward. Thanks Jeff

