[PATCHv2] Re: Convert to use OptionParser for command line handling.
nicolas.s-dev at laposte.net
Tue Dec 14 21:49:14 GMT 2010
On Mon, Dec 13, 2010 at 06:15:06PM -0600, Sebastian wrote:
> From: Sebastian Spaeth <Sebastian at SSpaeth.de>
> THe main purpose of this patch is to move the command line option
> handling from the more manual "getopts" module to the full fledged
Would change to:
Move the command line option handling from the more manual "getopts"
module to the full fledged "OptionParser".
> It has the advantage of e.g. automatically creating
> the Help text that is shown with the "-h/--help" option rather than
> having to hardcode the text and format the line breaking manually.
> (admittedly in a beautifully justified manner :-)).
> Also, do away with the versino check when invoking OfflineImap so we
...and "versino" to "version".
> do not have to change the version number in three places when bumping
> Signed-Off-By: Sebastian Spaeth <Sebastian at SSpaeth.de>
> This patch addresses all comments in Nicolas' review.
> Thanks for spotting all that!!!
> I suggest that further wording improvments go into separate patches though.
Yes. But the current patch does not apply.
More information about the OfflineIMAP-project