Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Startup script for systemd (fancurve) #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fireheadman
Copy link

Welcome to use this... I have it running on my system and it works great!

This file needs to go into /lib/systemd/system/fancurve.service
Then do reload systemd and enable the service, followed by a start and verify
sudo systemctl daemon-reload
sudo systemd enable fancurve
sudo systemctl status fancurve -l
@FedoraTipper
Copy link
Owner

FedoraTipper commented Apr 12, 2018

Rather package this with an install script, which we can build upon at a later stage. In the current state, it is not advisable to merge.

Take a look at https://github.com/FedoraTipper/Nvidia-Fan-Curve-Linux/tree/install-script

@fireheadman
Copy link
Author

absolutely.... I'm just throwing what I use for now. An installer method would be great!

One thing I will add that I noticed last night was after implementing this I would not get my prompt back unless I used --no-pager option....otherwise, it works quite well.

@FedoraTipper
Copy link
Owner

I've spent a bit of time on the script. https://github.com/FedoraTipper/Nvidia-Fan-Curve-Linux/blob/install-script/src/install.pl Not by any stretch of the imagination even close to complete.

I've decided perl might be the best for what I want the script to do. Both quick and easy. Take a look, and let me know if there is anything that needs to be adjusted. I've also reconstructed the folders & files, as well as adding a bit of regex points for the script to substitute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants