dmick at angus> hg help nits
hg nits [-p PARENT]

check for stylistic nits in active files

     Run cddlchk, copyright, cstyle, hdrchk, jstyle, permchk, and
     keywords checks.

options:

  -p --parent  parent workspace

use "hg -v help nits" to show global options


dmick at angus> hg help pbchk
hg pbchk [-N] [-p PARENT]

pre-putback check all active files

     Run cddlchk, comchk, copyright, cstyle, hdrchk, jstyle, permchk, tagchk,
     branchchk, keywords and rtichk checks.  Additionally, warn about
     uncommitted changes.

options:

  -p --parent   parent workspace
  -N --nocheck  skip RTI check

use "hg -v help pbchk" to show global options

Stephen Ostrowski wrote:
> I was told to do hg nits, how is that different then hg pbchk ?
> 
> 
> 
> James Carlson wrote:
>> Cyril Plisko writes:
>>   
>>> I was under impression that as a part of full nightly run there is
>>> cstyle cleanness check being run as well. However, yesterday I noticed
>>> that while one of my files wasn't cstyle clean the nightly mail msg
>>> says:
>>>     
>> [...]
>>   
>>> So, was I under false impression ?
>>>     
>>
>> Yes.  That section actually shows the output of "dmake check" in the
>> $SRC directory.  Lower-level makefiles that have some provision for
>> the "check" target will then perform whatever checks are coded for.
>>
>> In general, there are checks for files delivered as source (headers
>> and manifests), but not otherwise.
>>
>>   
>>> Is there any automatic mean to help
>>> keep sources cstyled ? (I mean other than human running cstyle
>>> manually).
>>>     
>>
>> The current means is to demand that users do "hg pbchk" before putting
>> back, and to ask them (as part of the RTI process) whether they've
>> done so.  The gatekeepers also have scripts that check each putback --
>> including doing incremental builds and other checks -- and will send
>> "is this you?" messages to apparent offenders.
>>
>> It's not perfect, but running cstyle on all of the code during every
>> nightly run would be really wasteful and would false on a huge number
>> of files.  (There's a lot of third-party code that's not clean and
>> never will be.)
>>
>>   
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> on-discuss mailing list
> on-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/on-discuss


Reply via email to