On Thu, Sep 20, 2012 at 11:17:01PM +0200, Pawel Jakub Dawidek wrote:
> On Wed, Sep 19, 2012 at 04:28:38PM +0000, Baptiste Daroussin wrote:
> > Author: bapt
> > Date: Wed Sep 19 16:28:38 2012
> > New Revision: 240698
> > URL: http://svn.freebsd.org/changeset/base/240698
> > 
> > Log:
> >   Update usage to show the possiblity to use jail name
> >   
> >   Reported by:      bdrewery
> >   MFC after:        1 month
> > 
> > Modified:
> >   head/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c
> > 
> > Modified: head/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c
> > ==============================================================================
> > --- head/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c        Wed Sep 19 
> > 16:21:23 2012        (r240697)
> > +++ head/cddl/contrib/opensolaris/cmd/zfs/zfs_main.c        Wed Sep 19 
> > 16:28:38 2012        (r240698)
> > @@ -240,9 +240,9 @@ get_usage(zfs_help_t idx)
> >             return (gettext("\tupgrade [-v]\n"
> >                 "\tupgrade [-r] [-V version] <-a | filesystem ...>\n"));
> >     case HELP_JAIL:
> > -           return (gettext("\tjail <jailid> <filesystem>\n"));
> > +           return (gettext("\tjail <jailid|jailname> <filesystem>\n"));
> 
> Please keep the style consistent. As you can see two lines above there
> are spaces around |.
> 

Sure but both syntax are present in the usage: <filesystem|volume|snapshot> in
fact the only one with space around | is the line above you are showing, all the
rest of the usage is without space, That is why I did it without spaces.

If you tell me it is better with spaces, I'll add space.

regards,
Bapt

Attachment: pgpaFkc10tY6b.pgp
Description: PGP signature

Reply via email to