develooper Front page | perl.perl5.porters | Postings from June 2008

Re: [patch@34044] Enable getgrgid on VMS

Thread Previous | Thread Next
From:
John E. Malmberg
Date:
June 13, 2008 21:31
Subject:
Re: [patch@34044] Enable getgrgid on VMS
Message ID:
485349A5.7020109@qsl.net
Craig A. Berry wrote:
> On Thu, Jun 12, 2008 at 7:50 PM, John E. Malmberg <wb8tyw@qsl.net> 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
wb8tyw@qsl.net
Personal Opinion Only

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About