[EMAIL PROTECTED] wrote:
> Folks,
> 
> Brock found this bug while we were discussing the fate of another bug
> regarding error handling.  This fixes a case where filelist is throwing
> a RuntimeError where it really shouldn't.
> 
> Webrev for the fix is here:
> 
>       http://cr.opensolaris.org/~johansen/webrev-4381/

Instead of raising a FileListRetrievalError, I'd rather throw an 
exception specific to the problem.  Maybe a StorageException instead or 
  a StoragePermissionsException?

Of course, that depends on whether our exceptions are intended to be 
specific to the operation or to the error...

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

Reply via email to