Skip to content
This repository has been archived by the owner on Apr 14, 2022. It is now read-only.

Fixed error with battery notification and added dependencies to README #6

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zothma
Copy link

@zothma zothma commented Sep 6, 2020

Battery notifications

When battery percentage goes under 15%, awesome should send a warning notification every 5 minutes. However, there was an error every time it reached this percentage. Turned out that the error was coming from the use of _G.time() (which does not exist) instead of os.time() in the battery widget code.

Dependencies

This same battery widget is using the acpi package for getting the battery percentage. However, this package isn't installed by default on Arch.
The sound control slider and the keybindings for the sound (XF86Audio... keys) use the alsa-utils package, and especially its amixer command to handle the volume of the sound. However, this package isn't installed by default on Arch.

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

Successfully merging this pull request may close these issues.

1 participant