Re: Add guix install command

2015-08-18 Thread Chris Marusich
Hi, That sounds like it could be a reasonable compromise to me, as long as it's clearly documented in the Texinfo and help text that it's an alias. However, it seems to me that it would duplicate the well known features of any shell (i.e. defining your own aliases or functions to make certain freq

Re: Add guix install command

2015-08-18 Thread Ludovic Courtès
Hi! I sympathize with the idea that ‘guix package -i’ and co. can be cumbersome to type and maybe non-obvious, so I would be happy to add ‘guix install’, ‘guix remove’, and ‘guix search’. I would prefer to document ‘guix package’ as the canonical interface though, for the reasons David and Chris

Re: Add guix install command

2015-08-18 Thread Ludovic Courtès
Chris Marusich skribis: > I think keeping things simple makes a lot of sense, especially since you > can add your own convenience wrapper easily using shell aliases etc. One could add a ‘guix-install’ alias, but not a ‘guix install’ alias, though. > Thanks for listening to my opinion - Guix is

Re: Add guix install command

2015-08-07 Thread Chris Marusich
Hi, As a new user myself, I think I'd like to add my own opinion. When the subcommands are explicitly namespaced (as in "guix package -i foo"), I find it easier to (1) understand and (2) look up usage information. In my opinion, shoving many disparate actions (and thus, options) into a single glo

Re: Add guix install command

2015-08-07 Thread Alex Kost
Mathieu Lirzin (2015-08-07 13:08 +0300) wrote: > Alex Kost writes: > >> Andy Wingo (2015-08-05 12:36 +0300) wrote: >>[...] >>> Attached is a patch to implement "guix install". It just copies over >>> functionality from guix package. We could refactor more but I would >>> start with this as it i

Re: Add guix install command

2015-08-07 Thread Alex Kost
Andy Wingo (2015-08-07 11:45 +0300) wrote: > On Fri 07 Aug 2015 09:47, Alex Kost writes: > >> Andy Wingo (2015-08-05 12:36 +0300) wrote: >> >> I don't mind the new commands, but I think this patch is unacceptable >> because of the huge code duplicating. Sorry if it was sounded rude > > Better to

Re: Add guix install command

2015-08-07 Thread Andy Wingo
Thanks for the comments. Just a couple points, happy to wait for Ludovic to get back. On Fri 07 Aug 2015 14:22, "Thompson, David" writes: >> In the case of "guix install", yes. However what would you show for >> "guix install --help"? What would error messages show: "guix package" >> or "guix

Re: Add guix install command

2015-08-07 Thread Thompson, David
On Fri, Aug 7, 2015 at 8:01 AM, Andy Wingo wrote: > On Fri 07 Aug 2015 10:48, Ricardo Wurmus writes: > Attached is a patch to implement "guix install". It just copies over functionality from guix package. We could refactor more but I would start with this as it is; when things ne

Re: Add guix install command

2015-08-07 Thread Andy Wingo
On Fri 07 Aug 2015 10:48, Ricardo Wurmus writes: >>> Attached is a patch to implement "guix install". It just copies over >>> functionality from guix package. We could refactor more but I would >>> start with this as it is; when things need changes is the time to >>> refactor, at least for me.

Re: Add guix install command

2015-08-07 Thread Mathieu Lirzin
Alex Kost writes: > Andy Wingo (2015-08-05 12:36 +0300) wrote: >[...] >> Attached is a patch to implement "guix install". It just copies over >> functionality from guix package. We could refactor more but I would >> start with this as it is; when things need changes is the time to >> refactor,

Re: Add guix install command

2015-08-07 Thread Ricardo Wurmus
>> Attached is a patch to implement "guix install". It just copies over >> functionality from guix package. We could refactor more but I would >> start with this as it is; when things need changes is the time to >> refactor, at least for me. What do you think? :) > > I don't mind the new command

Re: Add guix install command

2015-08-07 Thread Andy Wingo
On Fri 07 Aug 2015 09:47, Alex Kost writes: > Andy Wingo (2015-08-05 12:36 +0300) wrote: > > I don't mind the new commands, but I think this patch is unacceptable > because of the huge code duplicating. Sorry if it was sounded rude Better to just not be rude, please. I know "this patch is unac

Re: Add guix install command

2015-08-07 Thread Alex Kost
Andy Wingo (2015-08-05 12:36 +0300) wrote: > Hi, > > I think Guix's tooling is OK but it could be better. As a user -- and > I'm really trying to be a user and not a developer :-) -- my main > interactions with a Guix system are installing, searching, and > upgrading. Sometimes mucking about wit

Re: Add guix install command

2015-08-06 Thread Mathieu Lirzin
I really welcome this patch! :) > From 7f1ac0b748a40b0a093d20c27401f305ce6aec9f Mon Sep 17 00:00:00 2001 > From: Andy Wingo > Date: Wed, 5 Aug 2015 11:27:48 +0200 > Subject: [PATCH] Add guix install command > > * guix/scripts/install.scm: New file. > * doc/guix.texi: Change guix package -i invocations to

Add guix install command

2015-08-05 Thread Andy Wingo
he time to refactor, at least for me. What do you think? :) Regards, Andy >From 7f1ac0b748a40b0a093d20c27401f305ce6aec9f Mon Sep 17 00:00:00 2001 From: Andy Wingo Date: Wed, 5 Aug 2015 11:27:48 +0200 Subject: [PATCH] Add guix install command * guix/scripts/install.scm: New file. * doc/guix