Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 922870 - mate-base/mate-applets-1.26.1 unconditionally depends on net-wireless/wireless-tools
Summary: mate-base/mate-applets-1.26.1 unconditionally depends on net-wireless/wireles...
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: immolo
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-01-24 20:53 UTC by rajas
Modified: 2024-01-26 17:51 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
mate-applets-1.26.1-r1-wireless-tools-conditionally.patch (mate-applets-1.26.1-r1-wireless-tools-conditionally.patch,855 bytes, patch)
2024-01-24 20:59 UTC, rajas
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description rajas 2024-01-24 20:53:44 UTC
The package mate-base/mate-applets-1.26.1 unconditionally depends on >=net-wireless/wireless-tools-28_pre9:0 even if the use flag "netlink" is set.

According to `configure.ac` the package either uses libnl OR iwlib/wireless-tools. So the dependency on net-wireless/wireless-tools should be depend on the "netlink" use flag.

Reproducible: Always

Steps to Reproduce:
1. USE='netlink' emerge -vp mate-applets
Actual Results:  
2. dependency "net-wireless/wireless-tools" gets pulled in
3. :-(

Expected Results:  
The package can be built without "net-wireless/wireless-tools" when using the "netlink" use flag.

https://github.com/mate-desktop/mate-applets/blob/d7165dfbde04c06b1c9c4040d1db7ac9db544118/configure.ac#L417

The line in the upstream source code stating there can only be libnl OR iwlib/wireless-tools.
Comment 1 rajas 2024-01-24 20:59:19 UTC
Created attachment 883061 [details, diff]
mate-applets-1.26.1-r1-wireless-tools-conditionally.patch

Patch to conditionally depend on net-wireless/wireless-tools
Comment 2 OzTiram 2024-01-25 18:45:42 UTC
I think this report and the patch make sense.
Thanks for suggesting the fix.
Comment 3 rajas 2024-01-26 17:51:43 UTC
Thank you! You are welcome.