bshah created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  - It looks if we are on FreeBSD, then it sets the USE_SYSCTL
  - If USE_SYSCTL is set, it will try to use acpi_video interface to set
  
  the brightness
  
  - However, this whole upower backend depends on libudev which is not
  
  available on FreeBSD and it doesn't compile and link with FreeBSD
  
  I propose to remove this code from here, and add it back when we split
  out the backlighthelper from the upower backend

TEST PLAN
  builds and works on Linux

REPOSITORY
  R122 Powerdevil

BRANCH
  remove-unreachable-code

REVISION DETAIL
  https://phabricator.kde.org/D5778

AFFECTED FILES
  daemon/backends/upower/backlighthelper.cpp
  daemon/backends/upower/backlighthelper.h

To: bshah, #plasma, #freebsd, broulik
Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, lukas

Reply via email to