search

nan0s7 / nfancurve

A small and lightweight POSIX script for using a custom fan curve in Linux for those with an Nvidia GPU.
GNU General Public License v3.0
314 stars 57 forks source link

maybe change default config path? #17

Closed cj360 closed 4 years ago

cj360 commented 5 years ago

This may just be an issue from the aur not this script, so if this should be on the aur's page not here let me know.

In this script's case since the install dir is the default config location, it seems bad that its looking for a config among /usr/bin. On the other hand, it seems weird to ask the person that made the aur package to change the original script. Just a nitpick really not actually a problem if I just cloned the repo.

nan0s7 commented 5 years ago

Yeah this can't really be helped unless the default path is forced to be somewhere in the users home directory. It does suit that the default config location is the install location because the main script has no sense of what the default values should be.

But of course that is why there's an option to run the script with a config path command line option.

ReneHollander commented 4 years ago

In the new AUR package the config is located in /etc/nfancurve.conf. The provided systemd user unit will use that config.

Just beware, enabling the nfancurve.service does not start it at boot. I am still trying to get it to work, but running systemd --user start nfancurve.service works fine.