All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bruce Ashfield <bruce.ashfield@windriver.com>
To: Zhenhua Luo <zhenhua.luo@freescale.com>, <yocto@yoctoproject.org>
Cc: b40290@freescale.com, b40527@freescale.com
Subject: Re: [meta-cloud-services][PATCH] erlang: skip the host library path check to fix cross-compile issue
Date: Tue, 19 Nov 2013 08:29:29 -0500	[thread overview]
Message-ID: <528B67B9.8030505@windriver.com> (raw)
In-Reply-To: <1384856869-10658-1-git-send-email-zhenhua.luo@freescale.com>

Thanks for the patch!

A couple of requests.

Can you list the compile issue here ? i.e. just a bit of the error
message showing it clearly. Someone else searching for a problem will
find it very useful!

On 13-11-19 05:27 AM, Zhenhua Luo wrote:
> Signed-off-by: Zhenhua Luo <zhenhua.luo@freescale.com>
> ---
>   .../erlang/erlang-fix-build-issue-in-Yocto.patch   | 25 ++++++++++++++++++++++
>   .../recipes-devtools/erlang/erlang_R15B.bb         |  2 ++
>   2 files changed, 27 insertions(+)
>   create mode 100644 meta-openstack/recipes-devtools/erlang/erlang/erlang-fix-build-issue-in-Yocto.patch
>
> diff --git a/meta-openstack/recipes-devtools/erlang/erlang/erlang-fix-build-issue-in-Yocto.patch b/meta-openstack/recipes-devtools/erlang/erlang/erlang-fix-build-issue-in-Yocto.patch
> new file mode 100644
> index 0000000..2d4b7b5
> --- /dev/null
> +++ b/meta-openstack/recipes-devtools/erlang/erlang/erlang-fix-build-issue-in-Yocto.patch
> @@ -0,0 +1,25 @@
> +Upstream-Status: Inappropriate [The fix is specific to Yocto build env]
> +
> +Skip host library path check to fix following Yocto insane check issue:
> +ERROR: This autoconf log indicates errors, it looked at host include and/or library paths while determining system capabilities.
> +Rerun configure task after fixing this.
> +
> +Signed-off-by: Zhenhua Luo <zhenhua.luo@freescale.com>
> +
> +--- otp_src_R15B/erts/configure.in.orig	2011-12-14 04:22:11.000000000 -0600
> ++++ otp_src_R15B/erts/configure.in	2013-11-19 04:15:33.694334610 -0600
> +@@ -1469,10 +1469,10 @@
> + 	# ln -s libdlpi.so.1 $try_dlpi_lib/libdlpi.so
> + 	])
> +    fi
> +-   LDFLAGS="-L$try_dlpi_lib -R$try_dlpi_lib $LDFLAGS"
> +-   unset -v try_dlpi_lib
> +-   AC_MSG_NOTICE([Extending the search to include /lib])
> +-   AC_CHECK_LIB(dlpi, dlpi_open)
> ++#   LDFLAGS="-L$try_dlpi_lib -R$try_dlpi_lib $LDFLAGS"
> ++#   unset -v try_dlpi_lib
> ++#   AC_MSG_NOTICE([Extending the search to include /lib])
> ++#   AC_CHECK_LIB(dlpi, dlpi_open)
> +    if test x"$ac_cv_lib_dlpi_dlpi_open" = x"no"; then
> +       LDFLAGS="$save_ldflags"
> +    fi
> diff --git a/meta-openstack/recipes-devtools/erlang/erlang_R15B.bb b/meta-openstack/recipes-devtools/erlang/erlang_R15B.bb
> index 68d144b..8d8e28b 100644
> --- a/meta-openstack/recipes-devtools/erlang/erlang_R15B.bb
> +++ b/meta-openstack/recipes-devtools/erlang/erlang_R15B.bb
> @@ -9,6 +9,7 @@ EXTRA_OEMAKE = "BUILD_CC='${BUILD_CC}'"
>
>   EXTRA_OECONF = "--with-ssl=${STAGING_DIR_HOST}${layout_exec_prefix}"
>
> +EXTRA_OECONF_append_powerpc = " --disable-smp-support --disable-hipe"

This should be a separate patch. Can you split the commit and send
this as a two part series ?

Bruce

>   EXTRA_OECONF_append_arm = " --disable-smp-support --disable-hipe"
>   EXTRA_OECONF_append_armeb = " --disable-smp-support --disable-hipe"
>   EXTRA_OECONF_append_mipsel = " --disable-smp-support --disable-hipe"
> @@ -55,5 +56,6 @@ FILES_${PN}-staticdev += "${libdir}/*/*/*/*.a ${libdir}/*/*/*/*/*.a ${libdir}/*/
>
>   FILES_${PN}-libs-dbg += " ${libdir}/erlang/*/.debug ${libdir}/erlang/*/*/.debug ${libdir}/erlang/*/*/*/.debug ${libdir}/erlang/*/*/*/*/.debug ${libdir}/erlang/*/*/*/*/*/.debug "
>
> +SRC_URI += "file://erlang-fix-build-issue-in-Yocto.patch"
>   SRC_URI[md5sum] = "dd6c2a4807551b4a8a536067bde31d73"
>   SRC_URI[sha256sum] = "5bc34fc34fc890f84bae7ff1f7c81fbec2c9aa28a0ef51a57d7a8192204d8aa2"
>



  reply	other threads:[~2013-11-19 13:29 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-19 10:27 [meta-cloud-services][PATCH] erlang: skip the host library path check to fix cross-compile issue Zhenhua Luo
2013-11-19 13:29 ` Bruce Ashfield [this message]
2013-11-20 10:34   ` Zhenhua Luo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=528B67B9.8030505@windriver.com \
    --to=bruce.ashfield@windriver.com \
    --cc=b40290@freescale.com \
    --cc=b40527@freescale.com \
    --cc=yocto@yoctoproject.org \
    --cc=zhenhua.luo@freescale.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.