Patchwork [1/2] coreutils: Switch to disabling gmp support

login
register
mail settings
Submitter Tom Rini
Date March 22, 2011, 6:21 p.m.
Message ID <3c41c3e7dc35b364f0c16d3694397adea2aa184a.1300817337.git.tom_rini@mentor.com>
Download mbox | patch
Permalink /patch/1695/
State New, archived
Headers show

Comments

Tom Rini - March 22, 2011, 6:21 p.m.
Coreutils can optionally build with libgmp support and for our uses
it's fine to just not build it rather than add the dependency.

Signed-off-by: Tom Rini <tom_rini@mentor.com>
---
 meta/recipes-core/coreutils/coreutils_8.9.bb |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)
Khem Raj - March 22, 2011, 6:32 p.m.
On (22/03/11 11:21), Tom Rini wrote:
> Coreutils can optionally build with libgmp support and for our uses
> it's fine to just not build it rather than add the dependency.
> 
To be clear What we lose here is entry for coreutils release notes

* Noteworthy changes in release 7.0 (2008-10-05) [beta] 

   If the GNU MP library is available at configure time, factor and 
   expr support arbitrarily large numbers.  Pollard's rho algorithm is 
   used to factor large numbers. 

and I think we never intended to use it. Its just that now it finds the
library staged due to gcc requiring it.

Acked-by: Khem Raj <raj.khem@gmail.com>

> Signed-off-by: Tom Rini <tom_rini@mentor.com>
> ---
>  meta/recipes-core/coreutils/coreutils_8.9.bb |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-core/coreutils/coreutils_8.9.bb b/meta/recipes-core/coreutils/coreutils_8.9.bb
> index 4b9a655..d75c080 100644
> --- a/meta/recipes-core/coreutils/coreutils_8.9.bb
> +++ b/meta/recipes-core/coreutils/coreutils_8.9.bb
> @@ -7,9 +7,9 @@ BUGTRACKER = "http://debbugs.gnu.org/coreutils"
>  LICENSE = "GPLv3+"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504\
>                      file://src/ls.c;startline=5;endline=16;md5=e1a509558876db58fb6667ba140137ad"
> -PR = "r0"
> +PR = "r1"
>  DEPENDS = "perl-native gmp"
> -DEPENDS_virtclass-native = "perl-native gmp-native"
> +DEPENDS_virtclass-native = "perl-native"
>  
>  inherit autotools gettext
>  
> @@ -18,6 +18,8 @@ SRC_URI = "http://ftp.gnu.org/gnu/coreutils/${BP}.tar.gz"
>  SRC_URI[md5sum] = "36909ae68840d73a800120cf74af794a"
>  SRC_URI[sha256sum] = "aa991fa4296b22ff929a31a5cb5528bb783c84cdef4503c4ff311cfbeaebf50a"
>  
> +EXTRA_OECONF_virtclass-native = "--without-gmp"
> +
>  # [ gets a special treatment and is not included in this
>  bindir_progs = "base64 basename chcon cksum comm csplit cut dir dircolors dirname du \
>                  env expand expr factor fmt fold groups head hostid id install \
> -- 
> 1.7.0.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Patch

diff --git a/meta/recipes-core/coreutils/coreutils_8.9.bb b/meta/recipes-core/coreutils/coreutils_8.9.bb
index 4b9a655..d75c080 100644
--- a/meta/recipes-core/coreutils/coreutils_8.9.bb
+++ b/meta/recipes-core/coreutils/coreutils_8.9.bb
@@ -7,9 +7,9 @@  BUGTRACKER = "http://debbugs.gnu.org/coreutils"
 LICENSE = "GPLv3+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504\
                     file://src/ls.c;startline=5;endline=16;md5=e1a509558876db58fb6667ba140137ad"
-PR = "r0"
+PR = "r1"
 DEPENDS = "perl-native gmp"
-DEPENDS_virtclass-native = "perl-native gmp-native"
+DEPENDS_virtclass-native = "perl-native"
 
 inherit autotools gettext
 
@@ -18,6 +18,8 @@  SRC_URI = "http://ftp.gnu.org/gnu/coreutils/${BP}.tar.gz"
 SRC_URI[md5sum] = "36909ae68840d73a800120cf74af794a"
 SRC_URI[sha256sum] = "aa991fa4296b22ff929a31a5cb5528bb783c84cdef4503c4ff311cfbeaebf50a"
 
+EXTRA_OECONF_virtclass-native = "--without-gmp"
+
 # [ gets a special treatment and is not included in this
 bindir_progs = "base64 basename chcon cksum comm csplit cut dir dircolors dirname du \
                 env expand expr factor fmt fold groups head hostid id install \