Bug#860340: fancontrol: please add configuration file

2017-04-24 Thread Aurelien Jarno
control: tag -1 + wontfix

On 2017-04-16 09:07, Hans-J. Ullrich wrote:
> Package: fancontrol
> Version: 1:3.4.0-4
> Followup-For: Bug #860340
> 
> Dear Maintainer,
> 
> if you are willing to add a default configuration to fancontrol or maybe 
> better, build a package for EEEPC maybe names fancontrol-eeepc, here is my 
> improved /etc/fancontrol. This is working well for live-file-cd's as long
> as there are no related entries in /etc/modules. 

As said in the other mail, we are not going to handle the thousand of
machines who need fancontrol, and create one package for each of them,
with the risk of overheating hardware.

As there is already an eeepc-acpi-scripts package, maybe it can be added
there?

-- 
Aurelien Jarno  GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aurel32.net



Bug#860340: fancontrol: please add configuration file

2017-04-24 Thread Aurelien Jarno
On 2017-04-14 21:10, Hans-J. Ullrich wrote:
> Package: fancontrol
> Version: 1:3.4.0-4
> Severity: wishlist
> 
> Dear Maintainer,
> 
> it would be nice, if you could add a default configuration file 
> (/etc/fancontrol) to the package.

This not something possible, actually each machine needs a different
configuration file. We do not want to handle the thousands of different
configuration which exists in the work, and additionally they might
actually conflict and thus break hardware.

> This is useful, when used in livefile systems. In my case, I am regularly 
> building a kali livefile dvd
> for my EEEPC, who needs fancontrol and eeepc-acpi-scripts (so fancontrol or 
> eeepc-acpi-scripts should have the related dependency). 
> After install, it is necessary to use pwmconfig, to create /etc/fancontrol. 
> Without it, the fan will not run,
> and the cpu can be overheated. 
> 
> So, the solution is the following:
> 
> 1. Build the livefile with the command "acpi_osi=Linux" at grub (my part)
> 2. Take care of installing fancontrol and eeepc-acpi-scripts (my part)
> 3. Create /etc/fancontrol at installation (your part)

If your live CD is specific to EEEPC, why can't you actually include it
as part of the live CD creation in the step 3?

> If you like, you can use mine for example, it is running fine on EEEPC.
> 
> Here it is:
> 
> -
> 
> # Configuration file generated by pwmconfig, changes will be lost
> INTERVAL=5
> DEVPATH=hwmon0= hwmon2=devices/platform/eeepc
> DEVNAME=hwmon0=acpitz hwmon2=eeepc
> FCTEMPS= hwmon2/pwm1=hwmon0/temp1_input
> FCFANS= hwmon2/pwm1=
> MINTEMP= hwmon2/pwm1=55
> MAXTEMP= hwmon2/pwm1=65
> MINSTART= hwmon2/pwm1=50
> MINSTOP= hwmon2/pwm1=35
> MINPWM= hwmon2/pwm1=0
> MAXPWM= hwmon2/pwm1=255

As said above, this is specific to an EEEPC machine, or even one variant
of a EEEPC. At best it will make fancontrol to fail to start on all
non-EEEPC machine. At worst it might be incompatible with some other 
variants of EEEPC.

-- 
Aurelien Jarno  GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aurel32.net



Bug#860340: fancontrol: please add configuration file

2017-04-16 Thread Hans-J. Ullrich
Package: fancontrol
Version: 1:3.4.0-4
Followup-For: Bug #860340

Dear Maintainer,

if you are willing to add a default configuration to fancontrol or maybe 
better, build a package for EEEPC maybe names fancontrol-eeepc, here is my 
improved /etc/fancontrol. This is working well for live-file-cd's as long
as there are no related entries in /etc/modules. 

I saw there is a bug report about, that tells, sensors-detect is changing the 
sort of the sensors, but on livefile-system this will not happen. So this is 
working on EEEPC.

 snip ---

# Configuration file generated by pwmconfig, changes will be lost
INTERVAL=10
DEVPATH=hwmon0= hwmon1=devices/platform/eeepc
DEVNAME=hwmon0=acpitz hwmon1=eeepc
FCTEMPS= hwmon1/pwm1=hwmon0/temp1_input
FCFANS= hwmon1/pwm1=
MINTEMP= hwmon1/pwm1=55
MAXTEMP= hwmon1/pwm1=65
MINSTART= hwmon1/pwm1=50
MINSTOP= hwmon1/pwm1=35
MINPWM= hwmon1/pwm1=0
MAXPWM= hwmon1/pwm1=255


- snap -

For the mnoment I am using self made packages with an integrated 
/etc/fancontrol. By interest, I can send you this package top see, what I 
changed.

Thanks for your work!

Best regards

Hans

-- System Information:
Debian Release: 9.0
  APT prefers testing
  APT policy: (500, 'testing'), (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 4.9.0-2-686-pae (SMP w/2 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages fancontrol depends on:
ii  init-system-helpers  1.47
ii  lsb-base 9.20161125

fancontrol recommends no packages.

fancontrol suggests no packages.

-- no debconf information



Bug#860340: fancontrol: please add configuration file

2017-04-14 Thread Hans-J. Ullrich
Package: fancontrol
Version: 1:3.4.0-4
Severity: wishlist

Dear Maintainer,

it would be nice, if you could add a default configuration file 
(/etc/fancontrol) to the package.
This is useful, when used in livefile systems. In my case, I am regularly 
building a kali livefile dvd
for my EEEPC, who needs fancontrol and eeepc-acpi-scripts (so fancontrol or 
eeepc-acpi-scripts should have the related dependency). 
After install, it is necessary to use pwmconfig, to create /etc/fancontrol. 
Without it, the fan will not run,
and the cpu can be overheated. 

So, the solution is the following:

1. Build the livefile with the command "acpi_osi=Linux" at grub (my part)
2. Take care of installing fancontrol and eeepc-acpi-scripts (my part)
3. Create /etc/fancontrol at installation (your part)

If you like, you can use mine for example, it is running fine on EEEPC.

Here it is:

-

# Configuration file generated by pwmconfig, changes will be lost
INTERVAL=5
DEVPATH=hwmon0= hwmon2=devices/platform/eeepc
DEVNAME=hwmon0=acpitz hwmon2=eeepc
FCTEMPS= hwmon2/pwm1=hwmon0/temp1_input
FCFANS= hwmon2/pwm1=
MINTEMP= hwmon2/pwm1=55
MAXTEMP= hwmon2/pwm1=65
MINSTART= hwmon2/pwm1=50
MINSTOP= hwmon2/pwm1=35
MINPWM= hwmon2/pwm1=0
MAXPWM= hwmon2/pwm1=255

---

Thank you for your help! 

Best regards

Hans



-- System Information:
Debian Release: 9.0
  APT prefers testing
  APT policy: (500, 'testing'), (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 4.9.0-2-686-pae (SMP w/2 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages fancontrol depends on:
ii  init-system-helpers  1.47
ii  lsb-base 9.20161125

fancontrol recommends no packages.

fancontrol suggests no packages.

-- no debconf information