James Carlson <james.d.carlson at sun.com> writes:

> These are some easy bits of clean-up for these bugs:
>
>   441 tools/Makefile.tools INS.pyfile should be in Makefile.master
>   447 wx has errant references to onnv.eng
>
> The webrev is here:
>
>   http://cr.opensolaris.org/~carlsonj/webrev-441-447/
>
> I'll test by building the tools alone, and making sure that the
> resulting cdm and wx work, and checking the wx man page.

My only comment is a query.  Is there a reason not to just link
directly to the page in the second set of changes to wx.1?  I know
we'd be repeating the same information, but it seems nicer than making
people dig the link out (I imagine http://onnv.eng is less link-heavy
than the base ON community page).

Other than that (which I don't care that much about), it looks fine to me.

-- Rich

Reply via email to