I have updated the webrev http://cr.opensolaris.org/~padraig/ips-5175-v2/ to address Brock's comments.

Padraig

On 12/04/08 20:14, Brock Pytlik wrote:
Lines 147, 158-162, 178, 194, 261, 399-406, 409-413, 416-420, 423-427, 430-434, 437-441, 445, 448, 454, 459, 466, 611-614, 643, 725, 726, and more should not have \'s because they're inside parens. Since this is a formatting fix, it makes sense to fix these issues as well.

Brock


jmr wrote:
Simple formatting change. Looks fine.

JR



The webrev is at http://cr.opensolaris.org/~padraig/ips-5175-1/. 
<http://cr.opensolaris.org/%7Epadraig/ips-5175-1/>

This is a fix for http://defect.opensolaris.org/bz/show_bug.cgi?id=5175
Wrong spacing in the updatemanager.py

The only changes are to indentation.

Padraig

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

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

Reply via email to