<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On May 5, 2017 15:45, "Augie Fackler" <<a href="mailto:raf@durin42.com">raf@durin42.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">On Fri, May 05, 2017 at 10:10:17AM -0700, Martin von Zweigbergk via Mercurial-devel wrote:<br>
> # HG changeset patch<br>
> # User Martin von Zweigbergk <<a href="mailto:martinvonz@google.com">martinvonz@google.com</a>><br>
> # Date 1494004116 25200<br>
> #      Fri May 05 10:08:36 2017 -0700<br>
> # Node ID 992b37360cb54124e8718f70cfccc1<wbr>ee3a3f8d44<br>
> # Parent  2cfdf5241096f6c0c2d45d32b2f1a4<wbr>1575835025<br>
> outgoing: run on filtered repo<br>
><br>
> outgoing has been using a filtered repo since fe67107094fd (discovery:<br>
<br>
</div>should this instance of filtered be unfiltered?<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Indeed it should. Thanks for reading carefully. Please fix in flight if you don't mind.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="elided-text"><br>
> outgoing pass unfiltered repo to findcommonincoming (issue3776),<br>
> 2013-01-28). If I'm reading code and history correctly, it should be<br>
> safe to run _outgoing() on a filtered repo since c5456b64eb07<br>
> (discovery: run discovery on filtered repository, 2015-01-07). By<br>
> running _outgoing() on a filtered repo, we can also remove the<br>
> workaround there for ignoring filtered revisions.<br>
><br>
> diff --git a/mercurial/hg.py b/mercurial/hg.py<br>
> --- a/mercurial/hg.py<br>
> +++ b/mercurial/hg.py<br>
> @@ -869,7 +869,7 @@<br>
>          revs = [repo.lookup(rev) for rev in scmutil.revrange(repo, revs)]<br>
><br>
>      other = peer(repo, opts, dest)<br>
> -    outgoing = discovery.findcommonoutgoing(<wbr>repo.unfiltered(), other, revs,<br>
> +    outgoing = discovery.findcommonoutgoing(<wbr>repo, other, revs,<br>
>                                              force=opts.get('force'))<br>
>      o = outgoing.missing<br>
>      if not o:<br>
> diff --git a/mercurial/scmutil.py b/mercurial/scmutil.py<br>
> --- a/mercurial/scmutil.py<br>
> +++ b/mercurial/scmutil.py<br>
> @@ -120,10 +120,6 @@<br>
>      secretlist = []<br>
>      if excluded:<br>
>          for n in excluded:<br>
> -            if n not in repo:<br>
> -                # discovery should not have included the filtered revision,<br>
> -                # we have to explicitly exclude it until discovery is cleanup.<br>
> -                continue<br>
>              ctx = repo[n]<br>
>              if ctx.phase() >= phases.secret and not ctx.extinct():<br>
>                  secretlist.append(n)<br>
</div>> ______________________________<wbr>_________________<br>
> Mercurial-devel mailing list<br>
> <a href="mailto:Mercurial-devel@mercurial-scm.org">Mercurial-devel@mercurial-scm.<wbr>org</a><br>
> <a href="https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/<wbr>mailman/listinfo/mercurial-<wbr>devel</a><br>
</blockquote></div><br></div></div></div>