<div dir="ltr"><div>Queued.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 23, 2019 at 11:34 PM Pierre-Yves David <<a href="mailto:pierre-yves.david@ens-lyon.org">pierre-yves.david@ens-lyon.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Gentle ping on that patch. All the feedback from the previous round <br>
should have been applied.<br>
<br>
On 4/17/19 4:37 PM, Pierre-Yves David wrote:<br>> diff --git a/mercurial/configitems.py b/mercurial/configitems.py<br>
> --- a/mercurial/configitems.py<br>
> +++ b/mercurial/configitems.py<br>
> @@ -529,6 +529,13 @@ coreconfigitem('experimental', 'evolutio<br>
>   coreconfigitem('experimental', 'evolution.track-operation',<br>
>       default=True,<br>
>   )<br>
> +# repo-level config to exclude a revset visibility<br>
> +#<br>
> +# The target use case is to use `share` to expose different subset of the same<br>
> +# repository, especially server side. See also `server.view`.<br>
> +coreconfigitem('experimental', 'extra-filter-revs',<br></blockquote><div><br></div><div>I think it would be good to clarify in the name if these revisions are added or removed from the view, but this is okay as long as it's experimental.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> diff --git a/mercurial/repoview.py b/mercurial/repoview.py<br>
> --- a/mercurial/repoview.py<br>
> +++ b/mercurial/repoview.py<br>> @@ -154,6 +158,35 @@ filtertable = {'visible': computehidden,<br>> +    combine = lambda fname: fname + '%' + fid<br></blockquote><div><br></div><div>nit: "combine" is very generic. Something like "filename = lambda prefix: ..." might be clearer.</div><div><br></div></div></div>