>  I don't understand. misc.unlink_f() is:
> 
>     try:
>         os.unlink(filename)
>     except OSError, e:
>         if e.errno != errno.ENOENT:
>             raise
> 
> ...so which other OSError's are we protecting ourself from here

EPERM, this is completion-helper using root cachedir.

>  Also would be better to have an all_oserrors=True (or maybe just
> all_errors=True?) flag to unlink_f.

I'd add misc.unlink_ff() instead, but hey, 2 lines IMO
don't justify a new function.
_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to