Message119545
| Author |
lars.gustaebel |
| Recipients |
lars.gustaebel, loewis |
| Date |
2010年10月25日.11:05:36 |
| SpamBayes Score |
1.7001668e-05 |
| Marked as misclassified |
No |
| Message-id |
<1288004738.16.0.171824581032.issue10184@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I'm not entirely happy with the name of the "touch" argument. Apart from it being nice and short, I think it's a little too unix-y and might be misleading because it is not only about setting the modification time as the name implies, but also owner and mode. My proposal would be "restore_attrs" or "set_attrs" which isn't half as nice as "touch", but sums up better what's actually done. I leave this up to you.
I think the testcase wouldn't work on Windows the way it is now, would it?
Apart from these minor issues the patch gets my blessing, go ahead ;-) |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2010年10月25日 11:05:38 | lars.gustaebel | set | recipients:
+ lars.gustaebel, loewis |
| 2010年10月25日 11:05:38 | lars.gustaebel | set | messageid: <1288004738.16.0.171824581032.issue10184@psf.upfronthosting.co.za> |
| 2010年10月25日 11:05:36 | lars.gustaebel | link | issue10184 messages |
| 2010年10月25日 11:05:36 | lars.gustaebel | create |
|