After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 747851 - core: add nm-plugin-missing property for devices
core: add nm-plugin-missing property for devices
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-04-14 13:55 UTC by Jiri Klimes
Modified: 2015-04-20 08:19 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Jiri Klimes 2015-04-14 13:55:12 UTC
NetworkManager now supports various devices via plugins. If a plugin is not installed, it may not be clear to the user why the device is not working properly.

So let's add nm-plugin-missing property for devices and set it to true when the plugin is missing.
Comment 1 Jiri Klimes 2015-04-14 13:56:49 UTC
Code is in jk/nm-plugin-missing-bgo747851 branch.
Comment 2 Thomas Haller 2015-04-14 14:19:39 UTC
this is a helpful feature IMO.

missing "Since: 1.2" gtkdoc for nm_device_get_nm_plugin_missing() and PROP_NM_PLUGIN_MISSING (and possibly others?).
Comment 3 Jiri Klimes 2015-04-15 08:29:21 UTC
(In reply to Thomas Haller from comment #2)
> this is a helpful feature IMO.
> 
> missing "Since: 1.2" gtkdoc for nm_device_get_nm_plugin_missing() and
> PROP_NM_PLUGIN_MISSING (and possibly others?).

Fixed.
Comment 4 Dan Williams 2015-04-16 21:32:56 UTC
> device: add nm-plugin-missing property indicating NM device plugin not available

nm-device-private.h is really only supposed to be used by nm-device-*.c, so I'd put nm_device_set_nm_plugin_missing() into nm-device.h instead.

Rest looks fine.
Comment 5 Jiri Klimes 2015-04-20 08:19:02 UTC
(In reply to Dan Williams from comment #4)
> > device: add nm-plugin-missing property indicating NM device plugin not available
> 
> nm-device-private.h is really only supposed to be used by nm-device-*.c, so
> I'd put nm_device_set_nm_plugin_missing() into nm-device.h instead.
>

Fixed.

Pushed to master:
80a72a4 merge: add new nm-plugin-missing property for devices (bgo #747851)
1b9c1ed cli: add NM-PLUGIN-MISSING to devices
560e195 core: set nm-plugin-missing device property
4c398ab libnm: add nm-plugin-missing property indicating NM device plugin not available
fc6373b device: add nm-plugin-missing property indicating NM device plugin not available
823df33 cli: better indicate when a Wi-Fi plugin might not be available (rh #1168573)