[issue1747858] chown broken on 64bit

2009-12-23 Thread Gregory P. Smith
Gregory P. Smith g...@krypto.org added the comment: indeed, those were missed. fixed in trunk r77007 and release26-maint r77008. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1747858

[issue1747858] chown broken on 64bit

2009-12-02 Thread Antoine Pitrou
Antoine Pitrou pit...@free.fr added the comment: Looking at posixmodule.c, perhaps other instances of parsing an uid_t or a gid_t should have been fixed too (lchown, fchown for example)? -- nosy: +pitrou ___ Python tracker rep...@bugs.python.org

[issue1747858] chown broken on 64bit

2008-03-18 Thread Gregory P. Smith
Gregory P. Smith [EMAIL PROTECTED] added the comment: fixed in trunk r61540. I'm leaving this open until i backport it to release25-maint. -- resolution: - remind versions: -Python 2.6 _ Tracker [EMAIL PROTECTED]

[issue1747858] chown broken on 64bit

2008-03-18 Thread Gregory P. Smith
Gregory P. Smith [EMAIL PROTECTED] added the comment: backported to 2.5 in r61542 and r61544. it'll go into py3k via the regular merges from trunk. The fix just changed the int - long and 'ii' - 'll' and added unit test coverage. The patch attached to this bug was rejected as too complex: If

[issue1747858] chown broken on 64bit

2008-03-17 Thread Gregory P. Smith
Gregory P. Smith [EMAIL PROTECTED] added the comment: i'll take a look at this during the sprint. -- assignee: - gregory.p.smith nosy: +gregory.p.smith _ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue1747858

[issue1747858] chown broken on 64bit

2008-01-16 Thread Sean Reifschneider
Sean Reifschneider added the comment: I've reviewed the chown system call under Linux and I think the included patch will resolve the problem. With that patch on a Fedora 8 64-bit system I'm getting: os.stat('/tmp/foo') posix.stat_result(st_mode=33188, st_ino=5111823, st_dev=64769L,

[issue1747858] chown broken on 64bit

2008-01-16 Thread Andrew Ferguson
Andrew Ferguson added the comment: The idea of dynamic typing it seems quite heavy to me, but I'm not a Python hacker, so I don't know what's the norm. Notice that os.stat() does PyInt_FromLong((long)st-st_uid) on the stat structure's st_uid field. On my platform (OS X 10.4), st_uid is defined

[issue1747858] chown broken on 64bit

2008-01-15 Thread Sean Reifschneider
Sean Reifschneider added the comment: It seems like the problem here is that chown is defined to take -1 for the arguments, where the underlying platform may define chown taking unsigned uid/gid. I think in the short term we will need to apply a patch like one of the ones included here, but in

[issue1747858] chown broken on 64bit

2008-01-15 Thread Sean Reifschneider
Changes by Sean Reifschneider: _ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue1747858 _ ___ Python-bugs-list mailing list Unsubscribe:

[issue1747858] chown broken on 64bit

2008-01-15 Thread Martin v. Löwis
Martin v. Löwis added the comment: I believe that patch would break on a system where uid_t is a 64-bit value, yet unsigned int is 32 bits. _ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue1747858 _