On 08/23/11 18:33, Shawn Walker wrote:
Greetings,
The following webrev contains fixes for the following issue:
17252 pkg.client.api should provide history management interface
webrev:
https://cr.opensolaris.org/action/browse/pkg/swalker/pkg-history-1/webrev/
-Shawn
_______________________________________________
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
client.py:
is there any reason that img (lines 5754, 5761, 5939), continues to need
to be global? or even to exist at all?
5545: does defining gen_entries simply let you avoid indenting 5554-5646
(roughly)? If so, that's fine, but it might be worth a comment, and if
it serves another purpose, can you call that out please?
api.py:
gen_history could use a docstring
1880: the number "20" here seems a bit magical, perhaps the comment on
line 1886 could be moved up?
Brock
_______________________________________________
pkg-discuss mailing list
pkg-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
