On Wed 24 Feb 2010 at 07:28PM, TAKI, Yasushi/JPC wrote:
>
> This is correct.
>
> zone_pkgs=$(LC_ALL=C $PKG list --no-refresh -H "pkg://$zone_publisher/*"
> | \
> awk '!/^pkg:\/\/'${zone_publisher}'\/entire\$/{printf
> "pkg://'${zone_publisher}'/%s ",$1}' )
Taki, and all,
I have spoken at length with Bart about this fix. After much
consideration, we have decided that this code is no longer even
needed, and we are planning to remove it. The new solver takes
care of everything that this code was trying (and failing) to do.
We are concerned that adding pkg://$zone_publisher/ could have
some odd side effects if packages move between publishers. And
in general the risk of this seems higher than we would like for
something which seems to have essentially no effect. As a bonus, this
code is now faster.
Here is a respun webrev which removes this code.
http://cr.opensolaris.org/~dp/pkg-14673/
For those who look at this code, you'll no-doubt realize that
it is missing support for systems where 'entire' is not installed.
Gary is working on that, but it looks unlikely to make the cutoff
for this release.
Thanks,
-dp
--
Daniel Price, Solaris Kernel Engineering http://blogs.sun.com/dp
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
