Date:	Thu, 09 Aug 2001 18:38:45 +0300
From:	"Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>
Sender:	halo1 AT zahav DOT net DOT il
To:	acottrel AT ihug DOT com DOT au
Message-Id:	<7458-Thu09Aug2001183844+0300-eliz@is.elta.co.il>
X-Mailer:	Emacs 20.6 (via feedmail 8.3.emacs20_6 I) and Blat ver 1.8.9
CC:	djgpp-workers AT delorie DOT com, sandmann AT clio DOT rice DOT edu
In-reply-to:	<007701c120c6$2d72db80$0a02a8c0@acceleron> (acottrel@ihug.com.au)
Subject:	Re: Windows 2000 patch for utime.c v1
References:	<Pine DOT SUN DOT 3 DOT 91 DOT 1010808181731 DOT 27271C-100000 AT is> <007701c120c6$2d72db80$0a02a8c0 AT acceleron>
Reply-To:	djgpp-workers AT delorie DOT com
Errors-To:	nobody AT delorie DOT com
X-Mailing-List:	djgpp-workers AT delorie DOT com
X-Unsubscribes-To:	listserv AT delorie DOT com

> From: "Andrew Cottrell" <acottrel AT ihug DOT com DOT au>
> Date: Thu, 9 Aug 2001 21:26:46 +1000
> 
> > > +      r.x.bx = 0x03;    /* Set last write date / time */
> > > +      __dpmi_int(0x21, &r);
> > > +
> > > +      if (!(r.x.flags & 1))  /* Pass exit otherwise try other methods
> */
> > > +      {
> > > +          return 0;
> >
> > I think we should set access date/time as well (using subfunction 05),
> > like we do on Windows 9X.  Please see if subfunction 05 works, and if
> > so, does it support setting the access time, not only date.
> All of the sub functions wrt to time appeared to work when I tested them.

Btw, did they work on directories, i.e. could you change the time
stamp of a directory?

> I have added setting of subfunction 05 in both patches. What about
> the subfunction 07h set last write date/time?

Subfunction 07 changes the create date/time, not last write
date/time.  `utime' lets you change only last write (a.k.a. modify)
and access times, but not the file's creation time.  So we shouldn't
use subfunction 07, only 03 and 05.

> Sorry, but to save time I have attached two version of an updated utime.c
> patch. The patch called utime_v2.dif does not include setting 7143
> subfunction 07. The patch file called utime_v2_full.dif includes setting all
> times included in 7143.

The second patch looks fine to me.

- Raw text -