On 12/21/11 02:13, Saurabh Vyas wrote:
Hi All,

Please review the fix for

18958 - Error message concerning /var/pkg/lock & lack of privileges
could be improved
7123469 error handling in __api_plan_exception can be improved

webrev : https://cr.opensolaris.org/action/browse/pkg/saurabhv/18958/

For this fix, I have created a new exception class & used a simplified
error msg (rather than ' Could not operate on /var/pkg/lock' which is
moreover an implementation detail).

This also needed bumping up the PKG_CLIENT_VERSION (to 71, as suggested
by Shawn)

Please let me know your thoughts / comments.

BTW I ran the test suite & I got 2 errors (which I got on the baseline
too)

======================================================================
BASELINE MISMATCH: The following results didn't match the baseline.
----------------------------------------------------------------------
cli.t_pkg_sysrepo.py TestSysrepo.test_11_https_repos: error
cli.t_https.py TestHTTPS.test_01_basics: error
----------------------------------------------------------------------

I ran the test suite on my own system last night and didn't see a failure.

If you could provide the error output of the failures, we can then determine the cause of them and eliminate any potential issues.

-Shawn

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

Reply via email to