On 11.08.22 12:02, Thomas Munro wrote:
  * The concept of a no-Unix-socket build is removed.  We should be
able to do that now, right?  Peter E seemed to say approximately that
in the commit message for 797129e5.  Or is there a thought that a new
operating system might show up that doesn't have 'em and we'd wish
we'd kept this stuff well marked out?

Most uses of HAVE_UNIX_SOCKETS are not useful independent of that question. For example, you patch has

@@ -348,7 +343,6 @@ StreamServerPort(int family, const char *hostName, unsigned short portNumber,
        hint.ai_flags = AI_PASSIVE;
        hint.ai_socktype = SOCK_STREAM;

-#ifdef HAVE_UNIX_SOCKETS
        if (family == AF_UNIX)
        {
                /*

But on a platform without support for Unix sockets, family just won't be AF_UNIX at run time, so there is no need to hide that if branch.

Note that we already require that AF_UNIX is defined on all platforms, even if the kernel doesn't support Unix sockets.

But maybe it would be better to make that a separate patch from the sys/un.h configure changes, just so there is more clarity around it.


Reply via email to