[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add sdparm.
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH] gnu: Add sdparm. |
Date: |
Tue, 2 Feb 2016 11:36:17 +0100 |
address@hidden writes:
> From: Tobias Geerinckx-Rice <address@hidden>
>
> * gnu/packages/disk.scm (sdparm): New variable.
> ---
> gnu/packages/disk.scm | 27 +++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
This patch looks very good! Thank you!
> + (description "Sdparm reads and modifies SCSI device parameters.
> +These devices can be SCSI disks, in which case @command{sdparm}'s role is
> +similar to its namesake: the Linux @command{hdparm} utility originally
> +designed for ATA disks. However, @command{sdparm} can be used to access
> +parameters on any device that uses a SCSI command set. Such devices
> +include CD/DVD drives (irrespective of transport), SCSI and ATAPI tape
> +drives, and SCSI enclosures. A small set of commands associated with
> +starting and stopping the media, loading and unloading removable media
> +and some other housekeeping functions can also be sent with this utility.")
I would remove “Linux” from the description as neither “sdparm” nor
“hdparm” seem to have anything to do with the kernel.
“[...] functions can [...] be sent” sounds odd to me. It probably is
meant to be understood as “commands associated with [...] housekeeping
functions”, but that’s not how I parsed it at first. Maybe it would be
clearer to reorder the sentence as “This utility can also send commands
associated with [this, that, and] some other housekeeping functions”.
If you’re okay with these changes I’ll push it after these slight
modifications.
Thanks again and welcome!
~~ Ricardo
Re: [PATCH] gnu: Add sdparm., Ricardo Wurmus, 2016/02/05