-
-
Notifications
You must be signed in to change notification settings - Fork 286
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
Not installable on Fedora Silverblue 36 (35 too) #394
Comments
Not sure how you installed auto-cpufreq (git or snap package), but I see references to read-only problems with Silveblue on Snap for Fedora and github issue, seems to be Silveblue specific issue and nothing related to auto-cpufreq. |
It is a silverblue specific issue since it's built around a read only base system. @AdnanHodzic There are ways that auto-cpufreq could work around this, by
cc @MiMillieuh Mind confirming if auto-cpufreq works in a toolbox? I also intend to switch to silverblue later |
While I would definitely like for auto-cpufreq to work properly on Fedora Silveblue. Since I don't use it and I don't have much time to look into it, if one of you would like to give it a go, I always encourage code contributions! |
Installed Silverblue, I seem to have got it working with modifications. Although I worry that these silverblue specific modifications might interfere with other distributions, which wouldn't be preferred. Due to this, I am going to make a fork of the repository specifically for silverblue, hope you understand 👍🏻 |
https://github.com/gimbles/auto-cpufreq-silverblue @MiMillieuh Enjoy! Remember to install it outside of a toolbox. |
@gimbles good stuff. But I would rather have this code as part of auto-cpufreq instead of auto-cpufreq README pointing to a fork. Especially since from beginning auto-cpufreq was made to work and works great with some many different distributions. Hence, as I pointed out before, please feel free to make these changes as part of auto-cpufreq. |
Installing in toolbox would be pointless since its not system |
Hi @AdnanHodzic While I understand why you want to merge this into the main repository, my edits might cause issues on non-silverblue distributions. Unfortunately, I don't particularly have the time to write a proper solution, I just did a quick grep replace in the source code and that was pretty much it. However, you may look into the commit diff and figure out how to implement this into the main project. Hope you understand 👋 |
At least it works like a charm ^^ Thanks |
Were the changes eventually merged? The fork is gone :( |
No, PR was never created ... @gimbles @MiMillieuh do you happen to have those changes on one of your branches? It would really be great if a PR could be created for auto-cpufreq to have these changes in its |
I don't have them anymore, The fork got deleted so I guess no more autocpufreq for silverblue. Maybe ask the original owner of that fork, he might have it somewhere on his computer or on a private branch I don't know. |
Great work @gimbles, changes merged with auto-cpufreq |
Hello,
I know this issue was fixed in the past but it seems to happen again. (see #129)
I was used to install it on Fedora workstation and it was working fine but since silverblue is on a read only system it seems to be unable to install it...
Also snap package doesn't seem to work
By the way keep going your software is super useful ^^
Thanks
Error output:
System information:
OS: Fedora Linux 36.20220510.0 (Silverblue) x86_64
Kernel: 5.17.5-300.fc36.x86_64
CPU: AMD Ryzen 7 4700U with Radeon Graphics (8) @ 2.000GHz
GPU: AMD ATI 04:00.0 Renoir
Add/paste output of:
Without providing requested information your issue might be ignored!
If installation failed with auto-cpufreq-installer have you tried installing auto-cpufreq using Snap package?
The text was updated successfully, but these errors were encountered: