[oe] [OE-core][PATCH v2] nodejs: fix target ldflags using

[Khem Raj]

On Mon, Mar 18, 2019 at 11:47 AM Andrej Valek <andrej.valek at siemens.com> wrote:
>
>  - do not drop target LDFLAGS
>   - even if the target LDFLAGS have been specified, tools like torque was
>     linked with system libraries
>
> Signed-off-by: Andrej Valek <andrej.valek at siemens.com>
> ---
>  .../nodejs/nodejs/0006-Use-target-ldflags.patch    | 24 ++++++++++++++++++++++
>  meta-oe/recipes-devtools/nodejs/nodejs_10.15.1.bb  |  1 +
>  2 files changed, 25 insertions(+)
>  create mode 100644 meta-oe/recipes-devtools/nodejs/nodejs/0006-Use-target-ldflags.patch
>
> diff --git a/meta-oe/recipes-devtools/nodejs/nodejs/0006-Use-target-ldflags.patch b/meta-oe/recipes-devtools/nodejs/nodejs/0006-Use-target-ldflags.patch
> new file mode 100644
> index 000000000..c6d0cec4b
> --- /dev/null
> +++ b/meta-oe/recipes-devtools/nodejs/nodejs/0006-Use-target-ldflags.patch

cant apply this patch on a ubuntu 14.04 host, can you fix it ? may be
use git format-patch to generate the patch itself.

https://errors.yoctoproject.org/Errors/Details/233590/



> @@ -0,0 +1,24 @@
> +The target LDFLAGS have been ignored. Tools like torque
> +have been loaded from system libraries, even if a native
> +one was the target.
> +|$ ldd torque
> +|    libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1
> +|    libcrypto.so.1.1 => /usr/lib/x86_64-linux-gnu/libcrypto.so.1.1
> +|    libssl.so.1.1 => /usr/lib/x86_64-linux-gnu/libssl.so.1.1
> +|    libicui18n.so.63 => not found
> +|    libicuuc.so.63 => not found
> +...
> +
> +Signed-off-by: Andrej Valek <andrej.valek at siemens.com>
> +
> +diff -Naur node-v10.15.1/deps/v8/gypfiles/common.gypi node-v10.15.1/deps/v8/gypfiles/common.gypi
> +--- node-v10.15.1/deps/v8/gypfiles/common.gypi 2019-03-18 15:01:39.000000000 +0100
> ++++ node-v10.15.1/deps/v8/gypfiles/common.gypi 2019-03-18 15:04:08.628361308 +0100
> +@@ -1106,6 +1106,7 @@
> +             'cflags': [ '-fno-strict-aliasing' ],
> +           }],
> +         ],  # conditions
> ++        'ldflags+': [ '$(LDFLAGS)' ],
> +       }],
> +       ['OS=="solaris"', {
> +         'defines': [ '__C99FEATURES__=1' ],  # isinf() etc.
> diff --git a/meta-oe/recipes-devtools/nodejs/nodejs_10.15.1.bb b/meta-oe/recipes-devtools/nodejs/nodejs_10.15.1.bb
> index 17c9bb0c4..672cd8440 100644
> --- a/meta-oe/recipes-devtools/nodejs/nodejs_10.15.1.bb
> +++ b/meta-oe/recipes-devtools/nodejs/nodejs_10.15.1.bb
> @@ -17,6 +17,7 @@ SRC_URI = "http://nodejs.org/dist/v${PV}/node-v${PV}.tar.xz \
>             file://0003-Crypto-reduce-memory-usage-of-SignFinal.patch \
>             file://0004-Make-compatibility-with-gcc-4.8.patch \
>             file://0005-Link-atomic-library.patch \
> +           file://0006-Use-target-ldflags.patch \
>             "
>  SRC_URI_append_class-target = " \
>             file://0002-Using-native-torque.patch \
> --
> 2.11.0
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel