* [gentoo-commits] gentoo-x86 commit in net-misc/networkmanager/files: networkmanager-0.9.1.90-force-libnl1.1.patch networkmanager-0.9.1.90-rfkill.patch networkmanager-0.9.1.90-if.h.patch 01-org.freedesktop.NetworkManager.settings.modify.system.pkla
@ 2011-10-29 6:30 Alexandre Rostovtsev (tetromino)
0 siblings, 0 replies; only message in thread
From: Alexandre Rostovtsev (tetromino) @ 2011-10-29 6:30 UTC (permalink / raw
To: gentoo-commits
tetromino 11/10/29 06:30:47
Added: networkmanager-0.9.1.90-force-libnl1.1.patch
networkmanager-0.9.1.90-rfkill.patch
networkmanager-0.9.1.90-if.h.patch
01-org.freedesktop.NetworkManager.settings.modify.system.pkla
Log:
Bump to 0.9.1.90 from the gnome overlay. Allow users in plugdev group to modify system connections (so dropped wireless connections no longer bring up a modal root password prompt), thanks to Samuli Suominen for the solution. Numerous code changes.
(Portage version: 2.2.0_alpha71/cvs/Linux x86_64)
Revision Changes Path
1.1 net-misc/networkmanager/files/networkmanager-0.9.1.90-force-libnl1.1.patch
file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.1.90-force-libnl1.1.patch?rev=1.1&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.1.90-force-libnl1.1.patch?rev=1.1&content-type=text/plain
Index: networkmanager-0.9.1.90-force-libnl1.1.patch
===================================================================
Build against libnl:1.1 only. Otherwise, networkmanager will link to
libnl-3.so and to libiWmxSdk (from wimax-1.5.2) which links to libnl.so,
and then explode spectacularly at runtime due to symbol collisions.
diff --git a/configure.ac b/configure.ac
index 117dd91..b8b11de 100644
--- a/configure.ac
+++ b/configure.ac
@@ -323,24 +323,6 @@ if (test "${have_libnl1}" = "yes"); then
have_libnl="yes"
fi
-PKG_CHECK_MODULES(LIBNL2, libnl-2.0, [have_libnl2=yes], [have_libnl2=no])
-if (test "${have_libnl2}" = "yes"); then
- AC_DEFINE(HAVE_LIBNL2, 1, [Define if you require specific libnl-2 support])
- LIBNL_CFLAGS="$LIBNL2_CFLAGS"
- LIBNL_LIBS="$LIBNL2_LIBS"
- libnl_version="2"
- have_libnl="yes"
-fi
-
-PKG_CHECK_MODULES(LIBNL3, libnl-3.0, [have_libnl2=yes], [have_libnl2=no])
-if (test "${have_libnl2}" = "yes"); then
- AC_DEFINE(HAVE_LIBNL3, 1, [Define if you require specific libnl-3 support])
- LIBNL_CFLAGS="$LIBNL3_CFLAGS"
- LIBNL_LIBS="$LIBNL3_LIBS"
- libnl_version="3"
- have_libnl="yes"
-fi
-
if (test "${have_libnl}" = "no"); then
AC_MSG_ERROR([libnl development header are required])
fi
1.1 net-misc/networkmanager/files/networkmanager-0.9.1.90-rfkill.patch
file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.1.90-rfkill.patch?rev=1.1&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.1.90-rfkill.patch?rev=1.1&content-type=text/plain
Index: networkmanager-0.9.1.90-rfkill.patch
===================================================================
From 339229e4c698c61e20a28bfc33d8501490891427 Mon Sep 17 00:00:00 2001
From: Gary Ching-Pang Lin <chingpang@gmail.com>
Date: Tue, 20 Sep 2011 08:36:35 +0000
Subject: core: improving handling of rfkill (bgo #655773)
This commit improves the handling of rfkill.
- The original two passes check gathers the states of platform
and non-platform switches in two separate loops. Now we gather
the both states in one loop and determine the final states later.
- A new rule is used to determine the states of switches.
if (platform_state == UNBLOCKED)
choose non_platform_state;
else
choose platform_state;
The state is UNBLOCKED if and only if both the platform and
non-platform switches are unblocked, so the ambiguous state in
bgo#655773 will not happen.
Original code always preferred the platform switch state over
the device switch state, so if the platform switch was UNBLOCKED
but the device was BLOCKED, NM would treat the device as
UNBLOCKED and try to activate it, and obviously fail.
---
diff --git a/src/nm-udev-manager.c b/src/nm-udev-manager.c
index 72501c2..3e855b7 100644
--- a/src/nm-udev-manager.c
+++ b/src/nm-udev-manager.c
@@ -195,78 +195,50 @@ recheck_killswitches (NMUdevManager *self)
NMUdevManagerPrivate *priv = NM_UDEV_MANAGER_GET_PRIVATE (self);
GSList *iter;
RfKillState poll_states[RFKILL_TYPE_MAX];
+ RfKillState platform_states[RFKILL_TYPE_MAX];
gboolean platform_checked[RFKILL_TYPE_MAX];
int i;
/* Default state is unblocked */
for (i = 0; i < RFKILL_TYPE_MAX; i++) {
poll_states[i] = RFKILL_UNBLOCKED;
+ platform_states[i] = RFKILL_UNBLOCKED;
platform_checked[i] = FALSE;
}
- /* Perform two passes here; the first pass is for non-platform switches,
- * which typically if hardkilled cannot be changed except by a physical
- * hardware switch. The second pass checks platform killswitches, which
- * take precedence over device killswitches, because typically platform
- * killswitches control device killswitches. That is, a hardblocked device
- * switch can often be unblocked by a platform switch. Thus if we have
- * a hardblocked device switch and a softblocked platform switch, the
- * combined state should be softblocked since the platform switch can be
- * unblocked to change the device switch.
- */
-
- /* Device switches first */
+ /* Poll the states of all killswitches */
for (iter = priv->killswitches; iter; iter = g_slist_next (iter)) {
Killswitch *ks = iter->data;
GUdevDevice *device;
RfKillState dev_state;
int sysfs_state;
- if (ks->platform == FALSE) {
- device = g_udev_client_query_by_subsystem_and_name (priv->client, "rfkill", ks->name);
- if (device) {
- sysfs_state = g_udev_device_get_property_as_int (device, "RFKILL_STATE");
- dev_state = sysfs_state_to_nm_state (sysfs_state);
+ device = g_udev_client_query_by_subsystem_and_name (priv->client, "rfkill", ks->name);
+ if (device) {
+ sysfs_state = g_udev_device_get_property_as_int (device, "RFKILL_STATE");
+ dev_state = sysfs_state_to_nm_state (sysfs_state);
+ if (ks->platform == FALSE) {
if (dev_state > poll_states[ks->rtype])
poll_states[ks->rtype] = dev_state;
- g_object_unref (device);
- }
- }
- }
-
- /* Platform switches next; their state overwrites device state */
- for (iter = priv->killswitches; iter; iter = g_slist_next (iter)) {
- Killswitch *ks = iter->data;
- GUdevDevice *device;
- RfKillState dev_state;
- int sysfs_state;
-
- if (ks->platform == TRUE) {
- device = g_udev_client_query_by_subsystem_and_name (priv->client, "rfkill", ks->name);
- if (device) {
- sysfs_state = g_udev_device_get_property_as_int (device, "RFKILL_STATE");
- dev_state = sysfs_state_to_nm_state (sysfs_state);
-
- if (platform_checked[ks->rtype] == FALSE) {
- /* Overwrite device state with platform state for first
- * platform switch found.
- */
- poll_states[ks->rtype] = dev_state;
- platform_checked[ks->rtype] = TRUE;
- } else {
- /* If there are multiple platform switches of the same type,
- * take the "worst" state for all of that type.
- */
- if (dev_state > poll_states[ks->rtype])
- poll_states[ks->rtype] = dev_state;
- }
- g_object_unref (device);
+ } else {
+ platform_checked[ks->rtype] = TRUE;
+ if (dev_state > platform_states[ks->rtype])
+ platform_states[ks->rtype] = dev_state;
}
+ g_object_unref (device);
}
}
/* Log and emit change signal for final rfkill states */
for (i = 0; i < RFKILL_TYPE_MAX; i++) {
+ if (platform_checked[i] == TRUE) {
+ /* blocked platform switch state overrides device state, otherwise
+ * let the device state stand. (bgo #655773)
+ */
+ if (platform_states[i] != RFKILL_UNBLOCKED)
+ poll_states[i] = platform_states[i];
+ }
+
if (poll_states[i] != priv->rfkill_states[i]) {
nm_log_dbg (LOGD_RFKILL, "%s rfkill state now '%s'",
rfkill_type_to_desc (i),
--
cgit v0.9.0.2-2-gbebe
1.1 net-misc/networkmanager/files/networkmanager-0.9.1.90-if.h.patch
file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.1.90-if.h.patch?rev=1.1&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/networkmanager-0.9.1.90-if.h.patch?rev=1.1&content-type=text/plain
Index: networkmanager-0.9.1.90-if.h.patch
===================================================================
commit 00f2b0a9bb806be64c5868af44f43dbe55bdd75b
Author: Thomas Graf <tgraf@redhat.com>
Date: Fri Sep 23 13:46:41 2011 +0200
core: Include <linux/if.h> instead of <net/if.h>
NM already includes <linux/if.h> in some places, f.e. nm-netlink-monitor and
we can't mix usage of the two. Stick to using <linux/if.h> as it provides
additional flag definitions such as operational link state and link mode.
Signed-off-by: Thomas Graf <tgraf@redhat.com>
[ Alexandre Rostovtsev <tetromino@gentoo.org>: remove parts of commit
unrelated to if.h changes. ]
diff --git a/include/wireless-helper.h b/include/wireless-helper.h
index d150ef7..2e4509a 100644
--- a/include/wireless-helper.h
+++ b/include/wireless-helper.h
@@ -27,6 +27,6 @@
#include <sys/types.h>
#include <linux/types.h>
#include <sys/socket.h>
-#include <net/if.h>
+#include <linux/if.h>
#include <wireless.h>
diff --git a/src/nm-device.c b/src/nm-device.c
index c0b1b87..559606c 100644
--- a/src/nm-device.c
+++ b/src/nm-device.c
@@ -25,7 +25,6 @@
#include <dbus/dbus.h>
#include <netinet/in.h>
#include <string.h>
-#include <net/if.h>
#include <unistd.h>
#include <errno.h>
#include <sys/ioctl.h>
@@ -34,6 +33,7 @@
#include <sys/wait.h>
#include <arpa/inet.h>
#include <fcntl.h>
+#include <linux/if.h>
#include "nm-glib-compat.h"
#include "nm-device-interface.h"
diff --git a/src/nm-system.c b/src/nm-system.c
index 0b29468..473fcec 100644
--- a/src/nm-system.c
+++ b/src/nm-system.c
@@ -40,7 +40,7 @@
#include <netdb.h>
#include <glib.h>
#include <ctype.h>
-#include <net/if.h>
+#include <linux/if.h>
#include "nm-system.h"
#include "nm-device.h"
diff --git a/src/ppp-manager/nm-ppp-manager.c b/src/ppp-manager/nm-ppp-manager.c
index 522c075..3546f8d 100644
--- a/src/ppp-manager/nm-ppp-manager.c
+++ b/src/ppp-manager/nm-ppp-manager.c
@@ -32,7 +32,7 @@
#include <sys/socket.h>
#include <sys/ioctl.h>
#include <asm/types.h>
-#include <net/if.h>
+#include <linux/if.h>
#include <sys/stat.h>
#include <linux/ppp_defs.h>
diff --git a/src/settings/plugins/ifcfg-rh/reader.c b/src/settings/plugins/ifcfg-rh/reader.c
index 691a176..910cca3 100644
--- a/src/settings/plugins/ifcfg-rh/reader.c
+++ b/src/settings/plugins/ifcfg-rh/reader.c
@@ -28,10 +28,10 @@
#include <ctype.h>
#include <sys/inotify.h>
#include <errno.h>
-#include <net/if.h>
#include <sys/ioctl.h>
#include <unistd.h>
#include <netinet/ether.h>
+#include <linux/if.h>
#ifndef __user
#define __user
diff --git a/src/wimax/iwmxsdk.c b/src/wimax/iwmxsdk.c
index ff6b553..9c3a78b 100644
--- a/src/wimax/iwmxsdk.c
+++ b/src/wimax/iwmxsdk.c
@@ -27,7 +27,8 @@
#include <stdlib.h>
#include <errno.h>
#include <string.h>
-#include <net/if.h>
+#include <sys/socket.h>
+#include <linux/if.h>
#include <glib.h>
diff --git a/src/wimax/nm-device-wimax.c b/src/wimax/nm-device-wimax.c
index b6afc27..6654140 100644
--- a/src/wimax/nm-device-wimax.c
+++ b/src/wimax/nm-device-wimax.c
@@ -23,7 +23,8 @@
#include <unistd.h>
#include <sys/ioctl.h>
#include <net/ethernet.h>
-#include <net/if.h>
+#include <sys/socket.h>
+#include <linux/if.h>
#include <WiMaxAPI.h>
#include <WiMaxAPIEx.h>
1.1 net-misc/networkmanager/files/01-org.freedesktop.NetworkManager.settings.modify.system.pkla
file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/01-org.freedesktop.NetworkManager.settings.modify.system.pkla?rev=1.1&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/net-misc/networkmanager/files/01-org.freedesktop.NetworkManager.settings.modify.system.pkla?rev=1.1&content-type=text/plain
Index: 01-org.freedesktop.NetworkManager.settings.modify.system.pkla
===================================================================
[Let users in plugdev group modify NetworkManager]
Identity=unix-group:plugdev
Action=org.freedesktop.NetworkManager.settings.modify.system
ResultAny=no
ResultInactive=no
ResultActive=yes
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2011-10-29 6:31 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-29 6:30 [gentoo-commits] gentoo-x86 commit in net-misc/networkmanager/files: networkmanager-0.9.1.90-force-libnl1.1.patch networkmanager-0.9.1.90-rfkill.patch networkmanager-0.9.1.90-if.h.patch 01-org.freedesktop.NetworkManager.settings.modify.system.pkla Alexandre Rostovtsev (tetromino)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox