WayfireWM / WayfireWM/wf-shell

panel : brightness : crash on startup if current user doesn't have permissions

Open
#529 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
179
Forks
55
Avg merge
16d 1h
Merged PRs (30d)
3

Description

If the current user doesn't have the necessary permissions for communicating on the i2c bus used by ddcutil, enumeration of displays by ddcutil will crash (throw ?), which will of course kill the panel.

  • add to the wiki an example udev rule to grant permissions
  • avoid the crash if conditions are met

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file or test is named. Start by tracing panel startup through ddcutil display enumeration and reproduce the failure for a user without i2c permissions; done means the panel remains running when enumeration cannot proceed, with a regression check for that condition.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
desktop
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.