Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: "Joseph S. Myers" <joseph@codesourcery.com>
Cc: gdb-patches@sourceware.org, binutils@sourceware.org,
	        gcc-patches@gcc.gnu.org,
	Joel Brobecker <brobecker@adacore.com>
Subject: Re: Fix ACX_LARGEFILE to use canonical host/target names
Date: Sun, 22 Nov 2009 14:20:00 -0000	[thread overview]
Message-ID: <20091122141806.GA10982@host0.dyn.jankratochvil.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0911202236450.4910@digraph.polyomino.org.uk>

On Fri, 20 Nov 2009 23:38:16 +0100, Joseph S. Myers wrote:
> The ACX_LARGEFILE macro checks the values of $host and $target, which
> may or may not be canonical names at this point depending on which
> other macros have been called first;

Missed that, thanks for catching it.


> it should require
> AC_CANONICAL_HOST and AC_CANONICAL_TARGET to ensure they are both
> canonical.  This shows up as build failures for GDB configured
> --build=i686-pc-linux-gnu --host=i686-solaris2.10 --target=i686-solaris2.10.

and so the config/largefile.m4 pattern
  sparc-*-solaris*|i[3-7]86-*-solaris*)
would not match `i686-solaris2.10'.  Canonical form of `i686-solaris2.10'
after AC_CANONICAL_HOST or AC_CANONICAL_TARGET is `i686-pc-solaris2.10' which
would already match the config/largefile.m4 pattern.

On native OpenSolaris 2009.06 snv_111b X86 tried that
  ./configure --host=i386-solaris2.11 --target=i386-solaris2.11; make
will fail due to it with:
  /usr/include/sys/old_procfs.h:57:2: #error "Cannot use procfs in the large file compilation environment"
(despite the OS is 64bit the default build targets are 32bit there; as on ppc).

While all the subdirectories calling ACX_LARGEFILE already contain
AC_CANONICAL_TARGET (equivalent form AC_CANONICAL_SYSTEM is not present there)
none of them explicitly calls AC_CANONICAL_HOST, bfd/ already contains it (by
../ltoptions.m4) but gdb/ does not - which causes the compilation error.


> with regeneration of the GDB and binutils configure scripts that use
> ACX_LARGEFILE?

I would like to get regenerated all the subdirectories calling ACX_LARGEFILE
(bfd/ binutils/ gas/ gprof/ ld/ gdb/) as the common config/largefile.m4 file
purpose was exactly to get all the subdirectories ABI compatible, as described
in the mails introducing ACX_LARGEFILE.



Thanks,
Jan


  reply	other threads:[~2009-11-22 14:20 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-20 22:39 Joseph S. Myers
2009-11-22 14:20 ` Jan Kratochvil [this message]
2009-11-28 20:11 ` Ping " Joseph S. Myers
2009-11-30 10:31 ` Nick Clifton

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=20091122141806.GA10982@host0.dyn.jankratochvil.net \
    --to=jan.kratochvil@redhat.com \
    --cc=binutils@sourceware.org \
    --cc=brobecker@adacore.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=joseph@codesourcery.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