Subject: Re: [PATCH] git-rebase.sh: Update USAGE string

From: Jakub Narebski <[email-masked]>
Date: Sun, 03 Feb 2008 13:14:21 -0800 (PST)

Jari Aalto <[email-masked]> writes:

> Present the options in -s|--long (short, long) order.

This is a good change (but might not make sense without the following
one)...

> Mention merge and new whitespace option.

...and this is not; usage string is now too long.

Please either use placeholder ([<whitespace>]), or enumerate
only short version of commands ([-i] [-v] [-m]) in synopsis,
or break synopsis into lines 80-column lines max.

> -USAGE='[--interactive | -i] [-v] [--onto <newbase>] <upstream> [<branch>]'
> +USAGE='[-i|--interactive] [-v|--verbose] [--whitespace={nowarn|warn|error|error-all|strip}] [-m|--merge] [--onto <newbase>] <upstream> [<branch>]'

-- 
Jakub Narebski
Poland
ShadeHawk on #git