On Sun, Dec 06, 2009 at 08:36:47PM -0600, Shawn Walker wrote:
> Greetings,
>
> The following webrev contains fixes for the following issues:
>
> 13110 image catalog rebuild could be faster
> 12991 pkg image-create can trigger excessive number of stat calls
>
> webrev:
> http://cr.opensolaris.org/~swalker/pkg-13110/
Just a few nits:
catalog.py:
- lines 589 and 590: You've changed line 590 to check for None
instead of True/False; however, line 589 hasn't been changed. If
pfmri is None it cannot have a pfmri.publisher. Is there ever a
situation where pfmri can be false but have a publisher? If not,
should the check on 589 be 'if not (pfmri or pub)' instead?
- Is there something special about how we tell if a pfmri exists?
There are a bunch of places where 'if pfmri' has been changed to 'if
pfmri is not None' and 'if not pfmri.publisher' was changed to 'if
pfmri.publisher is None.' This seems counterintuitive, which is why
I'm curious about this part of the change.
Thanks,
-j
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
