public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: orbea <orbea@riseup.net>
To: gdb-patches@sourceware.org
Subject: [PATCH] gdb: Don't overwrite CC when checking for BFD support
Date: Wed, 19 Apr 2023 11:16:28 -0700	[thread overview]
Message-ID: <20230419111628.0e1f33f0@Akita> (raw)

[-- Attachment #1: Type: text/plain, Size: 610 bytes --]

In gdb/acinclude.m4 'libtool' is hard coded within the provided custom
CC variable which results in a failure when using slibtool instead of
GNU libtool where the two different libtool implementations are
incompatible when both are used during the same build process at once.

This can be solved by using the default value for CC instead where
autotools is capable of providing an adequate value by default. I
attached a patch doing this.

A bug report was issued here:
https://sourceware.org/bugzilla/show_bug.cgi?id=30295

Please see comment 3 with the explanation where before I was missing
the real issue.

[-- Attachment #2: gdb_acinclude_m4.patch --]
[-- Type: text/x-patch, Size: 855 bytes --]

commit 31245fcf4fc31e732f05205f739317e9c8922da3
Author: orbea <orbea@riseup.net>
Date:   Fri Mar 31 09:46:51 2023 -0700

    gdb: Don't overwrite CC when checking for BFD support
    
    When building with an alternative libtool implementation such as
    slibtool the configure check for BFD features will fail in
    gdb/configure.ac because './libtool' was hardcoded.
    
    Simply removing the line allows the configure check to work correctly.

diff --git a/gdb/acinclude.m4 b/gdb/acinclude.m4
index 173e40b440..5f7589f9f6 100644
--- a/gdb/acinclude.m4
+++ b/gdb/acinclude.m4
@@ -237,7 +237,6 @@ AC_DEFUN([GDB_AC_CHECK_BFD], [
   LDFLAGS="-L../bfd -L../libiberty $LDFLAGS"
   intl=`echo $LIBINTL | sed 's,${top_builddir}/,,g'`
   LIBS="-lbfd -liberty $intl $LIBS"
-  CC="./libtool --quiet --mode=link $CC"
   AC_CACHE_CHECK(
     [$1],
     [$2],

             reply	other threads:[~2023-04-19 18:16 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-19 18:16 orbea [this message]
2023-04-21 14:51 ` Tom Tromey
2023-04-21 18:13   ` orbea

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=20230419111628.0e1f33f0@Akita \
    --to=orbea@riseup.net \
    --cc=gdb-patches@sourceware.org \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).