Craig A. Berry wrote:
On Thu, Jun 12, 2008 at 7:50 PM, John E. Malmberg <[EMAIL PROTECTED]> wrote:
Even though configure.com was detecting the getgrgid and friends, they were
not being configured because HAS_GROUPS was not defined in vmsish.h.
Good catch. Obviously it's HAS_GROUP, not HAS_GROUPS, but that's
correct in the patch itself.
I am evaluating what is left to be done for supporting ODS-5 pathnames
by looking to see what is currently being skipped or failing when I turn
on the DECC features.
--- /rsync_root/perl/vms/vmsish.h Thu Feb 21 18:20:57 2008
+++ vms/vmsish.h Thu Jun 12 08:34:47 2008
@@ -448,8 +448,15 @@
* This symbol, if defined, indicates that the getgrnam() and
* getgrgid() routines are available to get group entries.
* The getgrent() has a separate definition, HAS_GETGRENT.
+ *
+ * These have been available in VMS for a while, but configure.com
+ * only checks for the recursive versions.
The _r is for reentrant, not recursive.
Opps, I do not know why I wrote that.
*/
+#if defined HAS_GETGRNAM_R || defined HAS_GETGRGID_R
Wouldn't it be simpler to just do:
#if __CRTL_VER >= 70302000
plus that self-documents when they became available.
The reason I used the symbols is that I was not sure if Configure
exposed them at the same as they showed up in VMS.
-John
[EMAIL PROTECTED]
Personal Opinion Only