From mboxrd@z Thu Jan  1 00:00:00 1970
From: Kei Yamashita <kei@openmailbox.org>
Subject: Re: [PATCH] Add thinkfan.
Date: Sun, 1 May 2016 17:24:53 -0400
Message-ID: <20160501172453.5d9f2327@openmailbox.org>
References: <87mvoay9wo.fsf@saiph.selenimh>
	<20160501211418.GA10221@jasmine>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:52179)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kei@openmailbox.org>) id 1awys0-0001kM-4x
	for guix-devel@gnu.org; Sun, 01 May 2016 17:25:46 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <kei@openmailbox.org>) id 1awyre-0003VY-Sw
	for guix-devel@gnu.org; Sun, 01 May 2016 17:25:34 -0400
Received: from mail2.openmailbox.org ([62.4.1.33]:37467)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kei@openmailbox.org>) id 1awyrc-0003QB-H9
	for guix-devel@gnu.org; Sun, 01 May 2016 17:25:18 -0400
In-Reply-To: <20160501211418.GA10221@jasmine>
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel/>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: "Guix-devel" <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
To: leo@famulari.name
Cc: guix-devel@gnu.org

On Sun, 1 May 2016 17:14:18 -0400
Leo Famulari <leo@famulari.name> wrote:

> On Sun, May 01, 2016 at 01:05:59AM +0200, Nicolas Goaziou wrote:
> > * gnu/packages/linux.scm (thinkfan): New variable.  
> 
> Thanks!
> 
> > +    (arguments
> > +     `(#:tests? #f                      ;no test target
> > +       #:configure-flags
> > +       `("-DUSE_ATASMART:BOOL=ON")))  
> 
> Can you add a brief comment explaining this flag?
> 
> Also, can you add a phase that installs the "rcscripts"? They will be
> useful for Guix users on foreign distros.
> 
> I'm not sure where to install them. Since they will probably have to
> be copied or symlinked out of ~/.guix-profile anyways, how about
> installing them into 'share/misc'?
> 
> I'm open to better ideas!
> 
> > +    (synopsis "Simple fan control program")  
> 
> Is it specific to ThinkPads? If so, can you change the synopsis to
> "Simple fan control program for ThinkPads" and mention the limitation
> in the description?
> 

It would be helpful if you told the user where and how to install each
script for each of the major distribution families, similar to how Guix
does this for SystemD and Upstart systems[1].

[1]
https://gnu.org/software/guix/manual/html_node/Binary-Installation.html