Hi Robert,

>> -----Original Message-----
>> From: Rainer Orth <r...@cebitec.uni-bielefeld.de>
>> Sent: Friday, April 11, 2025 05:26
>> To: gcc-patches@gcc.gnu.org
>> Cc: Robert Dubner <rdub...@symas.com>; James K. Lowden
>> <jklow...@cobolworx.com>
>> Subject: [PATCH] libgcobol: Heed --enable-libgcobol
>>
>> If some target isn't listed as supported in configure.tgt,
>> --enable-libgcobol cannot override that.  However, that's what should
>> happen just like an explicit --enable-languages=cobol forces the
>> frontend to be built.
>>
>> This patch, shamelessly adapted from libphobos, does just that.
>>
>> Tested on amd64-pc-solaris2.11, sparcv9-sun-solaris2.11, and
>> x86_64-pc-linux-gnu.
>>
>> Ok for trunk?
>
> I was unable to apply this patch.  "git apply ..." results in
>
> :~/repos/gcc-cobol$ git apply libgcobol-enable-libgcobol.patch
> error: patch failed: libgcobol/configure:788
> error: libgcobol/configure: patch does not apply
> error: patch failed: libgcobol/configure.ac:40
> error: libgcobol/configure.ac: patch does not apply
>
> I don't understand the problem, but I don't know much about how diff and 
> apply work.

That's my fault actually: I had rebased the patch locally a few times
since submission due to upstream changes to libgcobol/configure.ac.
However, since the gist of the patch wasn't affected by those, I'd
neglected to post those rebased versions.

> I have no way of checking the solaris part of it; I was just trying to do 
> "due diligence", and check that it didn't adversely affect x86_64-linux-gnu.

You'd have been able to verify it on every target not currently marked
as supported in libgcobol/configure.tgt, actually.  I'd already run
bootstraps with it on Linux/x86_64 just to make sure nothing breaks.

> But since I can't do that, all I can say is, I see no reason for you not to 
> apply a patch you know works.

Thanks.  All four patches have been committed now.

        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to