From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-x432.google.com (mail-wr1-x432.google.com [IPv6:2a00:1450:4864:20::432]) by sourceware.org (Postfix) with ESMTPS id 563F53858437 for ; Wed, 22 Sep 2021 15:30:52 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 563F53858437 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=embecosm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=embecosm.com Received: by mail-wr1-x432.google.com with SMTP id t18so8166090wrb.0 for ; Wed, 22 Sep 2021 08:30:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Dm+389nxU9lM6HbZXiwNdCiQIdfD8qt0I4kDlhANEj4=; b=Uw2Tqt3o4ULuy5dVsVaYX/VCnUbT+kSumOZQNWG1tGtLtYJOAjXHoz7wtb33cHZaZv gqHbKOsxR0PfUYsdyzzbljJ5TEqRpY0HMj6WjLBkHWqvrrNlzKbxqYE6IsaHBf3BKu+K 14uthpm0OelmYLlFk9dLDO1Rah1+uhvMQHp3R5GkS35wA/1N8RB/3/AjVjMfQOsPqTK2 LHk1wqXYspYmlEBeyqAHr0M7Y7HuhHWq1j5WemlkYBn852q79kXvZ8CbALf+ET9gBq+c 02cpoL4jgVKRZ1gn7zhqm43XR+IpPvY6GVaZJDoUAidmplU+vEsq7mqImG0lCCNEGHmt 6EVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Dm+389nxU9lM6HbZXiwNdCiQIdfD8qt0I4kDlhANEj4=; b=RQxVUHPu6s7rczRv3d2NYbK7xpcksdZq1NkCFYpde1Ar2HqX1ky4ZKtdsxf+lADXS8 EmNmRtJ8+gxr9kf645+StvBq3tPTwaZWSRl0C+RjqnW2CSHD3HWUaSBFUanoCsUWixks vgPTVPfYyv+tj7Vzqa48ULqRKTKN7L2B/pp4jwyA+kAaP908llIKwgHEZQiAzGV1xnsJ uMoBQ8Sbr21rtadsoOUpH7Nb7v/KZyKTsvzuP3Yk44SI7rNbDwMSWBkGxr+YHoq01Yfe oeCSLmc8hflOYGJ7oxtNAxB40wrlLi3vjcLXuJoCvHg/8AWo4DXIwX1THzVo9Kiy11Bb Knzg== X-Gm-Message-State: AOAM531wp3aAfH4KNzEeiMr8ULzrkDyiuRy1qXTEbRqu9GuBJaf947eZ O4yZGzL5E6kVoDW6hPTiyG5SskbSaOxiZw== X-Google-Smtp-Source: ABdhPJzLwM03PbeBv1FQHrzkGMX/i6JG6yf1dH6yuTtEPV+zBQ7djq2sOEfg9oHxA87fBm4Ci5bGcw== X-Received: by 2002:a1c:192:: with SMTP id 140mr428wmb.186.1632324649017; Wed, 22 Sep 2021 08:30:49 -0700 (PDT) Received: from localhost (host86-153-58-62.range86-153.btcentralplus.com. [86.153.58.62]) by smtp.gmail.com with ESMTPSA id w14sm2487270wro.8.2021.09.22.08.30.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 22 Sep 2021 08:30:48 -0700 (PDT) From: Andrew Burgess To: gcc-patches@gcc.gnu.org Subject: [PATCH] top-level configure: setup target_configdirs based on repository Date: Wed, 22 Sep 2021 16:30:42 +0100 Message-Id: <20210922153042.3491108-1-andrew.burgess@embecosm.com> X-Mailer: git-send-email 2.25.4 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-10.5 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_BARRACUDACENTRAL, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Sep 2021 15:30:56 -0000 The top-level configure script is shared between the gcc repository and the binutils-gdb repository. The target_configdirs variable in the configure.ac script, defines sub-directories that contain components that should be built for the target using the target tools. Some components, e.g. zlib, are built as both host and target libraries. This causes problems for binutils-gdb. If we run 'make all' in the binutils-gdb repository we end up trying to build a target version of the zlib library, which requires the target compiler be available. Often the target compiler isn't immediately available, and so the build fails. The problem with zlib impacted a previous attempt to synchronise the top-level configure scripts from gcc to binutils-gdb, see this thread: https://sourceware.org/pipermail/binutils/2019-May/107094.html And I'm in the process of importing libbacktrace in to binutils-gdb, which is also a host and target library, and triggers the same issues. I believe that for binutils-gdb, at least at the moment, there are no target libraries that we need to build. My proposal then is to make the value of target_libraries change based on which repository we are building in. Specifically, if the source tree has a gcc/ directory then we should set the target_libraries variable, otherwise this variable is left entry. I think that if someone tries to create a single unified tree (gcc + binutils-gdb in a single source tree) and then build, this change will not have a negative impact, the tree still has gcc/ so we'd expect the target compiler to be built, which means building the target_libraries should work just fine. However, if the source tree lacks gcc/ then we assume the target compiler isn't built/available, and so target_libraries shouldn't be built. There is already precedent within configure.ac for check on the existence of gcc/ in the source tree, see the handling of -enable-werror around line 3658. I've tested a build of gcc on x86-64, and the same set of target libraries still seem to get built. On binutils-gdb this change resolves the issues with 'make all'. Any thoughts? ChangeLog: * configure: Regenerate. * configure.ac (target_configdirs): Only set this when building within the gcc repository. --- ChangeLog | 6 ++++++ configure | 12 ++++++++++-- configure.ac | 12 ++++++++++-- 3 files changed, 26 insertions(+), 4 deletions(-) diff --git a/configure b/configure index 85ab9915402..3ef5c2b553f 100755 --- a/configure +++ b/configure @@ -2849,9 +2849,17 @@ target_tools="target-rda" ## We assign ${configdirs} this way to remove all embedded newlines. This ## is important because configure will choke if they ever get through. ## ${configdirs} is directories we build using the host tools. -## ${target_configdirs} is directories we build using the target tools. +## +## ${target_configdirs} is directories we build using the target +## tools, these are only needed when working in the gcc tree. This +## file is also reused in the binutils-gdb tree, where building any +## target stuff doesn't make sense. configdirs=`echo ${host_libs} ${host_tools}` -target_configdirs=`echo ${target_libraries} ${target_tools}` +if test -d ${srcdir}/gcc; then + target_configdirs=`echo ${target_libraries} ${target_tools}` +else + target_configdirs="" +fi build_configdirs=`echo ${build_libs} ${build_tools}` diff --git a/configure.ac b/configure.ac index 1df038b04f3..d1217e3f886 100644 --- a/configure.ac +++ b/configure.ac @@ -180,9 +180,17 @@ target_tools="target-rda" ## We assign ${configdirs} this way to remove all embedded newlines. This ## is important because configure will choke if they ever get through. ## ${configdirs} is directories we build using the host tools. -## ${target_configdirs} is directories we build using the target tools. +## +## ${target_configdirs} is directories we build using the target +## tools, these are only needed when working in the gcc tree. This +## file is also reused in the binutils-gdb tree, where building any +## target stuff doesn't make sense. configdirs=`echo ${host_libs} ${host_tools}` -target_configdirs=`echo ${target_libraries} ${target_tools}` +if test -d ${srcdir}/gcc; then + target_configdirs=`echo ${target_libraries} ${target_tools}` +else + target_configdirs="" +fi build_configdirs=`echo ${build_libs} ${build_tools}` m4_divert_text([PARSE_ARGS], -- 2.25.4