> What about
> 
>     if refresh_catalog or purge_catalog:
>         destroy
>         destroy
> 
>         if refresh_catalog and refresh_allowed:
>             retrieve
> 
>         cache

Okay, one further question about tidiness, since Shawn brought up the
case where an authority has its origin changed and is simultaneously
disabled.

Do you prefer:

                if disabled:
                        purge_catalog = True
                        refresh_catalog = False
                else:
                        refresh_catalog = True

        if refresh_catalog or purge_catalog:
                destroy
                destroy

                if refresh_catalog and refresh_allowed:
                        retrieve
                else:
                        cache

or:

                if disabled:
                        purge_catalog = True
                else:
                        refresh_catalog = True

        if refresh_catalog or purge_catalog:
                destroy
                destroy

                if purge_catalog:
                        cache
                elif refresh_allowed:
                        retrieve


Thanks,

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

Reply via email to