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

Add ledcontrol #1516

Closed
wants to merge 7 commits into from
Closed

Add ledcontrol #1516

wants to merge 7 commits into from

Conversation

Juliandev02
Copy link

So, i added Support for armhf.

  • I have designed an icon. With Paint :)
    And Pi 3 is supported anyway, since the two use the same LED brightness map.
    More Details in:

@Sussy-OS
Copy link
Contributor

Nice!

@Sussy-OS
Copy link
Contributor

@Juliandev02 when I try to run it, it shows up this: sh: 1: cannot create /sys/class/leds/led0/brightness: Permission denied sh: 1: cannot create /sys/class/leds/led0/brightness: Permission denied sh: 1: cannot create /sys/class/leds/led0/brightness: Permission denied

#!/bin/bash

#Allow packages required by this app to be uninstalled
sudo rm /usr/bin/ledcontrol
Copy link
Owner

Choose a reason for hiding this comment

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

Use rm -f to prevent this script from failing if /usr/bin/ledcontrol has already been deleted.

Copy link
Author

Choose a reason for hiding this comment

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

@@ -0,0 +1,3 @@
Do it simple - Control the status LEDs of your Raspberry Pi!
A simple program to control the status LEDs of your Raspberry Pi!
Copy link
Owner

Choose a reason for hiding this comment

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

Lines 1 and 2 seem to be very similar.

@Botspot
Copy link
Owner

Botspot commented Feb 21, 2022

You may want to re-design the icon - right now it depicts a Pi, not its LEDs.
No, this is not just me being picky, this is me observing that apps with meaningful and interesting icons have more users than apps that don't.

@Sussy-OS
Copy link
Contributor

You may want to re-design the icon - right now it depicts a Pi, not its LEDs. No, this is not just me being picky, this is me observing that apps with meaningful and interesting icons have more users than apps that don't.

Do you have any idea why I get those errors I mentioned?

@Juliandev02
Copy link
Author

@Juliandev02 when I try to run it, it shows up this: sh: 1: cannot create /sys/class/leds/led0/brightness: Permission denied sh: 1: cannot create /sys/class/leds/led0/brightness: Permission denied sh: 1: cannot create /sys/class/leds/led0/brightness: Permission denied

You need to run with sudo.

@Sussy-OS
Copy link
Contributor

ah, okay

@Juliandev02 Juliandev02 closed this by deleting the head repository Sep 2, 2022
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.

5 participants