Bug#895445: Drop (unused) Build-Depends on network-manager-dev
Source: cinnamon-control-center
Version: 3.6.5-1.1
Severity: wishlist
Tags: patch
User: pkg-utopia-maintainers@lists.alioth.debian.org
Usertags: libnm
Hi,
cinnamon-control-center declares a Build-Depends on network-manager-dev.
Afaics, the only reason why NetworkManager.pc is required is due to the
following in configure.ac:
NM_VPN_CONFIG_DIR=`$PKG_CONFIG --variable configdir NetworkManager`/VPN
NM_VPN_MODULE_DIR=`$PKG_CONFIG --variable plugindir NetworkManager`
AC_SUBST(NM_VPN_CONFIG_DIR)
AC_SUBST(NM_VPN_MODULE_DIR)
Those defines, NM_VPN_CONFIG_DIR and NM_VPN_MODULE_DIR are never used in
the code though. It thus seems safe to drop the Build-Depends on
network-manager-dev and clean up the use of
NM_VPN_CONFIG_DIR/NM_VPN_MODULE_DIR (patch for that is attached).
Please consider applying the attached patch in your next upload and drop
the Build-Depends on network-manager-dev.
Thanks,
Michael
-- System Information:
Debian Release: buster/sid
APT prefers unstable
APT policy: (500, 'unstable'), (200, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386
Kernel: Linux 4.15.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8), LANGUAGE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
Description: Stop requiring NetworkManager.pc
We don't actually use NM_VPN_CONFIG_DIR and NM_VPN_MODULE_DIR, along with the
corresponding configure check.
Author: Michael Biebl <biebl@debian.org>
--- cinnamon-control-center-3.6.5.orig/configure.ac
+++ cinnamon-control-center-3.6.5/configure.ac
@@ -157,10 +157,6 @@ if test "x$enable_networkmanager" = xyes
if test x${have_networkmanager} = xyes; then
AC_DEFINE(HAVE_NETWORK_MANAGER, 1, [Define to 1 if NetworkManager is available])
- NM_VPN_CONFIG_DIR=`$PKG_CONFIG --variable configdir NetworkManager`/VPN
- NM_VPN_MODULE_DIR=`$PKG_CONFIG --variable plugindir NetworkManager`
- AC_SUBST(NM_VPN_CONFIG_DIR)
- AC_SUBST(NM_VPN_MODULE_DIR)
fi
if test x${have_nma_18} = xyes; then
--- cinnamon-control-center-3.6.5.orig/panels/network/connection-editor/Makefile.am
+++ cinnamon-control-center-3.6.5/panels/network/connection-editor/Makefile.am
@@ -37,9 +37,7 @@ libconnection_editor_la_CPPFLAGS = \
$(PANEL_CFLAGS) \
-I$(srcdir)/../wireless-security \
$(NETWORK_PANEL_CFLAGS) \
- $(NETWORK_MANAGER_CFLAGS) \
- -DNM_VPN_CONFIG_DIR=\""$(NM_VPN_CONFIG_DIR)"\" \
- -DNM_VPN_MODULE_DIR=\""$(NM_VPN_MODULE_DIR)"\"
+ $(NETWORK_MANAGER_CFLAGS)
libconnection_editor_la_LIBADD = \
$(builddir)/../wireless-security/libwireless-security.la \
Reply to: