mbox series

[bug#34020,0/2] Re-purpose '--verbosity' to something useful

Message ID 20190109133145.1144-1-ludo@gnu.org
Headers show
Series Re-purpose '--verbosity' to something useful | expand

Message

Ludovic Courtès Jan. 9, 2019, 1:31 p.m. UTC 
Hello Guix!

These patches re-purposes ‘--verbosity’ so that it better matches
user expectations.  The previous ‘--verbosity’ option, which is
about the daemon’s debugging output, is renamed to ‘--debug’.
In addition, ‘--verbosity’ now has a shorthand ‘-v’.

Most commands that build stuff support -v/--verbosity so users can
easily override the default verbosity level.

Thoughts?

Thanks,
Ludo’.

Ludovic Courtès (2):
  status: Add 'with-status-verbosity'.
  guix build: Re-purpose '--verbosity' and add '--debug'.

 .dir-locals.el                 |   1 +
 build-aux/run-system-tests.scm |   4 +-
 doc/guix.texi                  |  30 +++++---
 guix/scripts/build.scm         | 134 +++++++++++++++++----------------
 guix/scripts/environment.scm   |  14 +++-
 guix/scripts/pack.scm          |  14 +++-
 guix/scripts/package.scm       |  23 ++++--
 guix/scripts/pull.scm          |  12 ++-
 guix/scripts/system.scm        |  18 +++--
 guix/status.scm                |  17 ++++-
 10 files changed, 165 insertions(+), 102 deletions(-)

Comments

Mike Gerwitz Jan. 10, 2019, 3:38 a.m. UTC | #1 
On Wed, Jan 09, 2019 at 14:31:45 +0100, Ludovic Courtès wrote:
> These patches re-purposes ‘--verbosity’ so that it better matches
> user expectations.  The previous ‘--verbosity’ option, which is
> about the daemon’s debugging output, is renamed to ‘--debug’.
> In addition, ‘--verbosity’ now has a shorthand ‘-v’.
>
> Most commands that build stuff support -v/--verbosity so users can
> easily override the default verbosity level.
>
> Thoughts?

(Just having looked at the patches, without actually trying it out.)

This is much better, thank you!  I was confused by the previous behavior.
Ludovic Courtès Jan. 11, 2019, 11:15 a.m. UTC | #2 
Hi,

Mike Gerwitz <mtg@gnu.org> skribis:

> On Wed, Jan 09, 2019 at 14:31:45 +0100, Ludovic Courtès wrote:
>> These patches re-purposes ‘--verbosity’ so that it better matches
>> user expectations.  The previous ‘--verbosity’ option, which is
>> about the daemon’s debugging output, is renamed to ‘--debug’.
>> In addition, ‘--verbosity’ now has a shorthand ‘-v’.
>>
>> Most commands that build stuff support -v/--verbosity so users can
>> easily override the default verbosity level.
>>
>> Thoughts?
>
> (Just having looked at the patches, without actually trying it out.)
>
> This is much better, thank you!  I was confused by the previous behavior.

Thanks for your feedback, pushed now!

I’ve also adjust ‘guix archive’, which I had forgotten in the patch I
sent.

Ludo’.