<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 6, 2016 at 1:17 PM, Augie Fackler <span dir="ltr"><<a href="mailto:raf@durin42.com" target="_blank">raf@durin42.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Dec 6, 2016 at 2:16 PM, Jeremy Wall <<a href="mailto:jeremy@marzhillstudios.com">jeremy@marzhillstudios.com</a>> wrote:<br>
> Should the BC changes be in this change or should they be separate changes?<br>
<br>
</span>I think I'd make it a separate change for clarity.<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> On Tue, Dec 6, 2016 at 11:44 AM, Augie Fackler <<a href="mailto:raf@durin42.com">raf@durin42.com</a>> wrote:<br>
>><br>
>> On Tue, Dec 6, 2016 at 12:38 PM, Martin von Zweigbergk via<br>
>> Mercurial-devel <<a href="mailto:mercurial-devel@mercurial-scm.org">mercurial-devel@mercurial-<wbr>scm.org</a>> wrote:<br>
>> > On Tue, Dec 6, 2016 at 9:29 AM, Mateusz Kwapich <<a href="mailto:mitrandir@fb.com">mitrandir@fb.com</a>><br>
>> > wrote:<br>
>> >> Excerpts from Jeremy Wall (zaphar)'s message of 2016-12-02 13:12:20<br>
>> >> -0600:<br>
>> >>> diff -r 9e29d4e4e08b -r 9cb1540e417d tests/test-phases-exchange.t<br>
>> >>> --- a/tests/test-phases-exchange.t    Tue Nov 29 04:11:05 2016 -0800<br>
>> >>> +++ b/tests/test-phases-exchange.t    Wed Nov 30 15:55:42 2016 -0600<br>
>> >>> @@ -384,7 +384,7 @@<br>
>> >>>    $ hg push ../alpha # from nu<br>
>> >>>    pushing to ../alpha<br>
>> >>>    searching for changes<br>
>> >>> -  no changes found<br>
>> >>> +  sending phase public for 145e75495359<br>
>> >>>    [1]<br>
>> >><br>
>> >> I suppose now, that we are addmitting that there was something to push<br>
>> >> we should change the RC to 0. Question to others: would such change be<br>
>> >> considered a BC-breakage or a fix?<br>
>> ><br>
>> > Good point! The exit code is misleading, so I'd definitely say that's<br>
>> > a fix that should be done regardless of changing the "no changes<br>
>> > found" message.<br>
>><br>
>><br>
>> My sense is that we should do both, but not change the exit status<br>
>> until we're printing some text that explains why the exit status is<br>
>> zero. I view this as a bugfix that we should tag (BC).<br></div></div></blockquote><div><br></div><div>Okay so next steps. Do I amend this commit with the modifications and then re-email it? or do I create a new commit with the new changes and email that? </div></div><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Jeremy Wall<br><a href="http://jeremy.marzhillstudios.com" target="_blank">http://jeremy.marzhillstudios.com</a><br><a href="mailto:Jeremy@marzhillstudios.com" target="_blank">Jeremy@marzhillstudios.com</a></div>
</div></div>