On Tue, Oct 06, 2009 at 10:41:30AM -0700, Danek Duvall wrote:
> Shawn Walker wrote:
> 
> > 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)
> 
> Three places in pkg.manifest, too.

Yes, it would be good to use consistent style.  I think I used the stat
contsants when I did some catalog permissions changes long ago.  When I
was making the manifest changes last week, I saw 644 used in the actions
code and used that instead.  Having this defined and used consistently
would certainly help avoid confusion.

-j
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to