[PATCH 1 of 2 V2] push: hide description about "-f" in the hint to prevent from using it easily
Matt Mackall
mpm at selenic.com
Wed Sep 18 18:42:16 CDT 2013
On Wed, 2013-09-04 at 22:27 +0900, FUJIWARA Katsunori wrote:
> # HG changeset patch
> # User FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
> # Date 1378300768 -32400
> # Wed Sep 04 22:19:28 2013 +0900
> # Node ID aa8788937c2e5b505b4096f48536249d29499829
> # Parent 73513cb8c379af12f3dfb9ae2dcab864239481ae
> push: hide description about "-f" in the hint to prevent from using it easily
>
> "use push -f to force" in the hint at abortion of "hg push" may cause
> novice users to execute "push -f" easily without understanding about
> problems of multiple branch heads in the repository.
>
> This patch hides description about "-f" in the hint, and leads into
> seeing "hg help push" for detail about pushing multiple heads.
>
> This patch also recommends to ask the management policy of the
> destination repository before using "-f".
Looks like there's some good ideas in here.
> diff --git a/mercurial/commands.py b/mercurial/commands.py
> --- a/mercurial/commands.py
> +++ b/mercurial/commands.py
> @@ -4648,7 +4648,10 @@
> only create a new branch without forcing other changes.
>
> Use -f/--force to override the default behavior and push all
> - changesets on all branches.
> + changesets on all branches. Before using --force, it is
> + recommended to ask the administrator of the destination repository
> + for the management policy of it. Pushing multiple heads in
> + particular is usually a mistake.
Let's make this its own patch (so we can bikeshed it separately). I
think we should definitely say that using --force is probably a mistake,
but the 'ideal' Mercurial environment doesn't have admins to consult.
> abort: push creates new remote head c922c0139ca0!
> - (did you forget to merge? use push -f to force)
> + (did you forget to merge? see "hg help push" for detail about pushing multiple heads)
Length problem. How about:
(did you forget to merge? see "hg help push" for more details)
> - (you should pull and merge or use push -f to force)
> + (you should pull and merge or see "hg help push" for detail about pushing multiple heads)
Another length problem. We don't usually use 'you should', so perhaps:
(pull and merge or see "hg help push")
--
Mathematics is the supreme nostalgia of our time.
More information about the Mercurial-devel
mailing list