<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Nov 3, 2017, at 15:01, Hollis Blanchard <<a href="mailto:hollis_blanchard@mentor.com" class="">hollis_blanchard@mentor.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" class="">
<div text="#000000" bgcolor="#FFFFFF" class="">
On 11/03/2017 11:25 AM, Hollis Blanchard wrote:<br class="">
<blockquote type="cite" cite="mid:666d3aaffadc4e92acfe.1509733515@aurora.wv.mentorg.com" class="">
<pre wrap="" class=""># HG changeset patch
# User Hollis Blanchard <a class="moz-txt-link-rfc2396E" href="mailto:hollis_blanchard@mentor.com"><hollis_blanchard@mentor.com></a>
# Date 1509731952 25200
# Fri Nov 03 10:59:12 2017 -0700
# Branch stable
# Node ID 666d3aaffadc4e92acfe3e4098ae8f2fa4086b7d
# Parent f445b10dc7fb3495d24d1c22b0996148864c77f7
outgoing: respect ":pushurl" paths
Make 'hg outgoing' respect "paths.default:pushurl" in addition to
"paths.default-push".
</pre>
</blockquote>
Here's some context I didn't think belonged in the commit message:<br class="">
<br class="">
'hg outgoing' has always meant "what will happen if I run 'hg
push'?" and it's<br class="">
still documented that way:<br class="">
<br class="">
Show changesets not found in the specified destination
repository or the<br class="">
default push location. These are the changesets that would be
pushed if a<br class="">
push was requested.<br class="">
<br class="">
If the user uses the now-deprecated "paths.default-push" path, it
continues to<br class="">
work that way. However, as described at<br class="">
<a class="moz-txt-link-freetext" href="https://bz.mercurial-scm.org/show_bug.cgi?id=5365">https://bz.mercurial-scm.org/show_bug.cgi?id=5365</a>, it doesn't behave
the same<br class="">
with "paths.default:pushurl".<br class=""></div></div></blockquote><div><br class=""></div><div>Yep, clear bug.</div><div><br class=""></div><div>This context is totally fine in a commit message. As I mentioned on IRC, maybe fold the test into test-outgoing.t? I can find time for it (next week probably, given how today has gone) if you don't have time, but if you're willing to do a v2 with that, could you also put (issue5365) at the end of the first line of the commit message so our bug bot notices?</div><div><br class=""></div><div>Thanks!</div><br class=""><blockquote type="cite" class=""><div class=""><div text="#000000" bgcolor="#FFFFFF" class="">
<br class="">
Why does it matter? Similar to the bugzilla reporter, I have a
read-only mirror<br class="">
of a non-Mercurial repository:<br class="">
<br class="">
<pre class="">upstream -> imported mirror -> user clone
^-----------------------/</pre>
<br class="">
Users push directly to upstream, and that content is then imported
into the<br class="">
mirror. However, those repositories are not the same; it's possible
that the<br class="">
mirroring has either broken completely, or an import process is
running and not<br class="">
yet complete. In those cases, 'hg outgoing' will list changesets
that have<br class="">
already been pushed.<br class="">
<br class="">
Mozilla's desired behavior described in bug 5365 can be accomplished
through<br class="">
other means (e.g. 'hg outgoing default'), preserving the consistency
and<br class="">
meaning of 'hg outgoing'.<br class="">
<br class="">
<pre class="moz-signature" cols="72">Hollis Blanchard <a class="moz-txt-link-rfc2396E" href="mailto:hollis_blanchard@mentor.com"><hollis_blanchard@mentor.com></a>
Mentor Graphics Emulation Division</pre>
</div>
_______________________________________________<br class="">Mercurial-devel mailing list<br class=""><a href="mailto:Mercurial-devel@mercurial-scm.org" class="">Mercurial-devel@mercurial-scm.org</a><br class="">https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel<br class=""></div></blockquote></div><br class=""></body></html>