From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 86225 invoked by alias); 4 Feb 2018 14:53:15 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 86198 invoked by uid 89); 4 Feb 2018 14:53:14 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-11.1 required=5.0 tests=BAYES_00,FREEMAIL_FROM,GIT_PATCH_2,GIT_PATCH_3,KAM_ASCII_DIVIDERS,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.2 spammy=HX-Received:10.80.142.25 X-Spam-User: qpsmtpd, 2 recipients X-HELO: mail-wm0-f47.google.com Received: from mail-wm0-f47.google.com (HELO mail-wm0-f47.google.com) (74.125.82.47) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 04 Feb 2018 14:53:12 +0000 Received: by mail-wm0-f47.google.com with SMTP id b21so21137430wme.4; Sun, 04 Feb 2018 06:53:11 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=rgxF0wkhf45Yn4WnBNiv1mHnSxL2KKvaCOIpmNrQhlA=; b=R02NZMnc0uphJKuid/CjQgM4WO862ELfAkZ3Q3Q7w8RHrQZjUma33tjUIDY7muZllz J5H/bjaozRO+zEKcYe6DuC8/yjO8Xx6P10FUSlMKwKNGYnRwLKrF1cxmJUnwGfpB1UcE tVqACeRZXbuBv5HPzDzotigBG3BtyvbYcMTQPLpio0c44MbSO9XmZ6u1KzHQ1CWFSvlp OVbguVVpVhPwAAFwjggEp2pLv8Z7RkoiwBfaVSTYrUl/cGg+S55n5swLOSdV3lmTCf+v XkDd8o8TURTH8qwfr+WJW1ouKhpi9P0So9rLskBZj8qShnIJm54gy2BlY8kHbiwKFo3S mpbw== X-Gm-Message-State: AKwxytc3vYdJe1PuQ+kiTXaOn/6xQ40ranrcX4pXFDIy3bqVWzs2DXpA /1O/fg37FfP/L00bTYQaoKsqIT3WdRXYIH9bTf4= X-Google-Smtp-Source: AH8x225O+ZpJeT2b27U9JbGJF458W0GPN6K9HDPUcMtHSCyN3aKKv8bvITJzirpEwFaqH0TMlm0scuFirKGHgVR/2LU= X-Received: by 10.80.142.25 with SMTP id 25mr58256981edw.127.1517755989798; Sun, 04 Feb 2018 06:53:09 -0800 (PST) MIME-Version: 1.0 Received: by 10.80.179.214 with HTTP; Sun, 4 Feb 2018 06:52:29 -0800 (PST) In-Reply-To: References: From: Pip Cet Date: Sun, 04 Feb 2018 14:53:00 -0000 Message-ID: Subject: Re: [PATCH] WebAssembly: Disable subdirectory configuration for unsupported LD To: "Maciej W. Rozycki" Cc: binutils@sourceware.org, gdb-patches@sourceware.org, gcc-patches@gcc.gnu.org Content-Type: text/plain; charset="UTF-8" X-SW-Source: 2018-02/txt/msg00056.txt.bz2 Hello Maciej, thank you for looking at this code! The change looks obviously correct to me. Regards Pip On Sat, Feb 3, 2018 at 3:16 PM, Maciej W. Rozycki wrote: > Remove an LD subdirectory configuration error: > > *** ld does not support target wasm32-unknown-none > *** see ld/configure.tgt for supported targets > make[1]: *** [configure-ld] Error 1 > > which prevents binutils for the WebAssembly target from being built > unless an explicit `--disable-ld' configuration option has been given. > Users must not have to disable features selected by default to get a > working configuration. > > / > * configure.ac (noconfigdirs): Add `ld'. > * configure: Regenerate. > --- > Hi, > > It should be obvious. Also this causes troubles with cross-target > testing. OK to apply? > > Maciej > --- > configure | 3 +++ > configure.ac | 3 +++ > 2 files changed, 6 insertions(+) > > binutils-wasm32-no-ld.diff > Index: binutils/configure > =================================================================== > --- binutils.orig/configure 2018-01-28 00:48:43.000000000 +0000 > +++ binutils/configure 2018-02-02 05:48:19.239569799 +0000 > @@ -3860,6 +3860,9 @@ case "${target}" in > vax-*-*) > noconfigdirs="$noconfigdirs target-newlib target-libgloss" > ;; > + wasm32-*-*) > + noconfigdirs="$noconfigdirs ld" > + ;; > esac > > # If we aren't building newlib, then don't build libgloss, since libgloss > Index: binutils/configure.ac > =================================================================== > --- binutils.orig/configure.ac 2018-01-28 00:48:43.000000000 +0000 > +++ binutils/configure.ac 2018-02-02 05:48:09.444938712 +0000 > @@ -1191,6 +1191,9 @@ case "${target}" in > vax-*-*) > noconfigdirs="$noconfigdirs target-newlib target-libgloss" > ;; > + wasm32-*-*) > + noconfigdirs="$noconfigdirs ld" > + ;; > esac > > # If we aren't building newlib, then don't build libgloss, since libgloss