[Buildroot] [git commit master] libnl: handle all installed libraries

[Buildroot] [git commit master] libnl: handle all installed libraries

[Peter Korsgaard]

commit: http://git.buildroot.net/buildroot/commit/?id=61901bd74a622450139358ea0c0f37d4568ad570
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master

libnl does not only install libnl.so, but also libnl-nf.so,
libnl-genl.so and libnl-route.so. Change the post-install
and uninstall hooks to handle these libraries as well.

Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
 CHANGES                |   10 +++++-----
 package/libnl/libnl.mk |    4 ++++
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/CHANGES b/CHANGES
index 20290dd..b1f4135 100644
--- a/CHANGES
+++ b/CHANGES
@@ -6,11 +6,11 @@
 	Updated/fixed packages: autoconf, busybox, directfb, dnsmasq,
 	e2fsprogs, festival, gstreamer, gst-plugins-bad, gst-plugins-base,
 	gst-plugins-good, imagemagick, iptables, kernel-headers, kismet,
-	libevent, libglib2, liblockfile, libmad, libupnp, libuuid, ltrace,
-	lua, mdadm, ncftp, ncurses, netkittelnet, ntfs-3g, openssl, php,
-	python, quagga, radvd, rsync, samba, shared-mime-info, speech-tools,
-	squashfs, syslinux, u-boot, util-linux, valgrind, vsftpd, wipe,
-	wpa-supplicant, xdata_xcursor-themes, zlib
+	libevent, libglib2, liblockfile, libmad, libnl, libupnp, libuuid,
+	ltrace, lua, mdadm, ncftp, ncurses, netkittelnet, ntfs-3g, openssl,
+	php, python, quagga, radvd, rsync, samba, shared-mime-info,
+	speech-tools, squashfs, syslinux, u-boot, util-linux, valgrind,
+	vsftpd, wipe, wpa-supplicant, xdata_xcursor-themes, zlib
 
 	Removed package: mdnsresponder, mpatrol
 
diff --git a/package/libnl/libnl.mk b/package/libnl/libnl.mk
index 1a66746..04bac2d 100644
--- a/package/libnl/libnl.mk
+++ b/package/libnl/libnl.mk
@@ -13,10 +13,14 @@ LIBNL_INSTALL_TARGET_OPT = DESTDIR=$(TARGET_DIR) install
 $(eval $(call AUTOTARGETS,package,libnl))
 
 $(LIBNL_HOOK_POST_INSTALL): $(LIBNL_TARGET_INSTALL_TARGET)
+ifneq ($(BR2_ENABLE_DEBUG),y)
 	$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libnl.so*
+	$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libnl-*.so*
+endif
 	touch $@
 
 $(LIBNL_TARGET_UNINSTALL):
 	$(call MESSAGE,"Uninstalling")
 	rm -f $(TARGET_DIR)/usr/lib/libnl.so*
+	rm -f $(TARGET_DIR)/usr/lib/libnl-*.so*
 	rm -f $(LIBNL_TARGET_INSTALL_TARGET) $(LIBNL_HOOK_POST_INSTALL)
-- 
1.6.3.3

[Buildroot] [git commit master] libnl: handle all installed libraries

[Gustavo Zacarias]

Peter Korsgaard wrote:

> commit: http://git.buildroot.net/buildroot/commit/?id=61901bd74a622450139358ea0c0f37d4568ad570
> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
> 
> libnl does not only install libnl.so, but also libnl-nf.so,
> libnl-genl.so and libnl-route.so. Change the post-install
> and uninstall hooks to handle these libraries as well.
> 
> Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>

This broke the current version of the libnl package in buildroot since
libnl-* doesn't exist for libnl-1.1 (it's all in libnl.so) thus the
strip fails miserably.
Sven: Using git/libnl2 snapshot?
Regards.

[Buildroot] [git commit master] libnl: handle all installed libraries

[Peter Korsgaard]

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 Gustavo> Peter Korsgaard wrote:
 >> commit: http://git.buildroot.net/buildroot/commit/?id=61901bd74a622450139358ea0c0f37d4568ad570
 >> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
 >> 
 >> libnl does not only install libnl.so, but also libnl-nf.so,
 >> libnl-genl.so and libnl-route.so. Change the post-install
 >> and uninstall hooks to handle these libraries as well.
 >> 
 >> Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>
 >> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>

 Gustavo> This broke the current version of the libnl package in buildroot since
 Gustavo> libnl-* doesn't exist for libnl-1.1 (it's all in libnl.so) thus the

Uhh, that's no good - Sven?

 Gustavo> strip fails miserably.
 Gustavo> Sven: Using git/libnl2 snapshot?

I've added his email in to: so there's a bigger chance he'll see the
mail.

-- 
Bye, Peter Korsgaard

[Buildroot] [git commit master] libnl: handle all installed libraries

[Sven Neumann]

On Mon, 2009-10-26 at 18:54 +0100, Peter Korsgaard wrote:
> >>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:
> 
>  Gustavo> Peter Korsgaard wrote:
>  >> commit: http://git.buildroot.net/buildroot/commit/?id=61901bd74a622450139358ea0c0f37d4568ad570
>  >> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
>  >> 
>  >> libnl does not only install libnl.so, but also libnl-nf.so,
>  >> libnl-genl.so and libnl-route.so. Change the post-install
>  >> and uninstall hooks to handle these libraries as well.
>  >> 
>  >> Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>
>  >> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
> 
>  Gustavo> This broke the current version of the libnl package in buildroot since
>  Gustavo> libnl-* doesn't exist for libnl-1.1 (it's all in libnl.so) thus the
> 
> Uhh, that's no good - Sven?
> 
>  Gustavo> strip fails miserably.
>  Gustavo> Sven: Using git/libnl2 snapshot?

Uh, I am sorry. Looks like we updated libnl to version 2.0 some time ago
and I didn't send this update upstream. If you want I can send you a
patch for this. But since this is not really an upstream release but
just a snapshot, it's probably better to just revert the latest libnl
change. Sorry for the inconvenience.


Sven

[Buildroot] [git commit master] libnl: handle all installed libraries

[Peter Korsgaard]

>>>>> "Sven" == Sven Neumann <s.neumann@raumfeld.com> writes:

 Sven> Uh, I am sorry. Looks like we updated libnl to version 2.0 some
 Sven> time ago and I didn't send this update upstream. If you want I
 Sven> can send you a patch for this. But since this is not really an
 Sven> upstream release but just a snapshot, it's probably better to
 Sven> just revert the latest libnl change. Sorry for the inconvenience.

Ok, thanks - I'll revert. Please try to be more careful in the future.

-- 
Bye, Peter Korsgaard

[Buildroot] [git commit master] libnl: handle all installed libraries

[Gustavo Zacarias]

Sven Neumann wrote:

> Uh, I am sorry. Looks like we updated libnl to version 2.0 some time ago
> and I didn't send this update upstream. If you want I can send you a
> patch for this. But since this is not really an upstream release but
> just a snapshot, it's probably better to just revert the latest libnl
> change. Sorry for the inconvenience.
> 
> 
> Sven

No problem, fixable and didn't go into a release so it's not a biggie.
Do you know if they'll be releasing libnl2 any time soon or we'll keep
using snapshots forever?
Thanks.