public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Roger Sayle <roger@eyesopen.com>
To: Eric Botcazou <ebotcazou@libertysurf.fr>
Cc: gcc-patches@gcc.gnu.org, <java-patches@gcc.gnu.org>
Subject: Re: [JAVA] /bin/sh portability issues in gen-classlist.sh.in
Date: Fri, 07 Jul 2006 13:36:00 -0000	[thread overview]
Message-ID: <Pine.LNX.4.44.0607070644550.14880-100000@www.eyesopen.com> (raw)
In-Reply-To: <200607071218.36304.ebotcazou@libertysurf.fr>


On Fri, 7 Jul 2006, Eric Botcazou wrote:
> > The following patch, the first in a series, fixes some shell portability
> > issues building libjava using /bin/sh on Solaris.  As pointed out by
> > Andrew Pinski, I should probably simply use CONFIG_SHELL, however I
> > think I've managed to resolve all the remaining shell issues preventing
> > bootstraping mainline GCC on Solaris/x86 with the native /bin/sh.
>
> I think it's a lost cause, in my experience /bin/sh randomly fails to
> configure target libraries on Solaris.  I'll personally make no effort
> to support /bin/sh on Solaris even if your patches go in, i.e. I'll keep
> closing PRs related to the problem as invalid.

I'm reminded that the GCC Development Mission Statement, explicitly
states "to ensure that GCC ... works on multiple architectures and
diverse environments".  http://gcc.gnu.org/gccmission.html

I don't disagree that there might still be fundamental bugs in Solaris'
/bin/sh or even the kernel that ultimately make this task fruitless.
However, knowing that /bin/sh has been a bane of the GCC build system
for many years, I was surprised at how easy the port was.  GCC 3.4.3 built
out of the box, and mainline three stage bootstrapped the compilers and
built most of the runtime libraries only to fail in multilibed boehmgc
and libjava.  This even makes this issue a regression.


I regularly use CONFIG_SHELL on several platforms, but I suspect most
often for performance rather than for correctness, or to workaround
portability issue.

Running "what /bin/sh" on my Solaris/x86 box shows that the executable
is dated 1st October 2005.  Perhaps after nearly/over? ten years of
brokeness SUN's code is now stable enough for GCC to meet them half
way?

There is also a fundamental difference between the problems I'm fixing
and the issues you report (that I may just not have encountered yet).
Instability issues and missing POSIX support are bugs that should be
filed with SUN Microsystems.  But using command line options and
functionality not available in the tool being invoked are bugs in
GCC's build system.  The latter I can do something about myself,
and the former may get fixed sooner or later (perhaps already?).

Roger
--

  reply	other threads:[~2006-07-07 13:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-07  4:32 Roger Sayle
2006-07-07  9:10 ` Andrew Haley
2006-07-07 12:59   ` Roger Sayle
2006-07-07 13:08     ` Andrew Haley
2006-07-07 10:14 ` Eric Botcazou
2006-07-07 13:36   ` Roger Sayle [this message]
2006-07-07 13:44     ` Eric Botcazou
2006-07-07 13:33 ` Paolo Bonzini
2006-07-08  0:57 ` Andrew Pinski
2006-07-08  1:17   ` Ralf Wildenhues
2006-08-18 19:48 ` Tom Tromey
2006-08-24 12:46   ` Roger Sayle
2006-10-10 17:10     ` Tom Tromey

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=Pine.LNX.4.44.0607070644550.14880-100000@www.eyesopen.com \
    --to=roger@eyesopen.com \
    --cc=ebotcazou@libertysurf.fr \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=java-patches@gcc.gnu.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).