[PATCH 5 of 6 V3] hgweb: blacklist heavyweight revset functions in hgweb search

Kevin Bullock kbullock+mercurial at ringworld.org
Thu Sep 12 22:11:48 CDT 2013

On 12 Sep 2013, at 7:39 PM, Augie Fackler wrote:

> On Sep 12, 2013, at 7:03 PM, Matt Mackall <mpm at selenic.com> wrote:
>> I _really_ should just dump absolutely everything from you currently in
>> my inbox so I can get back on top of things, but that means I would miss
>> existing review comments to figure out whether your future patches are
>> acceptable and what's already been addressed.
> I think Kevin and I may have exacerbated your problem. At this point, I think Kevin and I have gone through everything that came in while you were at BM, so perhaps just declare bankruptcy explicitly and we'll move on?

Augie went through patchwork and "deferred" everything from Alexander that wasn't already dealt with, at which point I went through _my_ backlog and deleted them all as well. The 4(!) series still in patchwork are all(?) re-sends or updates of the stuff we dropped due to congestion before. I agree, I think you can declare inbox bankruptcy here and move on.

Alexander, one thing that might be helpful here is if you use a --flag for each series that ties the different _versions_ of a series together. (Clearly that doesn't help for series you've already sent several versions of.)

I'm about to take a final look at that V5 (beginning with "revset: add helper function to get revset parse tree depth") and likely crew it.

pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
Kevin R. Bullock

More information about the Mercurial-devel mailing list