test_source harmful?

Ben Finney bignose+hates-spam at benfinney.id.au
Fri Feb 27 03:26:50 GMT 2009


Martin Pool <mbp at sourcefrog.net> writes:

> I'd tend to agree with Robert that a pqm rejection, if people ever
> hit it, is far too strong medicine.
> 
> I prefer not to have it, as it does make diffs noisier if it's added
> or removed incidentally, but a rejection is a big whack.

Okay. Is there any other means, though, for enforcing a policy of
“don't do this in committed code” if a team wants that policy
enforced on their repository?

> Probably what we need is an --ignore-trailing-whitespace option to
> diff, and also to annotate. It would be nice if they could be turned
> on by rule preferences too.

For reference, GNU diff has these options:

       -b  --ignore-space-change
              Ignore changes in the amount of white space.

       -w  --ignore-all-space
              Ignore all white space.

       -B  --ignore-blank-lines
              Ignore changes whose lines are all blank.

-- 
 \       “I believe in making the world safe for our children, but not |
  `\    our children's children, because I don't think children should |
_o__)                                     be having sex.” —Jack Handey |
Ben Finney




More information about the bazaar mailing list