[meta-oe][meta-networking][PATCH] cmake-native: add dependency on openssl-native libidn-native

[meta-oe][meta-networking][PATCH] cmake-native: add dependency on openssl-native libidn-native

[Martin Jansa]

From: Martin Jansa <martin.jansa@lge.com>

Signed-off-by: Martin Jansa <martin.jansa@lge.com>
---
 meta/recipes-devtools/cmake/cmake-native_3.6.1.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb b/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb
index 33930fb..c4d22d8 100644
--- a/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb
+++ b/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb
@@ -2,7 +2,7 @@ require cmake.inc
 inherit native
 
 # Using cmake's internal libarchive, so some dependencies are needed
-DEPENDS += "bzip2-native zlib-native"
+DEPENDS += "bzip2-native zlib-native openssl-native libidn-native"
 
 SRC_URI += "\
     file://cmlibarchive-disable-ext2fs.patch \
-- 
2.10.0

Re: [meta-oe][meta-networking][PATCH] cmake-native: add dependency on openssl-native libidn-native

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1088 bytes --]

On Mon, Sep 12, 2016 at 07:05:46PM +0200, Martin Jansa wrote:
> From: Martin Jansa <martin.jansa@lge.com>

Sent by accident (as subject tags suggests), but fixes
https://bugzilla.yoctoproject.org/show_bug.cgi?id=9639
as well.

> 
> Signed-off-by: Martin Jansa <martin.jansa@lge.com>
> ---
>  meta/recipes-devtools/cmake/cmake-native_3.6.1.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb b/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb
> index 33930fb..c4d22d8 100644
> --- a/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb
> +++ b/meta/recipes-devtools/cmake/cmake-native_3.6.1.bb
> @@ -2,7 +2,7 @@ require cmake.inc
>  inherit native
>  
>  # Using cmake's internal libarchive, so some dependencies are needed
> -DEPENDS += "bzip2-native zlib-native"
> +DEPENDS += "bzip2-native zlib-native openssl-native libidn-native"
>  
>  SRC_URI += "\
>      file://cmlibarchive-disable-ext2fs.patch \
> -- 
> 2.10.0
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

Re: [meta-oe][meta-networking][PATCH] cmake-native: add dependency on openssl-native libidn-native

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 2358 bytes --]

On 12 September 2016 at 18:05, Martin Jansa <martin.jansa@gmail.com> wrote:

> +DEPENDS += "bzip2-native zlib-native openssl-native libidn-native"
>

So this is coming from the embedded fork of curl inside cmake, which is
optional and likely has more dependencies that we didn't yet notice.  How
about adding a dependency on curl-native instead?

I've a few changes that end up adding curl-native to cmake-native.  Using
Markus's magic buildstats-diff script which he totally should submit to
oe-core:

  PKG                      TASK                 ABSDIFF    RELDIFF
CPUTIME1    CPUTIME2
  cmake-native             do_compile            -19.4s      -3.8%
510.6s -> 491.3s
  cmake-native             do_configure          -18.1s     -11.7%
155.3s -> 137.2s
  xz-native                do_configure           -3.6s     -20.6%
 17.6s -> 14.0s
  m4-native                do_configure           -1.5s      -7.3%
 20.5s -> 19.0s
++openssl-native           do_configure           +3.3s      +inf%
0.0s -> 3.3s
  cmake-native             do_unpack              +3.4s    +118.8%
2.8s -> 6.2s
++openssl-native           do_unpack              +3.6s      +inf%
0.0s -> 3.6s
++bzip2-native             do_configure           +4.9s      +inf%
0.0s -> 4.9s
++curl-native              do_unpack              +4.9s      +inf%
0.0s -> 4.9s
++makedepend-native        do_configure           +7.3s      +inf%
0.0s -> 7.3s
++xproto-native            do_configure           +7.6s      +inf%
0.0s -> 7.6s
++curl-native              do_compile            +20.2s      +inf%
0.0s -> 20.2s
++pkgconfig-native         do_compile            +20.5s      +inf%
0.0s -> 20.5s
++pkgconfig-native         do_configure          +32.3s      +inf%
0.0s -> 32.3s
++openssl-native           do_install            +38.0s      +inf%
0.0s -> 38.0s
++curl-native              do_configure          +48.8s      +inf%
0.0s -> 48.8s
++openssl-native           do_compile           +121.9s      +inf%
0.0s -> 121.9s

Cumulative Time
769.3s -> 1065.4s

cmake builds faster because it is using more libraries from the sysroot,
but we now need to build more pieces to get cmake-native building.   Your
patch is slightly faster (total time of 1023 seconds) but with less
determinism, which is an acceptable tradeoff as far as I'm concerned.

Ross

[-- Attachment #2: Type: text/html, Size: 4058 bytes --]