public inbox for lvm2-cvs@sourceware.org help / color / mirror / Atom feed
From: zkabelac@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 ./WHATS_NEW_DM tools/dmsetup.c Date: Tue, 03 Aug 2010 12:56:00 -0000 [thread overview] Message-ID: <20100803125601.26621.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: zkabelac@sourceware.org 2010-08-03 12:56:00 Modified files: . : WHATS_NEW_DM tools : dmsetup.c Log message: Fix return status 0 for "dmsetup info -c -o help". Solution returns success for _report_init when help is passed, and caller needs to check for _report existance. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW_DM.diff?cvsroot=lvm2&r1=1.401&r2=1.402 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/tools/dmsetup.c.diff?cvsroot=lvm2&r1=1.142&r2=1.143 --- LVM2/WHATS_NEW_DM 2010/08/03 07:56:03 1.401 +++ LVM2/WHATS_NEW_DM 2010/08/03 12:56:00 1.402 @@ -1,5 +1,6 @@ Version 1.02.54 - ================================ + Fix return status 0 for "dmsetup info -c -o help". Add check for kernel semaphore support and disable udev_sync if not available. Version 1.02.53 - 28th July 2010 --- LVM2/tools/dmsetup.c 2010/07/09 15:34:48 1.142 +++ LVM2/tools/dmsetup.c 2010/08/03 12:56:00 1.143 @@ -2652,6 +2652,9 @@ r = 1; out: + if (!strcasecmp(options, "help") || !strcmp(options, "?")) + r = 1; + if (len) dm_free(options); @@ -3360,8 +3363,15 @@ if (!_switches[COLS_ARG] && !strcmp(c->name, "splitname")) _switches[COLS_ARG]++; - if (_switches[COLS_ARG] && !_report_init(c)) - goto out; + if (_switches[COLS_ARG]) { + if (!_report_init(c)) + goto out; + if (!_report) { + if (!strcmp(c->name, "info")) + r = 0; /* info -c -o help */ + goto out; + } + } #ifdef UDEV_SYNC_SUPPORT if (!_set_up_udev_support(dev_dir))
next reply other threads:[~2010-08-03 12:56 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-08-03 12:56 zkabelac [this message] -- strict thread matches above, loose matches on Subject: below -- 2012-04-24 8:24 prajnoha 2012-04-24 8:00 prajnoha 2012-03-01 21:56 zkabelac 2012-02-15 12:08 prajnoha 2012-02-15 12:06 prajnoha 2012-02-15 12:03 prajnoha 2012-02-13 12:06 zkabelac 2011-08-11 17:06 mbroz 2011-07-08 17:08 agk 2011-06-17 14:55 prajnoha 2011-03-29 21:49 zkabelac 2011-03-02 2:44 agk 2010-08-03 13:04 zkabelac 2010-07-08 14:29 zkabelac 2009-06-03 20:44 agk 2009-06-03 18:35 agk 2009-04-24 11:30 prajnoha
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20100803125601.26621.qmail@sourceware.org \ --to=zkabelac@sourceware.org \ --cc=lvm-devel@redhat.com \ --cc=lvm2-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).