From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31692 invoked by alias); 31 Jan 2020 21:40:07 -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 31684 invoked by uid 89); 31 Jan 2020 21:40:07 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-16.4 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy=x, brobecker, Brobecker, cxx X-HELO: us-smtp-delivery-1.mimecast.com Received: from us-smtp-1.mimecast.com (HELO us-smtp-delivery-1.mimecast.com) (205.139.110.61) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 31 Jan 2020 21:40:02 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1580506801; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Wc/Sdnm/VCxn6X5hEple3RBF2B05Px7X7ykH1pw8c/Q=; b=A4NRU1a+7X0fAqdEnAbo323HMu/5jnKOUAkfWo/d04EO6nG6XrSOOxC3m4cMzlWhin8iKF bSGMFVswh8gbb8fPkVeaYEHzWksRgT4UrNssvjzQLS/jlu+R4adrBAfYlR/wV6lWAqzhY/ bDzItJ0ylJoV/fevqPh0UGfuZiUCN/Q= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-284-GMBh1Id8PyOBVxGiiNE69g-1; Fri, 31 Jan 2020 16:39:51 -0500 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id E2CB4109A748; Fri, 31 Jan 2020 21:39:49 +0000 (UTC) Received: from localhost (unused-10-15-17-196.yyz.redhat.com [10.15.17.196]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7B2E85DA81; Fri, 31 Jan 2020 21:39:49 +0000 (UTC) From: Sergio Durigan Junior To: Eli Zaretskii Cc: brobecker@adacore.com, simark@simark.ca, gdb-patches@sourceware.org, tom@tromey.com Subject: Re: [RFA/gdb-9-branch] Abort configure immediately if building GDB in tree References: <20200105073000.1012-1-brobecker@adacore.com> <4d350c75-6e52-5c4c-5901-4c391970b643@simark.ca> <20200117181554.GB7007@adacore.com> <20200117183203.GC7007@adacore.com> <87imksmz79.fsf@redhat.com> <834kwcm63u.fsf@gnu.org> <87r1zflbft.fsf@redhat.com> Date: Fri, 31 Jan 2020 22:52:00 -0000 In-Reply-To: <87r1zflbft.fsf@redhat.com> (Sergio Durigan Junior's message of "Fri, 31 Jan 2020 13:37:10 -0500") Message-ID: <87eevfl2ze.fsf@redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2020-01/txt/msg01006.txt.bz2 On Friday, January 31 2020, I wrote: > On Friday, January 31 2020, Eli Zaretskii wrote: > >>> From: Sergio Durigan Junior >>> Cc: Simon Marchi , gdb-patches@sourceware.org, Tom = Tromey >>> Date: Thu, 30 Jan 2020 16:06:18 -0500 >>>=20 >>> $ ./src-release.sh -x gdb >>> =3D=3D> Cleaning sources. >>> =3D=3D> Making gdb-9.0.90.20200130/ >>> =3D=3D> configure --target=3Di386-pc-linux-gnu --disable-binutils >>> --disable-gas --disable-gold --disable-gprof --disable-ld >>> --enable-gdb --enable-libctf --enable-libdecnumber --enable-readline >>> --enable-sim >>> configure: error: GDB must be configured and built in a directory sep= arate from its sources. >>> ... >>>=20 >>> I'm wondering what to do here. I think I found a hacky and ugly >>> solution here, but I'm trying to see if there's a better approach. >> >> A special command-line switch to 'configure', to be used by >> src-release.sh? > > A quick attempt here didn't really work. The script is able to finish > the build and create a tarball, but for some reason the gdb directory is > not configured/built. OK, I think I have something that works. When 'configure' is invoked with '--enable-src-release-build', a tarball is generated and, as far as I have checked, contains the correct snapshot. WDYT? --=20 Sergio GPG key ID: 237A 54B1 0287 28BF 00EF 31F4 D0EB 7628 65FC 5E36 Please send encrypted e-mail if possible http://sergiodj.net/ diff --git a/ChangeLog b/ChangeLog index 0bfe989885..de13206e2e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2020-01-31 Sergio Durigan Junior + + * configure.ac: Don't abort the build if trying to build GDB in tree + _and_ invoking with '--enable-src-release-build'. + * configure: Regenerate. + * src-release.sh (do_proto_toplev): Invoke 'configure' using + '--enable-src-release-build'. + 2020-01-17 Joel Brobecker =20 * configure.ac: Abort the build with an error if trying to build diff --git a/configure b/configure index 7168a4e90e..276f33fee3 100755 --- a/configure +++ b/configure @@ -2279,7 +2279,7 @@ ac_compiler_gnu=3D$ac_cv_c_compiler_gnu =20 =20 =20 -if test x"${enable_gdb}" !=3D x"no"; then +if test x"${enable_gdb}" !=3D x"no" -a x"${enable_src_release_build}" !=3D= x"yes"; then # For this branch, we do not support building GDB in-tree. # Try to detect whether we are in this situation or not by # searching for a couple of known files in the source directory. diff --git a/configure.ac b/configure.ac index e5ca1eaa57..cd0867dd76 100644 --- a/configure.ac +++ b/configure.ac @@ -33,7 +33,7 @@ m4_include([config/isl.m4]) AC_INIT(move-if-change) AC_DISABLE_OPTION_CHECKING =20 -if test x"${enable_gdb}" !=3D x"no"; then +if test x"${enable_gdb}" !=3D x"no" -a x"${enable_src_release_build}" !=3D= x"yes"; then # For this branch, we do not support building GDB in-tree. # Try to detect whether we are in this situation or not by # searching for a couple of known files in the source directory. diff --git a/src-release.sh b/src-release.sh index 275f0f24b5..68e824c749 100755 --- a/src-release.sh +++ b/src-release.sh @@ -99,8 +99,8 @@ do_proto_toplev() *) disables=3D"$disables --disable-$dir" ;; esac done - echo "=3D=3D> configure --target=3Di386-pc-linux-gnu $disables $enable= s" - ./configure --target=3Di386-pc-linux-gnu $disables $enables + echo "=3D=3D> configure --target=3Di386-pc-linux-gnu --enable-src-rele= ase-build=3Dyes $disables $enables" + ./configure --target=3Di386-pc-linux-gnu --enable-src-release-build=3D= yes $disables $enables $MAKE configure-host configure-target \ ALL_GCC=3D"" ALL_GCC_C=3D"" ALL_GCC_CXX=3D"" \ CC_FOR_TARGET=3D"$CC" CXX_FOR_TARGET=3D"$CXX"