diff options
| author | Keir Fraser <keir.fraser@citrix.com> | 2009-12-18 07:53:27 +0000 |
|---|---|---|
| committer | Keir Fraser <keir.fraser@citrix.com> | 2009-12-18 07:53:27 +0000 |
| commit | 344e13af46cdb531897899ac40708d431af0a1af (patch) | |
| tree | 8a072f30ccf0d7897c566c932e3e51bd33daf3de /tools/xenstore | |
| parent | e63db5c63e322d13826da893255efaa58e521266 (diff) | |
| download | xen-344e13af46cdb531897899ac40708d431af0a1af.tar.gz xen-344e13af46cdb531897899ac40708d431af0a1af.tar.bz2 xen-344e13af46cdb531897899ac40708d431af0a1af.zip | |
xenstore: Fix the method of get options and the usage
Add long option '--flat' correspond to short option '-f',
and let it just can be used for subcommand 'ls' (because
in fact it's useless for subcommand 'read' and 'list').
And fix the usage of subcommands 'ls', 'list' and 'chmod'.
Signed-off-by: Yu Zhiguo <yuzg@cn.fujitsu.com>
Diffstat (limited to 'tools/xenstore')
| -rw-r--r-- | tools/xenstore/xenstore_client.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/tools/xenstore/xenstore_client.c b/tools/xenstore/xenstore_client.c index 9d00c18259..aaf3014d53 100644 --- a/tools/xenstore/xenstore_client.c +++ b/tools/xenstore/xenstore_client.c @@ -88,13 +88,13 @@ usage(enum mode mode, int incl_mode, const char *progname) mstr = incl_mode ? "exists " : ""; case MODE_list: mstr = mstr ? : incl_mode ? "list " : ""; - errx(1, "Usage: %s %s[-h] [-s] key [...]", progname, mstr); + errx(1, "Usage: %s %s[-h] [-p] [-s] key [...]", progname, mstr); case MODE_ls: mstr = mstr ? : incl_mode ? "ls " : ""; - errx(1, "Usage: %s %s[-h] [-s] [path]", progname, mstr); + errx(1, "Usage: %s %s[-h] [-f] [-p] [-s] [path]", progname, mstr); case MODE_chmod: mstr = incl_mode ? "chmod " : ""; - errx(1, "Usage: %s %s[-h] [-s] key <mode [modes...]>", progname, mstr); + errx(1, "Usage: %s %s[-h] [-u] [-r] [-s] key <mode [modes...]>", progname, mstr); } } @@ -492,15 +492,16 @@ main(int argc, char **argv) int c, index = 0; static struct option long_options[] = { {"help", 0, 0, 'h'}, + {"flat", 0, 0, 'f'}, /* MODE_ls */ {"socket", 0, 0, 's'}, - {"prefix", 0, 0, 'p'}, /* MODE_read || MODE_list */ + {"prefix", 0, 0, 'p'}, /* MODE_read || MODE_list || MODE_ls */ {"tidy", 0, 0, 't'}, /* MODE_rm */ {"upto", 0, 0, 'u'}, /* MODE_chmod */ {"recurse", 0, 0, 'r'}, /* MODE_chmod */ {0, 0, 0, 0} }; - c = getopt_long(argc - switch_argv, argv + switch_argv, "fhsptur", + c = getopt_long(argc - switch_argv, argv + switch_argv, "hfsptur", long_options, &index); if (c == -1) break; @@ -510,7 +511,7 @@ main(int argc, char **argv) usage(mode, switch_argv, argv[0]); /* NOTREACHED */ case 'f': - if ( mode == MODE_read || mode == MODE_list || mode == MODE_ls ) { + if ( mode == MODE_ls ) { max_width = INT_MAX/2; desired_width = 0; show_whole_path = 1; |