[Patches] [Patch #101482] fix for Bug #114245

noreply@sourceforge.net noreply@sourceforge.net
Sun, 17 Sep 2000 18:35:32 -0700


Patch #101482 has been updated. 

Project: 
Category: core (C code)
Status: Postponed
Summary: fix for Bug #114245 

Follow-Ups:

Date: 2000-Sep-16 16:18
By: tim_one

Comment:
Mark, does this look reasonable to you?  The thrust is to make utime work for directories under Win32 (as well as for files).  THere's also an entry for this one in PEP 42 (Feature Requests).

-------------------------------------------------------

Date: 2000-Sep-16 16:31
By: mhammond

Comment:
With a quick look and no decent analysis, I can't see a good reason why this wouldnt work.  However, see http://support.microsoft.com/support/kb/articles/q167/2/96.asp for the official way of doing this - it is significantly different.

I have the above KB code all Python-ready, as it was used in the Windows CE port.  I am happy to send this to anyone and everyone, but I am afraid I don't have time to rework the patch.

Also note that I would _love_ to see whatever conversion routine you add made public in the DLL (and hence named appropriately).  Windows CE could use it, as could the Python Win32 extensions - keep the bloat size down by keeping it in Python, and the cost of making it public isnt too bad!

-------------------------------------------------------

Date: 2000-Sep-16 19:40
By: none

Comment:
The time conversion function time2Time is modeled after MS C run time library code. It looks official way of doing this also ;). However UnixTimeToFileTime looks simpler. But I do not know if these two functions do the same thing.
-------------------------------------------------------

Date: 2000-Sep-17 18:35
By: tim_one

Comment:
I changed this to Postponed:  the scope is growing, and the bug this is related to was already changed to a Feature Request and added to PEP 42.  Let's look at it again after 2.0final.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://sourceforge.net/patch/?func=detailpatch&patch_id=101482&group_id=5470