public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Ian Taylor <iant@golang.org>
To: Jan-Benedict Glaw <jbglaw@lug-owl.de>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: Patch RFA: Top-level configure patch: disable go on systems where it doesn't work
Date: Mon, 27 Oct 2014 15:20:00 -0000	[thread overview]
Message-ID: <CAOyqgcWO+C86WsGphJcwNQkTZSihFucJS5WrvvFN611mhgMb_g@mail.gmail.com> (raw)
In-Reply-To: <20141027150658.GZ28215@lug-owl.de>

On Mon, Oct 27, 2014 at 8:06 AM, Jan-Benedict Glaw <jbglaw@lug-owl.de> wrote:
>
> On Wed, 2014-10-22 20:36:53 -0700, Ian Taylor <iant@golang.org> wrote:
>> This patch to the top level GCC configure script disables the go
>> languages on some systems where it is known to not work.  Bootstrapped
>> on x86_64-unknown-gnu-linux.
>
> I don't have a clue here, but in what way is Go broken for these
> targets? Bacause this patch "breaks" a number of targets mentioned in
> contrib/config-list.mk.  Maybe Go didn't work on these, but it at
> least built.  Is it FUBAR there? Or just little fixes needed?

I'm not sure exactly what you mean by "breaks," here, as Go is never
in the set of default languages.  It should only break builds that are
using --enable-languages=go.  And for those targets, the Go support
has never worked, so they were already broken.

For Darwin I expect that only small fixes are needed.  This is
http://gcc.gnu.org/PR46986.

For Windows the compiler probably only needs small fixes, but the
libgo build will need a bunch of support, particularly in the syscall
package.  The support for Windows is mostly there in the code, because
it is in the master Go library, but it's not being built for libgo.

For AIX I'm not sure.  It's probably not too much work.

Ian

  reply	other threads:[~2014-10-27 15:19 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-23  6:30 Ian Taylor
2014-10-23 15:31 ` Pedro Alves
2014-10-23 15:35   ` Ian Taylor
2014-10-23 15:41     ` Pedro Alves
2014-10-23 18:27       ` Ian Taylor
2014-10-23 18:17 ` Jeff Law
2014-10-27 15:07 ` Jan-Benedict Glaw
2014-10-27 15:20   ` Ian Taylor [this message]
2014-10-27 16:03     ` Jan-Benedict Glaw
2014-10-27 16:57       ` Ian Taylor
2014-10-30 14:15         ` [PATCH] config-list.mk: Build Go only for supported targets (was: Patch RFA: Top-level configure patch: disable go on systems where it doesn't work) Jan-Benedict Glaw
2014-10-30 15:20           ` Ian Taylor
2014-10-30 19:43             ` Jan-Benedict Glaw
2014-10-31  0:44               ` Ian Taylor
2014-10-31 11:07                 ` Jan-Benedict Glaw

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=CAOyqgcWO+C86WsGphJcwNQkTZSihFucJS5WrvvFN611mhgMb_g@mail.gmail.com \
    --to=iant@golang.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jbglaw@lug-owl.de \
    /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).