[scm-migration-dev] code review request for usr/src/tools cleanup wad

2009-02-06 Thread Rainer Orth
Mark J. Nelson writes: > > usr/src/tools/lintdump/Makefile > > > > 33-34: already the default in Makefile.master; shouldn't be needed. > > I had imagined that any changes made here were done to silence protocmp, > but I'll ask Rainer. I can't find in my notes why I added this (probably the w

[scm-migration-dev] code review request for usr/src/tools cleanup wad

2009-02-05 Thread Mark J. Nelson
I pushed back a little, and have now completely removed SUNWmrtools as part of the tools protocmp fix. I believe that I have either addressed or incorporated all other suggestions. I'm doing some last minute sanity checking and filling out an RTI, so if anybody else was planning to chime in,

[scm-migration-dev] code review request for usr/src/tools cleanup wad

2009-02-04 Thread James Carlson
Mark J. Nelson writes: > > usr/src/tools/SUNWmrtools/prototype_com > > > > 47-53: what is this? > > A vestigial tail. ;-} > The SUNWmrtools package was used for sideband delivery to RE, when we > changed things they needed for cdkit. I think. > > I asked about removing it, and got mild pus

[scm-migration-dev] code review request for usr/src/tools cleanup wad

2009-02-04 Thread James Carlson
Mark J. Nelson writes: > > http://cr.opensolaris.org/~mjnelson/webrev.tools_cleanup_wad/ > > http://anthrax.central/net/mrliberal.central/export/ws/mjnelson/onnv-bugfixes/webrev.onswan/ usr/src/tools/Makefile.tools 48,69,75-77: these seem to be duplicates. Mismerge? usr/src/tools/SUNWmrtools/

[scm-migration-dev] code review request for usr/src/tools cleanup wad

2009-02-04 Thread Mark J. Nelson
David and Rainer: I embedded questions to each of you below. James Carlson wrote: > Mark J. Nelson writes: >>> http://cr.opensolaris.org/~mjnelson/webrev.tools_cleanup_wad/ >>> http://anthrax.central/net/mrliberal.central/export/ws/mjnelson/onnv-bugfixes/webrev.onswan/ > > usr/src/tools/Makefil

[scm-migration-dev] code review request for usr/src/tools cleanup wad

2009-02-04 Thread Mark J. Nelson
Howdy-- This is a request for review of fixes for the following: > - 6781574 transition to mercurial has brought CR 4800126 back to life > * 6414844 SUNWonbld should use protocmp > 6785939 on-SWAN env files should default to anonymous access for pull > 6777815 Cadmium breaks NetBeans 6.5 Merc