[PATCH] push: move handling of explicitly pushed bookmarks into localrepo.push()

Kevin Bullock kbullock+mercurial at ringworld.org
Fri Dec 7 13:22:43 CST 2012


On Dec 7, 2012, at 1:19 PM, Augie Fackler wrote:

> On Dec 7, 2012, at 12:51 PM, Kevin Bullock <kbullock+mercurial at ringworld.org> wrote:
> 
>> As a side effect, the undocumented exit code 2 for push (indicating
>> that bookmarks explicitly listed couldn't be synched) is eliminated.
> 
> It's undocumented, but I wonder if this would break people. Seems like that should be done as a separate change with (bc) in the summary line?

You're right, should probably be changed before moving the code into localrepo. I also missed some test breakage (largefiles, test-hook), so I'll resend in a minute.

pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
Kevin R. Bullock



More information about the Mercurial-devel mailing list