From: SourceForge.net <no...@so...> - 2007-12-10 00:22:33
|
Patches item #1844733, was opened at 2007-12-05 05:25 Message generated for change (Comment added) made by cgroves You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1844733&group_id=12867 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Closed >Resolution: Rejected Priority: 5 Private: No Submitted By: Mehendran (mehendran) Assigned to: Nobody/Anonymous (nobody) Summary: Patch for [ 1841639 ] Filecmp module is not in jython Initial Comment: As jython implementation of os.stat call is limited in returning the same values as with CPython, many of file properties are assigned to default which is zero including InodeProtectionMode. In CPython, InodeProtectionMode is to find out the type of the file given. so, i have modified the filecmp module with the support of java.io.File. Now all the test cases are working properly. The patch is attached herewith. please give your valuable comments and suggestions. ---------------------------------------------------------------------- >Comment By: Charles Groves (cgroves) Date: 2007-12-09 19:22 Message: Logged In: YES user_id=1174327 Originator: NO This patch adds a forked version of filecmp that will no longer work in CPython. Rather than committing it, I upgraded os.stat to fill in st_mode with the bits needed for filecmp that you're figuring out with the patch. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1844733&group_id=12867 |