On 04/21/10 11:13 PM, Richard Lowe wrote:
Richard Lowe wrote:
Shawn Walker wrote:
On 04/19/10 08:43 PM, Richard Lowe wrote:
The change I'm making here preserves the permissions of an existing
cfg_cache and uses 0644 as the default. I'm still on the fence about
attempting to preserve the existing permissions, in most cases I've
found the code treats the depot directory structure as entirely its own,
but I'm not sure if this is intent, and I'm also not sure it'd be the
right thing to do with cfg_cache anyway.
Further, should this attempt to preserve ownership? If so, what should
it do if it cannot?
The cfg_cache file is a bit special since the depot allows this file to live
outside of the repository via the --cfg-file option. In light of that,
preserving the permissions seems fine.
Shawn responded to a question I asked off list (that he respond to my
last paragraph) stating that he thinks we should try and preserve
ownership, but not complain if we fail to.
An updated webrev, which does this, is available from:
http://cr.opensolaris.org/~richlowe/pkg_15686-2
Please review with caution.
Seems right to me.
Cheers,
-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
