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
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
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]
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
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
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,
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
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
Changes by Sean Reifschneider:
_
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1747858
_
___
Python-bugs-list mailing list
Unsubscribe:
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
_
10 matches
Mail list logo