This looks good to me.

Padraig

On 12/16/09 14:23, Michal Pryc wrote:
Hello,
The webrev:
 http://cr.opensolaris.org/~migi/13309_13326_permissions/

Fixes two issues:
  http://defect.opensolaris.org/bz/show_bug.cgi?id=13309
Change priority of publishers without sufficient privileges got traceback

The list of priorities is a simple array, of three things and we were using enumerations from the main publisher list, which is wrong.

  http://defect.opensolaris.org/bz/show_bug.cgi?id=13326
  Package Manager fails to start if not run as root

If the user doesn't have permissions to the log file, we will send the error and info messages to the standard sys.stdout and sys.stderr, please note that I have moved the check for *log_exists to the try/except as the StreamHandler doesn't have the doRollover() method.

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

Reply via email to