On 01/22/10 09:59, Shawn Walker wrote:
line 860: so this else case handles both 'optional' and exclude deps? I'm probably missing something here.
optional and exclude dependencies are just mirror images of each other: optional @2.0 means that if present, fmri version must be at 2.0 or greater. exclude @2.0 means that if present, fmri version must be less than 2.0
modules/client/image.py: This is not part of your changeset; but is related: line 540 (in get_publisher_ranks()): The comment seems to indicate that this function should include disabled publishers. Is that true, or do you really only care about enabled publishers and publishers of installed packages? If it needs to include disabled publishers always, then line 545 needs inc_disabled=True passed to self.gen_publishers().
Actually, it doesn't matter; we always add publishers that sourced any installed pkgs; we now also handle random publishers in addition. - Bart -- Bart Smaalders Solaris Kernel Performance [email protected] http://blogs.sun.com/barts "You will contribute more with mercurial than with thunderbird." _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
