all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Thompson, David" <dthompson2@worcester.edu>
To: David Craven <david@craven.ch>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH 4/4] environment: Add '--debug' flag.
Date: Tue, 19 Jul 2016 13:14:04 -0400	[thread overview]
Message-ID: <CAJ=RwfbbH7ohJWU4Jr2n2BWLSp5FmGYHR6m743jObfgnHSAHBg@mail.gmail.com> (raw)
In-Reply-To: <20160719164916.6597-5-david@craven.ch>

Hi David,

On Tue, Jul 19, 2016 at 12:49 PM, David Craven <david@craven.ch> wrote:
> * guix/scripts/environment.scm: Add '--debug' flag to add common debugging
> packages (file, gdb, strace, valgrind) to environment.

Thanks for the patch and trying to make 'guix environment' more
friendly, but I don't really like this particular change for a few
reasons:

- The name "--debug" is misleading.  When I saw the patch subject
line, I thought it was a way to get 'guix environment' to print some
verbose output for the sake of debugging the generated environments.
As a user, I wouldn't expect --debug to affect the set of packages in
the environment.

- The debugging tools that a developer would want vary depending on
the project.  I don't use GDB or valgrind to debug my Guile programs,
for example.

- It tightly couples 'guix environment' to a particular, opinionated
set of packages.  What if the user wants to use a different variant of
gdb?

- Tools like strace are Linux-specific but we have plans to support
additional kernels like the Hurd.

I don't perceive a usability problem in this area, but that doesn't
mean there isn't one.  Could you explain your motivations behind this
patch?  Perhaps there's an alternative thing we could do, but in
general I don't envision 'guix environment' having a deep knowledge of
the available packages and having opinions about which packages users
should choose.

- Dave

  reply	other threads:[~2016-07-19 17:14 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-19 16:49 [PATCH 0/4] Dipping my toes into guix source David Craven
2016-07-19 16:49 ` [PATCH 1/4] lint: Add packages with suffix ':bin' to 'inputs-should-be-native' David Craven
2016-07-20 10:37   ` Ludovic Courtès
2016-07-20 10:50     ` David Craven
2016-07-20 11:19       ` Efraim Flashner
2016-07-21 12:24       ` Ludovic Courtès
2016-07-21 13:22         ` Vincent Legoll
2016-07-21 16:06           ` Ludovic Courtès
2016-07-21 19:27             ` Tobias Geerinckx-Rice
2016-07-22 13:32               ` Ludovic Courtès
2016-07-22 14:07                 ` Vincent Legoll
2016-07-19 16:49 ` [PATCH 2/4] gnu: Rename 'glib' label to 'glib:bin' where appropriate David Craven
2016-07-22 12:57   ` Ludovic Courtès
2016-07-22 13:03     ` David Craven
2016-07-22 17:55       ` David Craven
2016-07-19 16:49 ` [PATCH 3/4] lint: Add 'inputs-should-be-sorted' check David Craven
2016-07-21 13:17   ` Eric Bavier
2016-07-21 16:00     ` Ludovic Courtès
2016-07-19 16:49 ` [PATCH 4/4] environment: Add '--debug' flag David Craven
2016-07-19 17:14   ` Thompson, David [this message]
2016-07-19 18:00     ` Tobias Geerinckx-Rice
2016-07-19 18:10       ` David Craven
2016-07-19 19:02         ` David Craven
2016-07-19 19:06           ` Thompson, David
2016-07-19 19:15             ` David Craven
2016-07-20  4:17           ` Leo Famulari
2016-07-20 10:35         ` Ludovic Courtès
2016-07-20 10:56           ` David Craven
2016-07-19 20:11       ` Danny Milosavljevic
2016-07-20  7:18         ` Alex Kost
2016-07-20  7:35           ` Vincent Legoll
2016-07-20  9:15           ` Pjotr Prins
2016-07-22 19:54         ` Customising linux-libre (was: [PATCH 4/4] environment: Add '--debug' flag.) Tobias Geerinckx-Rice
2016-07-22 19:58           ` Vincent Legoll
2016-07-22 20:00             ` David Craven
2016-07-22 20:03               ` Vincent Legoll
2016-07-22 20:15               ` Customising linux-libre Tobias Geerinckx-Rice
2016-07-22 20:50                 ` David Craven
2016-07-22 20:11             ` Tobias Geerinckx-Rice
2016-07-23 10:38               ` Ludovic Courtès
2016-07-21 12:22   ` [PATCH 4/4] environment: Add '--debug' flag Ludovic Courtès
2016-07-21 12:30     ` David Craven
2016-07-21 16:03       ` Ludovic Courtès
2016-07-21 16:12         ` David Craven
2016-07-21 16:38           ` Ludovic Courtès

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='CAJ=RwfbbH7ohJWU4Jr2n2BWLSp5FmGYHR6m743jObfgnHSAHBg@mail.gmail.com' \
    --to=dthompson2@worcester.edu \
    --cc=david@craven.ch \
    --cc=guix-devel@gnu.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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.