Danek Duvall wrote:
brand/Makefil:- As far as I can tell, that definition isn't used at all; why not just remove the line? coverage.patch: - Why is this necessary? Without it, the coverage executable gets /usr/bin/python2.6 as its interpreter. ez_setup.py.patch: - Why is this necessary? As far as I can tell, ez_setup.py isn't ever executed -- it's just imported. svc-pkg-depot: - line 78: please use $() instead of ``; that also means you won't need the continuation character (which should have a space before it, anyway).
New webrev at: http://cr.opensolaris.org/~richb/pkg-12966-v2/ Thanks. _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
