public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Matthias Klose <doko@ubuntu.com>
To: Mike Stump <mikestump@comcast.net>,
	       Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
Cc: Iain Sandoe <iain_sandoe@mentor.com>, Jeff Law <law@redhat.com>,
	       Andrew Haley <aph@redhat.com>,
	GCC Patches <gcc-patches@gcc.gnu.org>,
	       GCJ-patches <java-patches@gcc.gnu.org>
Subject: Re: [PATCH] Delete GCJ
Date: Thu, 06 Oct 2016 23:59:00 -0000	[thread overview]
Message-ID: <10704c8f-fd72-8d13-17a1-f522e88345b2@ubuntu.com> (raw)
In-Reply-To: <01B885DF-4340-43C6-A784-21326A9CA8CF@comcast.net>

On 06.10.2016 20:00, Mike Stump wrote:
> On Oct 6, 2016, at 9:56 AM, Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> wrote:
>> I wouldn't hard-fail, but completely disable objc-gc with an appropriate
>> warning.  The Objective-C maintainers may have other preferences, though.

I think I can't do that in the top level make file very well (currently I only
have the pkg-config check there for an early failure, but that check doesn't
tell me if the library is present for all multilib variants). And I can't check
for multilibs because I don't know if the bootstrap compiler is multilib aware.

> gcc historically is fairly weak at complex configurations.  I need the 32 bit libraries to support -m32, but, those libraries might not be present, but do I build all the rest of my libraries, and if i do, do I test them once build, but what is other dependent external libraries are missing.  Do I turn off the multilib, or do I not?
> 
> I used to manage some of this by passing in configure flags to control multilibbing based upon what libraries were install and then run testing based upon that.  Of course, that's all external to gcc proper.  Doesn't really make gcc any easier to configure and build or advance gcc.
> 
> We could smell the system at configure time, and turn on and off multilib variants and things like objc gc.  Target specific, but I think it helps to ponder this in a target independent way.  This can then turn on and off objc gc support directly.  To get it on, one would need to install the needed libraries, and reconfigure and rebuild gcc.  I think I might like that the best.  Has a nice easy of use about it, and then everything gcc does is rather sane (no funny build errors when a needed library isn't present).
> 
> 
> So, I think, if I understand what you propose, I'm fine with that.

So your proposal is to replace the ": dnl ..." line in libobjc/configure.ac with
a hard error message and leave it to the user to correctly configure GCC?  That
would rely on the compiler to find the library in a system wide multilib aware
directory (e.g. /usr/lib/i386-linux-gnu, or /usr/lib32).  Is this the case for
Solaris and Darwin?

I'm fine with that, it wouldn't affect configurations like x86_64-linux-gnu
where multilib is the default (but objc-gc is not).

Looking back at libjava, I think everybody disabled multilibs for libjava,
because nobody had a complete gtk2 stack for multilibs, however that was a
complete subdir, not just a certain configuration in that subdir. Looking back
at libffi and separate released libffi's I first built multilib'ed libffi
libraries from the libffi source for Debian/Ubuntu, then dropped these because
they were not used, and until today GCC internal and external libffi are
hopelessly out of sync, so you couldn't use an external libffi to build libjava.

In the past I looked at updating boehm-gc to recent sources but never finished
because libjava relied on internals.  Afaics this is not the case for objc-gc,
so maybe you could update boehm-gc. But I don't want to go this road myself ...

Matthias

  reply	other threads:[~2016-10-06 23:59 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-05 15:13 Andrew Haley
2016-09-05 15:29 ` Matthias Klose
2016-09-05 15:31   ` Andrew Haley
2016-09-05 18:37   ` Eric Gallager
2016-09-05 18:47     ` Andrew Haley
2016-09-05 15:32 ` Matthew Fortune
2016-09-05 15:34   ` Andrew Haley
     [not found] ` <FB08E20B-5ED1-45BA-BFA0-11E7807840D2@gmail.com>
2016-09-05 16:17   ` Andrew Haley
2016-09-06  9:06     ` Richard Biener
2016-09-06  9:08       ` Jakub Jelinek
2016-09-06 21:18         ` Jeff Law
2016-09-07 11:41           ` Richard Earnshaw (lists)
2016-09-09 11:03       ` Ian Lance Taylor
2016-09-05 16:26 ` Gerald Pfeifer
2016-09-05 16:30   ` Andrew Haley
2016-09-05 18:40   ` Eric Gallager
2016-09-30 15:34   ` Andrew Haley
2017-01-22 16:51     ` Gerald Pfeifer
2017-01-22 17:17       ` Per Bothner
2017-01-22 18:41       ` Per Bothner
2017-01-23  9:01         ` Andrew Haley
2017-01-23  9:05           ` Jakub Jelinek
2017-01-23 12:52             ` Per Bothner
2017-01-23 13:42               ` Jakub Jelinek
2017-01-23 14:51                 ` Andrew Haley
2017-01-22 18:36     ` Gerald Pfeifer
2016-09-10 12:00 ` NightStrike
2016-09-11  8:25   ` Andrew Haley
2016-09-19 15:26     ` Andrew Hughes
2016-10-03 23:12 ` Matthias Klose
2016-10-04  8:40   ` Rainer Orth
2016-10-04  8:41     ` Andrew Haley
2016-10-04 17:23       ` Mike Stump
2016-10-04 17:25         ` Andrew Pinski
2016-10-04 18:40         ` Iain Sandoe
2016-10-05 16:28           ` Jeff Law
     [not found]             ` <2b5de569-1efd-f407-49c2-c9fa84cc5315@ubuntu.com>
2016-10-06 16:16               ` Matthias Klose
2016-10-06 16:43                 ` Rainer Orth
2016-10-06 16:47                   ` Iain Sandoe
2016-10-06 16:54                     ` Matthias Klose
2016-10-06 16:56                       ` Rainer Orth
2016-10-06 17:10                         ` Iain Sandoe
2016-10-06 18:01                         ` Mike Stump
2016-10-06 23:59                           ` Matthias Klose [this message]
2016-10-07  8:31                             ` Iain Sandoe
2016-10-10  4:03                               ` Matthias Klose
2016-10-10  7:58                                 ` Iain Sandoe
2016-11-20 20:42                                   ` Matthias Klose
2016-11-21  5:34                                     ` Sandra Loosemore
2016-11-21 10:24                                     ` Iain Sandoe
2016-11-21 12:57                                       ` Matthias Klose
2016-11-21 16:23                                         ` Sandra Loosemore
2016-11-21 16:40                                           ` Matthias Klose
2016-11-21 17:16                                             ` Rainer Orth
2016-11-21 23:23                                               ` Matthias Klose
2016-11-22 17:13                                                 ` Sandra Loosemore
2016-11-29 21:23                                                 ` Jeff Law
2016-11-30  1:11                                                   ` Remove stray '@' from install.texi (was Re: [PATCH] Delete GCJ) David Malcolm
2016-11-30  1:20                                                     ` Sandra Loosemore
2016-11-30  1:26                                                       ` David Malcolm
2016-11-21 17:16                                             ` [PATCH] Delete GCJ Peter Bergner
2016-09-05 22:49 Bernd Edlinger
2016-09-06  0:20 ` Eric Gallager

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=10704c8f-fd72-8d13-17a1-f522e88345b2@ubuntu.com \
    --to=doko@ubuntu.com \
    --cc=aph@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=iain_sandoe@mentor.com \
    --cc=java-patches@gcc.gnu.org \
    --cc=law@redhat.com \
    --cc=mikestump@comcast.net \
    --cc=ro@CeBiTec.Uni-Bielefeld.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).