Bruno Haible <br...@clisp.org> writes:

> Since no version of MSVC has <unistd.h>, it is redundant to list MSVC
> here.

Ah, okay I see. That makes sense.

> And you can also add
>   macOS
> and
>   AIX 7.3
> to the list of platforms. I just checked the result of your test program
> on these platform (using the compilefarm machines).

Thanks! I originally made a testdir with only my changes to the
test program. In the GitHub CI macOS 13, 14, and 15 were the only
platforms to pass. The cfarm machine uses macOS 12 so I have listed
that.

The attached patch makes those changes.

Collin

>From 5662b5eef083b688626d8b502290b886413d635d Mon Sep 17 00:00:00 2001
From: Collin Funk <collin.fu...@gmail.com>
Date: Sun, 16 Feb 2025 13:58:35 -0800
Subject: [PATCH] doc: Adjust platforms that define O_CLOEXEC in unistd.h.

Suggested by Bruno Haible.
* posix-headers/unistd.texi: Mention that macOS 12 and AIX 7.3 do not
define O_CLOEXEC in this header. Remove redundant MSVC.
---
 ChangeLog                     | 7 +++++++
 doc/posix-headers/unistd.texi | 2 +-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 21c1b8a491..3587f070e6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2025-02-16  Collin Funk  <collin.fu...@gmail.com>
+
+	doc: Adjust platforms that define O_CLOEXEC in unistd.h.
+	Suggested by Bruno Haible.
+	* posix-headers/unistd.texi: Mention that macOS 12 and AIX 7.3 do not
+	define O_CLOEXEC in this header. Remove redundant MSVC.
+
 2025-02-16  Bruno Haible  <br...@clisp.org>
 
 	strncasecmp_l: Add tests.
diff --git a/doc/posix-headers/unistd.texi b/doc/posix-headers/unistd.texi
index 4481bcfc1a..044cf83b8f 100644
--- a/doc/posix-headers/unistd.texi
+++ b/doc/posix-headers/unistd.texi
@@ -25,7 +25,7 @@ @node unistd.h
 mingw.
 @item
 This header file does not define @code{O_CLOEXEC} on some platforms:
-glibc 2.41, FreeBSD 14.0, NetBSD 10.0, OpenBSD 7.5, Solaris 11.4, Cygwin 3.5, mingw, MSVC 14.
+glibc 2.41, macOS 12, FreeBSD 14.0, NetBSD 10.0, OpenBSD 7.5, AIX 7.3, Solaris 11.4, Cygwin 3.5, mingw.
 
 @item
 Some platforms provide a @code{NULL} macro that cannot be used in arbitrary
-- 
2.48.1

Reply via email to