Rich Burridge wrote:

Hi all,

Looking for a code review of the proposed changes for bug #8132:

 http://defect.opensolaris.org/bz/show_bug.cgi?id=8132
 __licenses should use StringIO

Webrev is at: http://cr.opensolaris.org/~richb/pkg-8132-v1/

I note that the license information appears more than once,
but it did that before I made my change too.

I'd use cStringIO instead, but other than that, looks fine.

Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to