Re: [PATCH] fcntl-h: define O_CLOEXEC and O_EXEC if not defined; use new defines

[Paul Eggert]

On 09/10/10 12:13, Jim Meyering wrote:

> I've checked a few projects, and found none like that, so maybe there's
> no actual trouble, but it does deserve a mention in NEWS.

Yes, I did similar checking, though no doubt not as extensive as yours.
I'm aware of the header/kernel mismatch problem that Eric mentioned,
but that problem exists independently of this change so it should be OK.

I installed the following further patch.

>From 926ad675190e3f05b94acea66cfd29a014b99ab4 Mon Sep 17 00:00:00 2001
From: Paul Eggert <address@hidden>
Date: Fri, 10 Sep 2010 12:26:03 -0700
Subject: [PATCH] * NEWS: Document recent changes to fcntl-h.

---
 ChangeLog |    1 +
 NEWS      |    3 +++
 2 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 48a4d32..80e0736 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,7 @@
 2010-09-10  Paul Eggert  <address@hidden>
 
        fcntl-h: define O_CLOEXEC and O_EXEC if not defined; use new defines
+       * NEWS: Document recent changes to fcntl-h.
        * doc/posix-headers/fcntl.texi (fcntl.h): Document that
        O_CLOEXEC is now defined to 0 if it is not defined, like other flags.
        Also, O_EXEC is now defined to be O_RDONLY if O_EXEC is not defined.
diff --git a/NEWS b/NEWS
index e062efa..3732937 100644
--- a/NEWS
+++ b/NEWS
@@ -12,6 +12,9 @@ User visible incompatible changes
 
 Date        Modules         Changes
 
+2010-09-10 fcntl-h          This module now defaults O_CLOEXEC to 0, and
+                            it defaults O_EXEC and O_SEARCH to O_RDONLY.
+
 2010-08-28  realloc         This module is deprecated. Use 'realloc-gnu'
                             instead. It will be removed 2012-01-01.
 
-- 
1.7.2