FS#77608 - [powerdevil] Demote networkmanager-qt and bluez-qt to optdepends

Attached to Project: Arch Linux
Opened by Emil (xexaxo) - Tuesday, 21 February 2023, 22:12 GMT
Last edited by Antonio Rojas (arojas) - Wednesday, 22 February 2023, 07:58 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Antonio Rojas (arojas)
Felix Yan (felixonmars)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

I realise there has been many bug reports about this, although I would ask the maintainers to read until the end before closing as duplicate.

On the tracker we have many similar requests, roughly grouped in two:
- "remove" networkmanager/bluez
Not possible since it will break people who do use the NM and friends

- demote networkmanager/bluez to optional dependency
Possible today, see below for more.

If you look closely at the code you'll see that the nm/bluez functionality is limited to the action, which is it's own module (on-demand loaded DSO). As such neither the executables nor the main library powerdevilcore actually link to it.
Yes, currently the code claims that powerdevilcore does link [1], although that is an actual bug (over-linking) and I've sent a patch upstream [2].

I believe when the original reported came out, we did not have the automatic "as-needed" in our default linker options. Thus we were all mislead that the "executables" needed NM/Bluez.
Since that change, the overlinking is automatically resolved and things just work. See reproducer below for step-by-step guide


Additional info:
* package version(s)
powerdevil 5.27.1-1

* link to upstream bug report, if any
[1] https://github.com/KDE/powerdevil/commit/829ab7996660520dd8c342c71f8d5c351c52f39b
[2] https://invent.kde.org/plasma/powerdevil/-/merge_requests/136


Steps to reproduce:
- install plasma and friends
- pacman -R networkmanager-qt --assume-installed networkmanager-qt
- reboot
- start plasma -> system settings -> Power management
- note that the wireless power management "page" will be missing
- all other Power management pages and toggles work fine
This task depends upon

Closed by  Antonio Rojas (arojas)
Wednesday, 22 February 2023, 07:58 GMT
Reason for closing:  Implemented
Additional comments about closing:  powerdevil 5.27.1-2
Comment by Toolybird (Toolybird) - Wednesday, 22 February 2023, 02:01 GMT
With your patch merged, it does indeed seem feasible now.
Comment by Antonio Rojas (arojas) - Wednesday, 22 February 2023, 07:57 GMT
Nice. This has actually nothing to do with --as-needed, the functionality was moved to a plugin in upstream commit https://invent.kde.org/plasma/powerdevil/-/commit/3aa08ef577b295ea81387d3008c7cb02d9d8ccac

Loading...