This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author bethard
Recipients asampson, bethard, eric.araujo, georg.brandl, lukasz.langa, rhettinger
Date 2010-12-17.15:47:16
SpamBayes Score 4.3171337e-07
Marked as misclassified No
Message-id <1292600838.39.0.711739780819.issue9234@psf.upfronthosting.co.za>
In-reply-to
Content
The patch looks basically okay to me, though this line makes me nervous:

  dest += ' (%s)' % ', '.join(aliases)

Since this is just for help formatting, can't you just modify metavar instead? The dest is the attribute on the namespace where the result should be stored. The metavar is the value that should be displayed in help messages.

As to where the aliases should be printed, I don't have a strong preference. The svn aliases show up when you do a generic "svn help" (but not if you do a "svn help blame") and looks like:

Available subcommands:
   add
   blame (praise, annotate, ann)
   ...

The hg aliases show up when you do a "hg help commit" (but not if you do a "hg help") and looks like:

hg commit [OPTION]... [FILE]...

aliases: ci

I guess the patch makes it pretty easy to emulate the svn version.
History
Date User Action Args
2010-12-17 15:47:18bethardsetrecipients: + bethard, georg.brandl, rhettinger, eric.araujo, asampson, lukasz.langa
2010-12-17 15:47:18bethardsetmessageid: <1292600838.39.0.711739780819.issue9234@psf.upfronthosting.co.za>
2010-12-17 15:47:16bethardlinkissue9234 messages
2010-12-17 15:47:16bethardcreate