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

Reply via email to