> > 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 > > The latter, I guess, though I don't understand how the if clause differs > between the two examples, assuming that both purge_catalog and > refresh_catalog start out False.
Great. I got impatient and implemented the latter before you responded. :P The case that I'm specifically describing occurs when both start out False, but that refresh_catalog gets set to True before the if disabled case. Shawn observed that this could happen in a situation where we simultaneously change an authority's origin and mark it as disabled. -j _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
