On Tue, Dec 6, 2016 at 12:38 PM, Martin von Zweigbergk via Mercurial-devel <mercurial-devel@mercurial-scm.org> wrote: > On Tue, Dec 6, 2016 at 9:29 AM, Mateusz Kwapich <mitran...@fb.com> wrote: >> Excerpts from Jeremy Wall (zaphar)'s message of 2016-12-02 13:12:20 -0600: >>> diff -r 9e29d4e4e08b -r 9cb1540e417d tests/test-phases-exchange.t >>> --- a/tests/test-phases-exchange.t Tue Nov 29 04:11:05 2016 -0800 >>> +++ b/tests/test-phases-exchange.t Wed Nov 30 15:55:42 2016 -0600 >>> @@ -384,7 +384,7 @@ >>> $ hg push ../alpha # from nu >>> pushing to ../alpha >>> searching for changes >>> - no changes found >>> + sending phase public for 145e75495359 >>> [1] >> >> I suppose now, that we are addmitting that there was something to push >> we should change the RC to 0. Question to others: would such change be >> considered a BC-breakage or a fix? > > Good point! The exit code is misleading, so I'd definitely say that's > a fix that should be done regardless of changing the "no changes > found" message.
My sense is that we should do both, but not change the exit status until we're printing some text that explains why the exit status is zero. I view this as a bugfix that we should tag (BC). _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel