Re: CVS files merging procedure

[Jim Hyslop]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

abhay mehta wrote:
> Sir,
(side note: "Sir" is an old-fashioned salutation, which is generally
frowned upon these days because it excludes women - just thought you
should know.)

> Based on feedback from Jim, Russ Sherk and others, we reviewed our
> SCM  plan. However based on our requirements, we need an
> Administrator/  Reviewer  to review the code before it goes to
> repository.
> 
> This is very much required as we have number of programmers, who
> don't  know the whole system and so the implications/ side-effects of
> their  changes in other part of the system.

You still need to modify your engineering practises. This is not an SCM
problem, it is a software engineering problem.

No one person can ever completely understand a system. Furthermore, what
will you do when this person leaves the company? You're screwed.

This is why you write unit tests. When programmers make their changes,
they run the tests. If the tests pass, they check in their code. If the
tests fail, the programmers go back and fix up what they just broke.

If the tests pass, but the programmer has still introduced a problem,
then your first task is to fix the tests so they are more comprehensive.

- --
Jim Hyslop
Dreampossible: Better software. Simply.     http://www.dreampossible.ca
                 Consulting * Mentoring * Training in
    C/C++ * OOD * SW Development & Practices * Version Management
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (MingW32)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFDoD2KLdDyDwyJw+MRAqycAJ9Si3PysLsR5GWZszm5GOz5zXpUnQCg8iCT
qVgM4aTl3kaHb5NIcdcpU+g=
=M3dw
-----END PGP SIGNATURE-----