[PATCH 1/2] Fix typo in _REGEX_VERSION name
Siddhesh Poyarekar
siddhesh@redhat.com
Mon Sep 15 04:54:00 GMT 2014
On Fri, Sep 12, 2014 at 11:04:05AM -0700, Roland McGrath wrote:
> _REGEX_VERSION is never defined and in fact it was removed from the
> standard. For backward compatibility we must support _SC_REGEX_VERSION and
> return -1 without setting errno, but I don't think we should treat this
> like the real cases any more. That is, just unconditionally return -1
> and a comment about it having been removed in 1003.1-2004.
This is what I have committed.
Siddhesh
commit 61fe374a44a92621e0b75ec1f011ff1fba6c2148
Author: Siddhesh Poyarekar <siddhesh@redhat.com>
Date: Mon Sep 15 10:16:14 2014 +0530
Remove _POSIX_REGEX_VERSION
There is no _POSIX_REGEX_VERSION, so don't check for it.
_REGEX_VERSION has been removed as well[1], so only keep the -1 return
for backward compatibility. I found this when trying to make the
getconf environment variables typo-proof.
* sysdeps/posix/sysconf.c (__sysconf): Return -1 for
_SC_REGEX_VERSION.
[1] http://pubs.opengroup.org/onlinepubs/009695399/functions/sysconf.html
diff --git a/ChangeLog b/ChangeLog
index d010cc9..e316f8c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
2014-09-15 Siddhesh Poyarekar <siddhesh@redhat.com>
+ * sysdeps/posix/sysconf.c (__sysconf): Return -1 for
+ _SC_REGEX_VERSION.
+
* posix/getconf.c (vars): Add _POSIX_IPV6 and
_POSIX_RAW_SOCKETS.
diff --git a/sysdeps/posix/sysconf.c b/sysdeps/posix/sysconf.c
index cd2fb5a..e815cd5 100644
--- a/sysdeps/posix/sysconf.c
+++ b/sysdeps/posix/sysconf.c
@@ -983,12 +983,10 @@ __sysconf (name)
#else
return -1;
#endif
+ /* _REGEX_VERSION has been removed with IEEE Std 1003.1-2001/Cor 2-2004,
+ item XSH/TC2/D6/137. */
case _SC_REGEX_VERSION:
-#if _POSIX_REGEX_VERSION > 0
- return _POSIX_REGEX_VERSION;
-#else
return -1;
-#endif
case _SC_SHELL:
#if _POSIX_SHELL > 0
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20140915/1570b90b/attachment.sig>
More information about the Libc-alpha
mailing list