[Yum-devel] [PATCH] fix --cacheonly edge case. BZ 975619

Zdenek Pavlas zpavlas at redhat.com
Wed Jun 19 14:23:06 UTC 2013


>  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.


More information about the Yum-devel mailing list