I notice that a little more than an hour after making this request, the fix was integrated, so I presume that you got your code review. In the future, it might be polite in such a case to cancel the request for a code review.
Mark J. Nelson wrote: > Howdy-- > > Requesting review for > >> 6735422 RFE: add template to cdm so that `hg outgoing -v` will list >> files one-per-line > > Webrev location: http://cr.opensolaris.org/~mjnelson/webrev.hgstyle/ > > Notes: > > - I took the suggestion on the RPE alias to add a blank line before > "description," but only for the verbose format. Could also do it for > debug, I suppose, if somebody cares. > > - Otherwise, this is identical to ~jbeck/.hgstyle > > - This will make opt/onbld/etc/hgstyle part of SUNWonbld, and will cause > it to be setup in [ui] by hgsetup(1) > > - This wad also fixes an existing bug in hgsetup.sh, wherein we don't > look in the right directory when using `whence $0` to find tools based > on location of hgsetup executable > > - No additional ROOTDIRS necessary in usr/src/tools/Makefile, because > "etc" was already created by that target > > --Mark -- blu There are two rules in life: Rule 1- Don't tell people everything you know ---------------------------------------------------------------------- Brian Utterback - Solaris RPE, Sun Microsystems, Inc. Ph:877-259-7345, Em:brian.utterback-at-ess-you-enn-dot-kom
