> > This might be an inane question, but why bother with __set_perms() at > > all? Can you just chmod the tmpfile to 644 before you rename it into > > the catalog directory? > > It was done to centralise the permissions logic in one place since there > are multiple places that catalog files are created and used.
Okay, but it looks like you haven't finished the centralization. The code in recv() dorks with the permissions on the client side, but it looks like that code is unchanged. Did you mean that you're centralizing the depot portions of the permissions management? -j _______________________________________________ pkg-discuss mailing list pkg-discuss@opensolaris.org http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
