diff mbox

[V3,1/4] ltrace: Removes the libelf dependency

Message ID 1409228494-19492-2-git-send-email-gregory.clement@free-electrons.com
State Accepted
Headers show

Commit Message

Gregory CLEMENT Aug. 28, 2014, 12:21 p.m. UTC
The libelf is currently provided by 2 packages libelf and
elfutils. The first package provides an old version of the libelf
which is no more compatible with a recent version of ltrace. This
patch removes the dependency on the libelf package and only keep the
elfuils package which provides the accurate version of libelf for
ltrace.

It will also allow to remove the libelf package and to avoid conflicts
with two packages providing the same library.

Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
 package/ltrace/Config.in | 1 -
 package/ltrace/ltrace.mk | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

Comments

Yann E. MORIN Sept. 10, 2014, 8:14 p.m. UTC | #1
Gregory, All,

On 2014-08-28 14:21 +0200, Gregory CLEMENT spake thusly:
> The libelf is currently provided by 2 packages libelf and
> elfutils. The first package provides an old version of the libelf
> which is no more compatible with a recent version of ltrace. This
> patch removes the dependency on the libelf package and only keep the
> elfuils package which provides the accurate version of libelf for
> ltrace.
> 
> It will also allow to remove the libelf package and to avoid conflicts
> with two packages providing the same library.
> 
> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>

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

Regards,
Yann E. MORIN.

> ---
>  package/ltrace/Config.in | 1 -
>  package/ltrace/ltrace.mk | 2 +-
>  2 files changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/package/ltrace/Config.in b/package/ltrace/Config.in
> index 8cabfa2..dc759d8 100644
> --- a/package/ltrace/Config.in
> +++ b/package/ltrace/Config.in
> @@ -6,7 +6,6 @@ config BR2_PACKAGE_LTRACE
>  	# https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=756764.
>  	depends on (BR2_i386 || BR2_arm \
>  		|| BR2_powerpc || BR2_sparc || BR2_x86_64)
> -	select BR2_PACKAGE_LIBELF
>  	select BR2_PACKAGE_ELFUTILS
>  	depends on BR2_LARGEFILE # elfutils
>  	depends on BR2_USE_WCHAR # elfutils
> diff --git a/package/ltrace/ltrace.mk b/package/ltrace/ltrace.mk
> index 0a3a90c..48bf7bd 100644
> --- a/package/ltrace/ltrace.mk
> +++ b/package/ltrace/ltrace.mk
> @@ -6,7 +6,7 @@
>  
>  LTRACE_VERSION = 0896ce554f80afdcba81d9754f6104f863dea803
>  LTRACE_SITE = git://anonscm.debian.org/collab-maint/ltrace.git
> -LTRACE_DEPENDENCIES = libelf elfutils
> +LTRACE_DEPENDENCIES = elfutils
>  LTRACE_CONF_OPT = --disable-werror
>  LTRACE_LICENSE = GPLv2
>  LTRACE_LICENSE_FILES = COPYING
> -- 
> 1.9.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Peter Korsgaard Sept. 13, 2014, 9:58 p.m. UTC | #2
>>>>> "Gregory" == Gregory CLEMENT <gregory.clement@free-electrons.com> writes:

 > The libelf is currently provided by 2 packages libelf and
 > elfutils. The first package provides an old version of the libelf
 > which is no more compatible with a recent version of ltrace. This
 > patch removes the dependency on the libelf package and only keep the
 > elfuils package which provides the accurate version of libelf for
 > ltrace.

 > It will also allow to remove the libelf package and to avoid conflicts
 > with two packages providing the same library.

 > Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>

Committed, thanks.
diff mbox

Patch

diff --git a/package/ltrace/Config.in b/package/ltrace/Config.in
index 8cabfa2..dc759d8 100644
--- a/package/ltrace/Config.in
+++ b/package/ltrace/Config.in
@@ -6,7 +6,6 @@  config BR2_PACKAGE_LTRACE
 	# https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=756764.
 	depends on (BR2_i386 || BR2_arm \
 		|| BR2_powerpc || BR2_sparc || BR2_x86_64)
-	select BR2_PACKAGE_LIBELF
 	select BR2_PACKAGE_ELFUTILS
 	depends on BR2_LARGEFILE # elfutils
 	depends on BR2_USE_WCHAR # elfutils
diff --git a/package/ltrace/ltrace.mk b/package/ltrace/ltrace.mk
index 0a3a90c..48bf7bd 100644
--- a/package/ltrace/ltrace.mk
+++ b/package/ltrace/ltrace.mk
@@ -6,7 +6,7 @@ 
 
 LTRACE_VERSION = 0896ce554f80afdcba81d9754f6104f863dea803
 LTRACE_SITE = git://anonscm.debian.org/collab-maint/ltrace.git
-LTRACE_DEPENDENCIES = libelf elfutils
+LTRACE_DEPENDENCIES = elfutils
 LTRACE_CONF_OPT = --disable-werror
 LTRACE_LICENSE = GPLv2
 LTRACE_LICENSE_FILES = COPYING