On Mon, Apr 21, 2025 at 10:52:22AM +0200, Richard Biener wrote:
> On Fri, Apr 18, 2025 at 8:10 PM Jakub Jelinek <ja...@redhat.com> wrote:
> >
> > On Fri, Apr 18, 2025 at 06:04:29PM +0200, Rainer Orth wrote:
> > > That's one option, but maybe it's better the other way round: instead of
> > > excluding known-bad targets, restrict cobol to known-good ones
> > > (i.e. x86_64-*-linux* and aarch64-*-linux*) instead.
> > >
> > > I've been using the following for this (should be retested for safety).
> >
> > I admit I don't really know what works and what doesn't out of the box now,
> > but your patch looks reasonable to me for 15 branch.
> >
> > Richard, Robert and/or James, do you agree?
>
> I agree to restrict =all to enable cobol only for known-good platform triples.
> But IIRC that's what libgcobol configure.tgt does - IIRC intent was to allow
But libgcobol configure.tgt does it only for the target case.
Much more important right now is the host whitelist case.
The code in the FE is still not sufficiently portable and so even
i686-solaris -> x86_64-linux --enable-languages=all cross just won't build out
of the box (and even if it would, it wouldn't work properly).
Or x86_64-freebsd -> x86_64-linux might not either.
So, I think we want Rainer's patch here.
> a cobol build with explicit 'cobol' included even when configure.tgt claims
> unsupported? So why's *-*solaris now included in =all?
>
> I'm a bit confused, I thought we had =all restricted already.
> > > 2025-03-17 Rainer Orth <r...@cebitec.uni-bielefeld.de>
> > >
> > > PR cobol/119217
> > > * configure.ac: Restrict cobol to aarch64-*-linux*,
> > > x86_64-*-linux*.
> > > * configure: Regenerate.
Jakub
