[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add thinkfan.
From: |
Leo Famulari |
Subject: |
Re: [PATCH] Add thinkfan. |
Date: |
Sun, 1 May 2016 17:14:18 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
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?
- Re: [PATCH] Add thinkfan.,
Leo Famulari <=
- Re: [PATCH] Add thinkfan., Kei Yamashita, 2016/05/01
- Re: [PATCH] Add thinkfan., Nicolas Goaziou, 2016/05/03
- Re: [PATCH] Add thinkfan., Leo Famulari, 2016/05/06
- Re: [PATCH] Add thinkfan., Nicolas Goaziou, 2016/05/06
- Re: [PATCH] Add thinkfan., Leo Famulari, 2016/05/06
- Re: [PATCH] Add thinkfan., Nicolas Goaziou, 2016/05/07
- Re: [PATCH] Add thinkfan., Ludovic Courtès, 2016/05/09
- Re: [PATCH] Add thinkfan., Nicolas Goaziou, 2016/05/10
- Re: [PATCH] Add thinkfan., Ludovic Courtès, 2016/05/11