Re: lib/sh/mktime.c VMS specific code is not needed.

[John E. Malmberg]

On 6/4/2012 8:05 AM, Chet Ramey wrote:
> On 6/2/12 7:51 PM, John Malmberg wrote:
> > The lib/sh/mktime.c module has a VMS specific include of<sys/types.h>  to
> > pick up time_t.
>
> No, it doesn't.  VMS systems include<stddef.h>.

I see I read it wrong.

> > On VMS, the time_t type is defined in the<time.h>  module.
> >
> > So this VMS specific include can be removed.
>
> So there is nothing in<stddef.h>  that VMS requires for<time.h>?  That is
> the reason for that code block.

That is correct.  On VMS, <time.h> will pull in everything that it needs.

I also posted on May 12th, that VMS specific code in lib/glob/ndir.h was 
obsolete and would not work on current versions of VMS.  That header 
file is also not used when building bash on VMS, so it can just be 
removed.  I did not see any responses to that.

Regards,
-John