On Tue, Nov 17, 2009 at 03:09:00AM -0600, Shawn Walker wrote:
> [email protected] wrote:
> >Folks,
> >This fixes a permissions error that prevented un-privileged users from
> >performing operations like pkg install -nv.
> >
> >The fix is in image.py.  I also tried to further tidy up the calls to
> >img.cleanup_downloads().  It didn't appear to be getting called in all
> >of the paths that could create a temporary download directory.
> >
> >     http://cr.opensolaris.org/~johansen/webrev-12698/
> 
> This seems fine except I think add_publisher() in client/api.py
> should be changed to use a try/finally construct to ensure
> cleanup_downloads is called.

Thanks, I'll change this.

> I'm assuming that these changes allow 'info -r', 'contents -r' or
> 'list -s' (v0 repository) to work again for an unprivileged user
> when the manifest doesnt yet exist on-disk.

These work fine for me now.

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

Reply via email to