Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency
@ 2015-12-02 11:27 yegorslists at googlemail.com
  2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: yegorslists at googlemail.com @ 2015-12-02 11:27 UTC (permalink / raw)
  To: buildroot

From: Yegor Yefremov <yegorslists@googlemail.com>

Since version 3.1.3 eudev doesn't provide internal libgudev,
so this package can now be used as an ordinary stand-alone
package.

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/libgudev/Config.in | 2 --
 1 file changed, 2 deletions(-)

diff --git a/package/libgudev/Config.in b/package/libgudev/Config.in
index 9b64ffc..4596fa1 100644
--- a/package/libgudev/Config.in
+++ b/package/libgudev/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_LIBGUDEV
 	bool "libgudev"
-	depends on BR2_INIT_SYSTEMD
 	depends on BR2_USE_WCHAR # gettext
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_MMU # fork()
@@ -18,5 +17,4 @@ config BR2_PACKAGE_LIBGUDEV
 
 comment "libgudev needs a toolchain w/ wchar, threads"
 	depends on BR2_USE_MMU
-	depends on BR2_INIT_SYSTEMD
 	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 2/2] Use libgudev as regular dependency
  2015-12-02 11:27 [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency yegorslists at googlemail.com
@ 2015-12-02 11:27 ` yegorslists at googlemail.com
  2015-12-07 10:08   ` Yann E. MORIN
                     ` (2 more replies)
  2015-12-07  9:41 ` [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency Yann E. MORIN
                   ` (2 subsequent siblings)
  3 siblings, 3 replies; 8+ messages in thread
From: yegorslists at googlemail.com @ 2015-12-02 11:27 UTC (permalink / raw)
  To: buildroot

From: Yegor Yefremov <yegorslists@googlemail.com>

Since eudev 3.1.3 libgudev is not shipped with this package,
so libgudev can be used as regular dependency regardless of
udev implementation selected.

Fixes following build error:
  http://autobuild.buildroot.net/results/320/320c052bda0f1b5afb1e5c83a7fb4dca6227c5a1//
  http://autobuild.buildroot.net/results/3b0/3b096f882030325b9290ad8860bd9cd373a11dc2//

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/hidapi/Config.in                   | 7 +------
 package/hidapi/hidapi.mk                   | 6 +-----
 package/libmbim/Config.in                  | 2 +-
 package/libmbim/libmbim.mk                 | 6 +-----
 package/modem-manager/Config.in            | 2 +-
 package/modem-manager/modem-manager.mk     | 6 +-----
 package/network-manager/Config.in          | 2 +-
 package/network-manager/network-manager.mk | 6 +-----
 package/udisks/Config.in                   | 7 +------
 package/udisks/udisks.mk                   | 7 ++-----
 10 files changed, 11 insertions(+), 40 deletions(-)

diff --git a/package/hidapi/Config.in b/package/hidapi/Config.in
index 5d025bf..c48a096 100644
--- a/package/hidapi/Config.in
+++ b/package/hidapi/Config.in
@@ -3,12 +3,7 @@ config BR2_PACKAGE_HIDAPI
 	depends on BR2_PACKAGE_HAS_UDEV
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libusb
 	select BR2_PACKAGE_LIBUSB
-	# When eudev is used as the udev provider, libgudev is
-	# automatically provided as it is part of eudev. However, when
-	# systemd is used as the udev provider, libgudev is not
-	# provided, and needs to be built separately. This is why we
-	# select the libgudev package only if systemd is used.
-	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
+	select BR2_PACKAGE_LIBGUDEV
 	help
 	  HIDAPI is a multi-platform library which allows an application
 	  to interface with USB and Bluetooth HID-Class devices on
diff --git a/package/hidapi/hidapi.mk b/package/hidapi/hidapi.mk
index eb5d5c3..b16e4ae 100644
--- a/package/hidapi/hidapi.mk
+++ b/package/hidapi/hidapi.mk
@@ -14,10 +14,6 @@ HIDAPI_AUTORECONF = YES
 HIDAPI_LICENSE = GPLv3 or BSD-3c or HIDAPI license
 HIDAPI_LICENSE_FILES = LICENSE.txt LICENSE-gpl3.txt LICENSE-bsd.txt LICENSE-orig.txt
 
-HIDAPI_DEPENDENCIES = libusb
-
-ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
-HIDAPI_DEPENDENCIES += libgudev
-endif
+HIDAPI_DEPENDENCIES = libusb libgudev
 
 $(eval $(autotools-package))
diff --git a/package/libmbim/Config.in b/package/libmbim/Config.in
index e79d51f..0ef28db 100644
--- a/package/libmbim/Config.in
+++ b/package/libmbim/Config.in
@@ -3,7 +3,7 @@ config BR2_PACKAGE_LIBMBIM
 	depends on BR2_USE_WCHAR # libglib2
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
 	depends on BR2_PACKAGE_HAS_UDEV
-	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
+	select BR2_PACKAGE_LIBGUDEV
 	select BR2_PACKAGE_LIBGLIB2
 	help
 	  libmbim is a glib-based library for talking to WWAN modems and
diff --git a/package/libmbim/libmbim.mk b/package/libmbim/libmbim.mk
index cb6c905..3cb9235 100644
--- a/package/libmbim/libmbim.mk
+++ b/package/libmbim/libmbim.mk
@@ -11,11 +11,7 @@ LIBMBIM_LICENSE = LGPLv2+ (library), GPLv2+ (programs)
 LIBMBIM_LICENSE_FILES = COPYING
 LIBMBIM_INSTALL_STAGING = YES
 
-LIBMBIM_DEPENDENCIES = libglib2 udev
-
-ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
-LIBMBIM_DEPENDENCIES += libgudev
-endif
+LIBMBIM_DEPENDENCIES = libglib2 udev libgudev
 
 # we don't want -Werror
 LIBMBIM_CONF_OPTS = --enable-more-warnings=no
diff --git a/package/modem-manager/Config.in b/package/modem-manager/Config.in
index 1cd502b..aa7ed1e 100644
--- a/package/modem-manager/Config.in
+++ b/package/modem-manager/Config.in
@@ -6,7 +6,7 @@ config BR2_PACKAGE_MODEM_MANAGER
 	depends on BR2_USE_MMU # dbus
 	select BR2_PACKAGE_DBUS
 	select BR2_PACKAGE_DBUS_GLIB
-	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
+	select BR2_PACKAGE_LIBGUDEV
 	help
 	  ModemManager is a DBus-activated daemon which controls mobile
 	  broadband (2G/3G/4G) devices and connections.
diff --git a/package/modem-manager/modem-manager.mk b/package/modem-manager/modem-manager.mk
index 6d8dc10..451752c 100644
--- a/package/modem-manager/modem-manager.mk
+++ b/package/modem-manager/modem-manager.mk
@@ -9,13 +9,9 @@ MODEM_MANAGER_SOURCE = ModemManager-$(MODEM_MANAGER_VERSION).tar.xz
 MODEM_MANAGER_SITE = http://www.freedesktop.org/software/ModemManager
 MODEM_MANAGER_LICENSE = GPLv2+ (programs, plugins), LGPLv2+ (libmm-glib)
 MODEM_MANAGER_LICENSE_FILES = COPYING
-MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool
+MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool libgudev
 MODEM_MANAGER_INSTALL_STAGING = YES
 
-ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
-MODEM_MANAGER_DEPENDENCIES += libgudev
-endif
-
 ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
 MODEM_MANAGER_DEPENDENCIES += libqmi
 MODEM_MANAGER_CONF_OPTS += --with-qmi
diff --git a/package/network-manager/Config.in b/package/network-manager/Config.in
index 5a0e636..8b4607c 100644
--- a/package/network-manager/Config.in
+++ b/package/network-manager/Config.in
@@ -11,7 +11,7 @@ config BR2_PACKAGE_NETWORK_MANAGER
 	select BR2_PACKAGE_GNUTLS
 	select BR2_PACKAGE_LIBGCRYPT
 	select BR2_PACKAGE_LIBNL
-	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
+	select BR2_PACKAGE_LIBGUDEV
 	select BR2_PACKAGE_UTIL_LINUX
 	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
 	select BR2_PACKAGE_WIRELESS_TOOLS
diff --git a/package/network-manager/network-manager.mk b/package/network-manager/network-manager.mk
index f2fe147..98879d6 100644
--- a/package/network-manager/network-manager.mk
+++ b/package/network-manager/network-manager.mk
@@ -10,7 +10,7 @@ NETWORK_MANAGER_SOURCE = NetworkManager-$(NETWORK_MANAGER_VERSION).tar.xz
 NETWORK_MANAGER_SITE = http://ftp.gnome.org/pub/GNOME/sources/NetworkManager/$(NETWORK_MANAGER_VERSION_MAJOR)
 NETWORK_MANAGER_INSTALL_STAGING = YES
 NETWORK_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib libnl gnutls \
-	libgcrypt wireless_tools util-linux host-intltool readline libndp
+	libgcrypt wireless_tools util-linux host-intltool readline libndp libgudev
 NETWORK_MANAGER_LICENSE = GPLv2+ (app), LGPLv2+ (libnm-util)
 NETWORK_MANAGER_LICENSE_FILES = COPYING libnm-util/COPYING
 
@@ -63,10 +63,6 @@ ifeq ($(BR2_PACKAGE_DHCPCD),y)
 NETWORK_MANAGER_CONF_OPTS += --with-dhcpcd=/sbin/dhcpcd
 endif
 
-ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
-NETWORK_MANAGER_DEPENDENCIES += libgudev
-endif
-
 # uClibc by default doesn't have backtrace support, so don't use it
 ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
 NETWORK_MANAGER_CONF_OPTS += --disable-crashtrace
diff --git a/package/udisks/Config.in b/package/udisks/Config.in
index c9664af..8b8fd28 100644
--- a/package/udisks/Config.in
+++ b/package/udisks/Config.in
@@ -12,12 +12,7 @@ config BR2_PACKAGE_UDISKS
 	select BR2_PACKAGE_PARTED
 	select BR2_PACKAGE_LVM2
 	select BR2_PACKAGE_LIBATASMART
-	# When eudev is used as the udev provider, libgudev is automatically
-	# provided as it is part of eudev. However, when systemd is used as
-	# the udev provider, libgudev is not provided, and needs to be built
-	# separately. This is why we select the libgudev package only if
-	# systemd is used.
-	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
+	select BR2_PACKAGE_LIBGUDEV
 	help
 	  The udisks project provides
 
diff --git a/package/udisks/udisks.mk b/package/udisks/udisks.mk
index eccfbd2..eea79e5 100644
--- a/package/udisks/udisks.mk
+++ b/package/udisks/udisks.mk
@@ -18,14 +18,11 @@ UDISKS_DEPENDENCIES = 	\
 	polkit		\
 	parted		\
 	lvm2		\
-	libatasmart
+	libatasmart	\
+	libgudev
 
 UDISKS_CONF_OPTS = --disable-remote-access --disable-man-pages
 
-ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
-UDISKS_DEPENDENCIES += libgudev
-endif
-
 ifeq ($(BR2_PACKAGE_UDISKS_LVM2),y)
 UDISKS_CONF_OPTS += --enable-lvm2
 endif
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency
  2015-12-02 11:27 [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency yegorslists at googlemail.com
  2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
@ 2015-12-07  9:41 ` Yann E. MORIN
  2015-12-07 14:55 ` Vicente Olivert Riera
  2015-12-08 20:47 ` Peter Korsgaard
  3 siblings, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2015-12-07  9:41 UTC (permalink / raw)
  To: buildroot

Yegor,All,

On 2015-12-02 12:27 +0100, yegorslists at googlemail.com spake thusly:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> Since version 3.1.3 eudev doesn't provide internal libgudev,
> so this package can now be used as an ordinary stand-alone
> package.
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/libgudev/Config.in | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/package/libgudev/Config.in b/package/libgudev/Config.in
> index 9b64ffc..4596fa1 100644
> --- a/package/libgudev/Config.in
> +++ b/package/libgudev/Config.in
> @@ -1,6 +1,5 @@
>  config BR2_PACKAGE_LIBGUDEV
>  	bool "libgudev"
> -	depends on BR2_INIT_SYSTEMD
>  	depends on BR2_USE_WCHAR # gettext
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
>  	depends on BR2_USE_MMU # fork()
> @@ -18,5 +17,4 @@ config BR2_PACKAGE_LIBGUDEV
>  
>  comment "libgudev needs a toolchain w/ wchar, threads"
>  	depends on BR2_USE_MMU
> -	depends on BR2_INIT_SYSTEMD
>  	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
> -- 
> 2.1.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 2/2] Use libgudev as regular dependency
  2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
@ 2015-12-07 10:08   ` Yann E. MORIN
  2015-12-07 14:57   ` Vicente Olivert Riera
  2015-12-08 20:50   ` Peter Korsgaard
  2 siblings, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2015-12-07 10:08 UTC (permalink / raw)
  To: buildroot

Yegor, All,

On 2015-12-02 12:27 +0100, yegorslists at googlemail.com spake thusly:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> Since eudev 3.1.3 libgudev is not shipped with this package,
> so libgudev can be used as regular dependency regardless of
> udev implementation selected.
> 
> Fixes following build error:
>   http://autobuild.buildroot.net/results/320/320c052bda0f1b5afb1e5c83a7fb4dca6227c5a1//
>   http://autobuild.buildroot.net/results/3b0/3b096f882030325b9290ad8860bd9cd373a11dc2//
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
[yann.morin.1998 at free.fr: build-tested with a Network Manager
 autobuild failure]
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
>  package/hidapi/Config.in                   | 7 +------
>  package/hidapi/hidapi.mk                   | 6 +-----
>  package/libmbim/Config.in                  | 2 +-
>  package/libmbim/libmbim.mk                 | 6 +-----
>  package/modem-manager/Config.in            | 2 +-
>  package/modem-manager/modem-manager.mk     | 6 +-----
>  package/network-manager/Config.in          | 2 +-
>  package/network-manager/network-manager.mk | 6 +-----
>  package/udisks/Config.in                   | 7 +------
>  package/udisks/udisks.mk                   | 7 ++-----
>  10 files changed, 11 insertions(+), 40 deletions(-)
> 
> diff --git a/package/hidapi/Config.in b/package/hidapi/Config.in
> index 5d025bf..c48a096 100644
> --- a/package/hidapi/Config.in
> +++ b/package/hidapi/Config.in
> @@ -3,12 +3,7 @@ config BR2_PACKAGE_HIDAPI
>  	depends on BR2_PACKAGE_HAS_UDEV
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # libusb
>  	select BR2_PACKAGE_LIBUSB
> -	# When eudev is used as the udev provider, libgudev is
> -	# automatically provided as it is part of eudev. However, when
> -	# systemd is used as the udev provider, libgudev is not
> -	# provided, and needs to be built separately. This is why we
> -	# select the libgudev package only if systemd is used.
> -	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
> +	select BR2_PACKAGE_LIBGUDEV
>  	help
>  	  HIDAPI is a multi-platform library which allows an application
>  	  to interface with USB and Bluetooth HID-Class devices on
> diff --git a/package/hidapi/hidapi.mk b/package/hidapi/hidapi.mk
> index eb5d5c3..b16e4ae 100644
> --- a/package/hidapi/hidapi.mk
> +++ b/package/hidapi/hidapi.mk
> @@ -14,10 +14,6 @@ HIDAPI_AUTORECONF = YES
>  HIDAPI_LICENSE = GPLv3 or BSD-3c or HIDAPI license
>  HIDAPI_LICENSE_FILES = LICENSE.txt LICENSE-gpl3.txt LICENSE-bsd.txt LICENSE-orig.txt
>  
> -HIDAPI_DEPENDENCIES = libusb
> -
> -ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> -HIDAPI_DEPENDENCIES += libgudev
> -endif
> +HIDAPI_DEPENDENCIES = libusb libgudev
>  
>  $(eval $(autotools-package))
> diff --git a/package/libmbim/Config.in b/package/libmbim/Config.in
> index e79d51f..0ef28db 100644
> --- a/package/libmbim/Config.in
> +++ b/package/libmbim/Config.in
> @@ -3,7 +3,7 @@ config BR2_PACKAGE_LIBMBIM
>  	depends on BR2_USE_WCHAR # libglib2
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
>  	depends on BR2_PACKAGE_HAS_UDEV
> -	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
> +	select BR2_PACKAGE_LIBGUDEV
>  	select BR2_PACKAGE_LIBGLIB2
>  	help
>  	  libmbim is a glib-based library for talking to WWAN modems and
> diff --git a/package/libmbim/libmbim.mk b/package/libmbim/libmbim.mk
> index cb6c905..3cb9235 100644
> --- a/package/libmbim/libmbim.mk
> +++ b/package/libmbim/libmbim.mk
> @@ -11,11 +11,7 @@ LIBMBIM_LICENSE = LGPLv2+ (library), GPLv2+ (programs)
>  LIBMBIM_LICENSE_FILES = COPYING
>  LIBMBIM_INSTALL_STAGING = YES
>  
> -LIBMBIM_DEPENDENCIES = libglib2 udev
> -
> -ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> -LIBMBIM_DEPENDENCIES += libgudev
> -endif
> +LIBMBIM_DEPENDENCIES = libglib2 udev libgudev
>  
>  # we don't want -Werror
>  LIBMBIM_CONF_OPTS = --enable-more-warnings=no
> diff --git a/package/modem-manager/Config.in b/package/modem-manager/Config.in
> index 1cd502b..aa7ed1e 100644
> --- a/package/modem-manager/Config.in
> +++ b/package/modem-manager/Config.in
> @@ -6,7 +6,7 @@ config BR2_PACKAGE_MODEM_MANAGER
>  	depends on BR2_USE_MMU # dbus
>  	select BR2_PACKAGE_DBUS
>  	select BR2_PACKAGE_DBUS_GLIB
> -	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
> +	select BR2_PACKAGE_LIBGUDEV
>  	help
>  	  ModemManager is a DBus-activated daemon which controls mobile
>  	  broadband (2G/3G/4G) devices and connections.
> diff --git a/package/modem-manager/modem-manager.mk b/package/modem-manager/modem-manager.mk
> index 6d8dc10..451752c 100644
> --- a/package/modem-manager/modem-manager.mk
> +++ b/package/modem-manager/modem-manager.mk
> @@ -9,13 +9,9 @@ MODEM_MANAGER_SOURCE = ModemManager-$(MODEM_MANAGER_VERSION).tar.xz
>  MODEM_MANAGER_SITE = http://www.freedesktop.org/software/ModemManager
>  MODEM_MANAGER_LICENSE = GPLv2+ (programs, plugins), LGPLv2+ (libmm-glib)
>  MODEM_MANAGER_LICENSE_FILES = COPYING
> -MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool
> +MODEM_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib host-intltool libgudev
>  MODEM_MANAGER_INSTALL_STAGING = YES
>  
> -ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> -MODEM_MANAGER_DEPENDENCIES += libgudev
> -endif
> -
>  ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
>  MODEM_MANAGER_DEPENDENCIES += libqmi
>  MODEM_MANAGER_CONF_OPTS += --with-qmi
> diff --git a/package/network-manager/Config.in b/package/network-manager/Config.in
> index 5a0e636..8b4607c 100644
> --- a/package/network-manager/Config.in
> +++ b/package/network-manager/Config.in
> @@ -11,7 +11,7 @@ config BR2_PACKAGE_NETWORK_MANAGER
>  	select BR2_PACKAGE_GNUTLS
>  	select BR2_PACKAGE_LIBGCRYPT
>  	select BR2_PACKAGE_LIBNL
> -	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
> +	select BR2_PACKAGE_LIBGUDEV
>  	select BR2_PACKAGE_UTIL_LINUX
>  	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
>  	select BR2_PACKAGE_WIRELESS_TOOLS
> diff --git a/package/network-manager/network-manager.mk b/package/network-manager/network-manager.mk
> index f2fe147..98879d6 100644
> --- a/package/network-manager/network-manager.mk
> +++ b/package/network-manager/network-manager.mk
> @@ -10,7 +10,7 @@ NETWORK_MANAGER_SOURCE = NetworkManager-$(NETWORK_MANAGER_VERSION).tar.xz
>  NETWORK_MANAGER_SITE = http://ftp.gnome.org/pub/GNOME/sources/NetworkManager/$(NETWORK_MANAGER_VERSION_MAJOR)
>  NETWORK_MANAGER_INSTALL_STAGING = YES
>  NETWORK_MANAGER_DEPENDENCIES = host-pkgconf udev dbus-glib libnl gnutls \
> -	libgcrypt wireless_tools util-linux host-intltool readline libndp
> +	libgcrypt wireless_tools util-linux host-intltool readline libndp libgudev
>  NETWORK_MANAGER_LICENSE = GPLv2+ (app), LGPLv2+ (libnm-util)
>  NETWORK_MANAGER_LICENSE_FILES = COPYING libnm-util/COPYING
>  
> @@ -63,10 +63,6 @@ ifeq ($(BR2_PACKAGE_DHCPCD),y)
>  NETWORK_MANAGER_CONF_OPTS += --with-dhcpcd=/sbin/dhcpcd
>  endif
>  
> -ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> -NETWORK_MANAGER_DEPENDENCIES += libgudev
> -endif
> -
>  # uClibc by default doesn't have backtrace support, so don't use it
>  ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
>  NETWORK_MANAGER_CONF_OPTS += --disable-crashtrace
> diff --git a/package/udisks/Config.in b/package/udisks/Config.in
> index c9664af..8b8fd28 100644
> --- a/package/udisks/Config.in
> +++ b/package/udisks/Config.in
> @@ -12,12 +12,7 @@ config BR2_PACKAGE_UDISKS
>  	select BR2_PACKAGE_PARTED
>  	select BR2_PACKAGE_LVM2
>  	select BR2_PACKAGE_LIBATASMART
> -	# When eudev is used as the udev provider, libgudev is automatically
> -	# provided as it is part of eudev. However, when systemd is used as
> -	# the udev provider, libgudev is not provided, and needs to be built
> -	# separately. This is why we select the libgudev package only if
> -	# systemd is used.
> -	select BR2_PACKAGE_LIBGUDEV if BR2_INIT_SYSTEMD
> +	select BR2_PACKAGE_LIBGUDEV
>  	help
>  	  The udisks project provides
>  
> diff --git a/package/udisks/udisks.mk b/package/udisks/udisks.mk
> index eccfbd2..eea79e5 100644
> --- a/package/udisks/udisks.mk
> +++ b/package/udisks/udisks.mk
> @@ -18,14 +18,11 @@ UDISKS_DEPENDENCIES = 	\
>  	polkit		\
>  	parted		\
>  	lvm2		\
> -	libatasmart
> +	libatasmart	\
> +	libgudev
>  
>  UDISKS_CONF_OPTS = --disable-remote-access --disable-man-pages
>  
> -ifeq ($(BR2_PACKAGE_LIBGUDEV),y)
> -UDISKS_DEPENDENCIES += libgudev
> -endif
> -
>  ifeq ($(BR2_PACKAGE_UDISKS_LVM2),y)
>  UDISKS_CONF_OPTS += --enable-lvm2
>  endif
> -- 
> 2.1.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency
  2015-12-02 11:27 [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency yegorslists at googlemail.com
  2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
  2015-12-07  9:41 ` [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency Yann E. MORIN
@ 2015-12-07 14:55 ` Vicente Olivert Riera
  2015-12-08 20:47 ` Peter Korsgaard
  3 siblings, 0 replies; 8+ messages in thread
From: Vicente Olivert Riera @ 2015-12-07 14:55 UTC (permalink / raw)
  To: buildroot

Dear Yegor Yefremov,

On 02/12/15 11:27, yegorslists at googlemail.com wrote:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> Since version 3.1.3 eudev doesn't provide internal libgudev,
> so this package can now be used as an ordinary stand-alone
> package.
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
Reviewed-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>

Regards,

Vincent.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 2/2] Use libgudev as regular dependency
  2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
  2015-12-07 10:08   ` Yann E. MORIN
@ 2015-12-07 14:57   ` Vicente Olivert Riera
  2015-12-08 20:50   ` Peter Korsgaard
  2 siblings, 0 replies; 8+ messages in thread
From: Vicente Olivert Riera @ 2015-12-07 14:57 UTC (permalink / raw)
  To: buildroot

Dear Yegor Yefremov,

On 02/12/15 11:27, yegorslists at googlemail.com wrote:
> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
> Since eudev 3.1.3 libgudev is not shipped with this package,
> so libgudev can be used as regular dependency regardless of
> udev implementation selected.
> 
> Fixes following build error:
>   http://autobuild.buildroot.net/results/320/320c052bda0f1b5afb1e5c83a7fb4dca6227c5a1//
>   http://autobuild.buildroot.net/results/3b0/3b096f882030325b9290ad8860bd9cd373a11dc2//
> 
> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
Reviewed-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Tested-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>

Tested with this recent modem-manager autobuild failure:

http://autobuild.buildroot.net/results/373/3735baba46dee5bd846cc830e1e198ed783452c7/

Regards,

Vincent.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency
  2015-12-02 11:27 [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency yegorslists at googlemail.com
                   ` (2 preceding siblings ...)
  2015-12-07 14:55 ` Vicente Olivert Riera
@ 2015-12-08 20:47 ` Peter Korsgaard
  3 siblings, 0 replies; 8+ messages in thread
From: Peter Korsgaard @ 2015-12-08 20:47 UTC (permalink / raw)
  To: buildroot

>>>>> "yegorslists" == yegorslists  <yegorslists@googlemail.com> writes:

 > From: Yegor Yefremov <yegorslists@googlemail.com>
 > Since version 3.1.3 eudev doesn't provide internal libgudev,
 > so this package can now be used as an ordinary stand-alone
 > package.

 > Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
 > ---
 >  package/libgudev/Config.in | 2 --
 >  1 file changed, 2 deletions(-)

 > diff --git a/package/libgudev/Config.in b/package/libgudev/Config.in
 > index 9b64ffc..4596fa1 100644
 > --- a/package/libgudev/Config.in
 > +++ b/package/libgudev/Config.in
 > @@ -1,6 +1,5 @@
 >  config BR2_PACKAGE_LIBGUDEV
 >  	bool "libgudev"
 > -	depends on BR2_INIT_SYSTEMD

But is still needs udev, so it needs to depend on BR2_PACKAGE_HAS_UDEV,
and the toolchain comment needs to change.

Committed with that fixed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Buildroot] [PATCH 2/2] Use libgudev as regular dependency
  2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
  2015-12-07 10:08   ` Yann E. MORIN
  2015-12-07 14:57   ` Vicente Olivert Riera
