[PATCH libgpg-error] Add support for IBM z/OS

[Sachin T]

Hi Werner,

Thanks for the feedback.

Yes, this patch is not required as AF_UNIX and AF_LOCAL are indeed same value. I would also define MVS_SYSTEM for z/os platform.
Will share the updated patch with these changes.

I have also addressed Jacob’s comments in a separate thread.

Regards
Sachin

On 11/06/25, 1:15 PM, "Werner Koch" <wk at gnupg.org> wrote:
Hi!

On Tue, 10 Jun 2025 15:15, Sachin T said:
> +#if defined(__MVS__)
> +  if (socketpair (AF_UNIX, SOCK_STREAM, 0, filedes) == -1)
> +#else
>    if (socketpair (AF_LOCAL, SOCK_STREAM, 0, filedes) == -1)
> +#endif

AF_UNIX is the legacy name for AF_LOCAL and thus should share the same
value.  The configure script should thus detect a missing AF_LOCAL and
redefine it to AF_UNIX.  This way you don't need to clobber the code
with cpp conditionals.  This is the autoconf way of doing things.

However, sometimes it is useful to diretcly test for teh syste, In
particialr for Windows we need special code at some palces and for that
we use W32_SYSTEM.  I would also suggest to define MVS_SYSTEM so that we
don't need to directly use __MVS__.

Please see also Jacob's comments.


Shalom-Salam,

   Werner

--
The pioneers of a warless world are the youth that
refuse military service.             - A. Einstein

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnupg-devel/attachments/20250612/2ec62d7c/attachment.html>