Re: [PATCH v2 00/16] btrfs-progs: global verbose and quiet option

From: Anand Jain <anand.jain@oracle.com>
To: dsterba@suse.cz, David Sterba <dsterba@suse.com>,
	linux-btrfs@vger.kernel.org
Subject: Re: [PATCH v2 00/16] btrfs-progs: global verbose and quiet option
Date: Fri, 12 Jun 2020 02:13:29 +0800	[thread overview]
Message-ID: <17ad44fb-3e32-76f8-50f6-88160e7225fe@oracle.com> (raw)
In-Reply-To: <20200605092413.GA27795@twin.jikos.cz>

> Also, many options that have their own --verbose option should update
> the help text to note that it's an alias of the global.
> 
> I'm going to take another look today, the scope of the change might be
> too big to do in one go so some incremental steps might be needed.

  As global verbose depends on the local verbose to log messages, if
  local verbose used stderr then the global verbose uses the stderr.
  For example
  patch:
     btrfs-progs: send: use global verbose and quiet options

  ------------------
   ./btrfs send --help
  <snip>
     -v|--verbose     enable verbose output to stderr, each occurrence of
                      this option increases verbosity
     -q|--quiet       suppress all messages, except errors

     Global options:
     -v|--verbose       increase output verbosity
     -q|--quiet         print only errors
  ------------------

  IMO verbose should be stdout. Should it be ok to change to stdout as it
  is under verbose?

Thanks.