@ 2015-12-08 20:50   ` Peter Korsgaard
  2 siblings, 0 replies; 8+ messages in thread
From: Peter Korsgaard @ 2015-12-08 20:50 UTC (permalink / raw)
  To: buildroot

>>>>> "yegorslists" == yegorslists  <yegorslists@googlemail.com> writes:

 > From: Yegor Yefremov <yegorslists@googlemail.com>
 > Since eudev 3.1.3 libgudev is not shipped with this package,
 > so libgudev can be used as regular dependency regardless of
 > udev implementation selected.

 > Fixes following build error:
 >   http://autobuild.buildroot.net/results/320/320c052bda0f1b5afb1e5c83a7fb4dca6227c5a1//
 >   http://autobuild.buildroot.net/results/3b0/3b096f882030325b9290ad8860bd9cd373a11dc2//

 > Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2015-12-08 20:50 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-02 11:27 [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency yegorslists at googlemail.com
2015-12-02 11:27 ` [Buildroot] [PATCH 2/2] Use libgudev as regular dependency yegorslists at googlemail.com
2015-12-07 10:08   ` Yann E. MORIN
2015-12-07 14:57   ` Vicente Olivert Riera
2015-12-08 20:50   ` Peter Korsgaard
2015-12-07  9:41 ` [Buildroot] [PATCH 1/2] libgudev: remove systemd dependency Yann E. MORIN
2015-12-07 14:55 ` Vicente Olivert Riera
2015-12-08 20:47 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox