Reviewed offline with Padraig. LGTM.
JR
On 22/04/2010 11:36, Padraig O'Briain wrote:
I have respun it again so that it applies to the gate,
http://cr.opensolaris.org/~padraig/ips-3139-v4/
Padraig
On 04/22/10 11:00, Padraig O'Briain wrote:
I have respun the webrev,
http://cr.opensolaris.org/~padraig/ips-3139-v3/, to fix the issue you
identified.
Padraig
On 04/21/10 15:56, John Rice wrote:
Padraig - tested this and was able to install a package on my own
image, but the Install success dialog has a Close button on it
that's disabled. You would need to test the other remove, refresh
and Publisher functionality to make sure other dialogs are being
enabled appropriately.
JR
Testing setup I used:
Run:
$ cd /var/tmp
$ pkg image-create -a local=http://localhost:10011 testrepo
$ packagemanager -R /var/tmp/testrepo
Then in PM added pkg.opensolaris.org and installed a theme package
On 04/20/10 10:12 AM, Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-3139-v2/, fixes
3139 PM when user has permissions to access the image allow
install/removal of packages
This change enables BE management dialog if user has admin
privilege and disables IPS image related operations if original
refresh call fails with PermissionException.
Padraig
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
