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.

Sign up for GitHub

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

Jump to bottom

Update README.md #179

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #179

wants to merge 1 commit into from

Conversation

hydrian
Copy link

@hydrian hydrian commented Dec 11, 2019

  1. Cleaned up some grammar issues.
  2. Also, I moved the sudo file from /etc/sudoers to /etc/sudoers.d/radeon-profile for better support over OS upgrades and to prevent conflicting issues.

@hydrian
Update README.md
28b4f87 
Cleaned up some grammar issues.
Also, I moved the sudo file from /etc/sudoers to /etc/sudoers.d/radeon-profile for better support over OS upgrades and to prevent conflicting issues.
Oxalin
Oxalin suggested changes Dec 31, 2019
View reviewed changes
Copy link
Contributor

@Oxalin Oxalin left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please complete modifications with the following ones in bold:
"Otherwise, the app needs to be run with root privileges to change the power profiles (and clocks readings sometimes). You can create a text file containing username ALL = NOPASSWD: /usr/bin/radeon-profile under /etc/sudoers.d/radeon-profile. Here is a tip to run the app as a normal user. Note that this involves changing permissions to system files: http://bit.ly/1dvQMhS"

@some-username-here1
Copy link

some-username-here1 commented Jul 19, 2020 •
edited
Loading

Since they haven't responded since, I've included changes from hydrian's fork into my own, followed by overlaying Oxalin's corrections on top of it, plus some other changes in the README.md too.
some-username-here1@35e435d

@Oxalin
Copy link
Contributor

Oxalin commented Jan 20, 2023

Since the development seems to have stalled, I applied it to my fork. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Oxalin Oxalin Oxalin requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hydrian @some-username-here1 @Oxalin