Danek Duvall wrote:
Shawn Walker wrote:

Actually, there are, I had forgotten about that when Rich asked me to
peek at this before.  This should give mode 644:

stat.S_IRUSR|stat.S_IWUSR|stat.S_IRGRP|stat.S_IROTH

Which is a long incantation.  We have a half dozen instances of 644 (and
for 755) in the code -- perhaps we should define a constant in pkg.misc?

Only two other places are using the raw mode for 644:

actions/license.py:80:                        os.chmod(path, 0644)
actions/legacy.py:111:                os.chmod(pkginfo, 0644)

But lots more are using 755, so that seems reasonable ...

Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to