It looks fine to me as well.
Brock Pytlik wrote:
Anyone else want to take a look at this? If not, I'll put it back tomorrow.
Brock
[email protected] wrote:
On Tue, Jun 02, 2009 at 05:14:10PM -0700, Brock Pytlik wrote:
New webrev at http://cr.opensolaris.org/~bpytlik/ips-7314-v3/ . It's
just a resync to the current gate.
I'd like to get this back in soon as it reduces the time the test
suite takes substantially, I've got another speed improvement that
builds of this change, and it's the kind of patch that lends itself
to poor merges when other tests/changes are made in the relevant files.
For clarity, I've added another bug that this fixes, but no code
changes were made:
9288 search tests shouldn't send packages to persistent depos if
they're already there
http://defect.opensolaris.org/bz/show_bug.cgi?id=9288
Bug 7314 has all the details of the speed improvements that were seen.
I looked at this, but somehow forgot to reply. The changes look fine
AFACIT.
-j
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
