Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Adrian Bunk <bunk@stusta.de>
To: Khem Raj <raj.khem@gmail.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 8/9] binutils: Bail out if gold is used on 32/64 bit RISC-V
Date: Thu, 13 Feb 2020 09:31:51 +0200	[thread overview]
Message-ID: <20200213073151.GA20530@localhost> (raw)
In-Reply-To: <11cf7d82deafd496d30a4a28a38a3b1121762f9e.1581575578.git.raj.khem@gmail.com>

On Wed, Feb 12, 2020 at 10:38:22PM -0800, Khem Raj wrote:
> gold is not supporting risc-v yet and, this will cause subltle errors
> during build e.g. autoconf not finding certain features like visibility
> and it could take a while to unwind the problem. Its better to error out
> early
>...
> @@ -163,3 +165,8 @@ inherit update-alternatives
>  ALTERNATIVE_PRIORITY = "100"
>  
>  ALTERNATIVE_${PN}_class-target = "${USE_ALTERNATIVES_FOR}"
> +
> +python () {
> +    if bb.utils.contains('DISTRO_FEATURES', 'ld-is-gold', True, False, d):
> +        bb.fatal("Gold linker does not _yet_ support RISC-V architecture please remove ld-is-gold from DISTRO_FEATURES")
> +}

This lacks some check for RISC-V,
currently it errors for all architectures.

cu
Adrian


  reply	other threads:[~2020-02-13  7:31 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-13  6:38 [PATCH 0/9] recipe updates, testing fixes Khem Raj
2020-02-13  6:38 ` [PATCH 1/9] qemuarm: Disable highmem when QB_MACHINE is virt Khem Raj
2020-02-13  6:38 ` [PATCH 2/9] oeqa: Use --disable-maintainer-mode configure option Khem Raj
2020-02-13  6:38 ` [PATCH 3/9] libsolv: Enable rpm packageconfig by default only if rpm O_P_M is enabled Khem Raj
2020-02-13  6:38 ` [PATCH 4/9] dnf, libdnf: Ignore if PACKAGE_CLASSES does not have rpm Khem Raj
2020-02-13  6:38 ` [PATCH 5/9] musl: Update to latest tip Khem Raj
2020-02-13  6:38 ` [PATCH 6/9] gdb: Upgrade to 9.1 release Khem Raj
2020-02-13 23:38   ` Richard Purdie
2020-02-14  1:33     ` Khem Raj
2020-02-13  6:38 ` [PATCH 7/9] busybox: Backport patches to support removal of __NR_clock_gettime Khem Raj
2020-02-13  6:38 ` [PATCH 8/9] binutils: Bail out if gold is used on 32/64 bit RISC-V Khem Raj
2020-02-13  7:31   ` Adrian Bunk [this message]
2020-02-13 16:08     ` Khem Raj
2020-02-13  6:38 ` [PATCH 9/9] ltp: Fix tescases with 64bit time_t using 32bit arches Khem Raj
2020-02-13  8:21 ` [PATCH 0/9] recipe updates, testing fixes Alexander Kanavin
2020-02-13 16:07   ` Khem Raj
2020-02-13 16:19     ` Alexander Kanavin
2020-02-13 16:44       ` Khem Raj
2020-02-13 16:53         ` Alexander Kanavin
2020-02-13 21:37           ` Khem Raj
2020-02-13 22:05             ` Alexander Kanavin
2020-02-14  1:39               ` Khem Raj
2020-02-14  7:05                 ` Alexander Kanavin
2020-02-14  7:14                   ` Khem Raj
2020-02-14 12:19                     ` Alexander Kanavin
2020-02-14 19:21                       ` Alexander Kanavin
2020-02-14 19:42                         ` Khem Raj

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=20200213073151.GA20530@localhost \
    --to=bunk@stusta.de \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=raj.khem@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox