On Tue, Aug 25, 2020 at 11:38:04AM -0700, Nelson Elhage wrote: > It's unfortunately a bit unclear to me what _should_ happen here. We > really want a way to mark `89ea0c5` as "partially-shallow", and send > its second parent, but not its first parent, but shallowness is a > property of an entire commit, not of a specific commit/parent > relationship. However, it'd be nice if we at least ended up with a > consistent state, instead of with a repository with invalid `shallow` > marks. I think this is the same issue I reported recently here: https://lore.kernel.org/git/20200721160643.GA3288097@xxxxxxxxxxxxxxxxxxxxxxx/ AFAICT the shallow feature is just defective and can't accurately represent this situation. Unfortunately nobody seemed to have any bright ideas, and the developer who implemented most of the shallow features (including shallow-since) is no longer active. So I suspect it is fixable, but probably requires somebody to get pretty familiar with the shallow code, and propose a fix that involves both code and a protocol change. -Peff
