Re: [PATCH] gnu: Add sdparm.

2016-02-05 Thread Ricardo Wurmus
Hi Tobias, > From: Tobias Geerinckx-Rice > > * gnu/packages/disk.scm (sdparm): New variable. I just pushed your patch with the suggested modifications from Andreas and myself. Thanks again! ~~ Ricardo

[PATCH] gnu: Add sdparm.

2016-02-02 Thread tobias . geerinckx . rice
From: Tobias Geerinckx-Rice * gnu/packages/disk.scm (sdparm): New variable. --- gnu/packages/disk.scm | 27 +++ 1 file changed, 27 insertions(+) diff --git a/gnu/packages/disk.scm b/gnu/packages/disk.scm index d806e0c..6e824d2 100644 ---

[PATCH] gnu: Add sdparm.

2016-02-02 Thread tobias . geerinckx . rice
From: Tobias Geerinckx-Rice * gnu/packages/disk.scm (sdparm): New variable. --- gnu/packages/disk.scm | 27 +++ 1 file changed, 27 insertions(+) diff --git a/gnu/packages/disk.scm b/gnu/packages/disk.scm index d806e0c..6e824d2 100644 ---

Re: [PATCH] gnu: Add sdparm.

2016-02-02 Thread Tobias Geerinckx-Rice
Ugh, Sorry about the spam. I was having trouble with git send-email + Gmail, and saw a few other replies make it into the archives a few hours ago, while mine didn't^H^H^H^Happeared not to. Apologies, and please disregard the other almost-duplicate thread: I got bored waiting and changed the

Re: [PATCH] gnu: Add sdparm.

2016-02-02 Thread Ricardo Wurmus
tobias.geerinckx.r...@gmail.com writes: > From: Tobias Geerinckx-Rice > > * gnu/packages/disk.scm (sdparm): New variable. > --- > gnu/packages/disk.scm | 27 +++ > 1 file changed, 27 insertions(+) This patch looks very good! Thank you!

Re: [PATCH] gnu: Add sdparm.

2016-02-02 Thread Tobias Geerinckx-Rice
On 2 February 2016 at 17:39, Andreas Enge wrote: > One minor point: > [...] A good one. So, for (my) future reference: re-writing upstream's descriptions for Guix is no problem? Glad to hear that. T G-R

Re: [PATCH] gnu: Add sdparm.

2016-02-02 Thread Andreas Enge
On Tue, Feb 02, 2016 at 05:50:48PM +0100, Tobias Geerinckx-Rice wrote: > A good one. So, for (my) future reference: re-writing upstream's > descriptions for Guix is no problem? No, we do it all the time, to remove advertisements for non-freeness, to drop superfluous references to freeness, to

Re: [PATCH] gnu: Add sdparm.

2016-02-02 Thread Tobias Geerinckx-Rice
On 2 February 2016 at 11:36, Ricardo Wurmus wrote: > > tobias.geerinckx.r...@gmail.com writes: > >> From: Tobias Geerinckx-Rice >> >> * gnu/packages/disk.scm (sdparm): New variable. >> --- >> gnu/packages/disk.scm | 27

Re: [PATCH] gnu: Add sdparm.

2016-02-02 Thread Andreas Enge
One minor point: On Tue, Feb 02, 2016 at 05:30:07PM +0100, Tobias Geerinckx-Rice wrote: > >> +These devices can be SCSI disks, in which case @command{sdparm}'s role is "the role of @command{sdparm}", since the command is not a person and thus does not take a genitive "s". Andreas