[Opensim-dev] Please do not revert fixes without careful comtemplation

[Dahlia Trimble]

Is there a specific commit or mantis which this is addressing? I'm not aware
of which events your message applies to, but I can say in general that I've
only seen a code revert when the original changes  cripple functionality for
other users or imposes a needless barrier on further development, and then
the revert only happens after consultation with others had occurred.
This is also a fast moving project with many modifications made to the
source tree from many authors. Usually if a revert is required it needs to
happen quickly otherwise it becomes increasingly difficult to restore the
code back to a condition where it would be had the changes not been applied.
If the author of the changes had not been available at the time of the
decision to revert then notes are added to the mantis containing the patches
explaining the situation to the author.

On Fri, Feb 13, 2009 at 8:12 AM, Tommi Laukkanen <
tommi.s.e.laukkanen at gmail.com> wrote:

> Dear all
> If you decide to revert fixes the least you can do is to look into them in
> detail and run the unit tests to make sure that the revert does not make
> situation worse.
>
> This is also part of mutual respect between developers. Some of us are new
> in the project. Some of us are inexperienced. But all of us have intellect
> and feelings. Optimally if you find a problem which does not cause problems
> in the production it is best that the person in blame is given advice and
> gets to fix the issue himself.
>
> This is a large project and we should appreciate everyone who contributes.
> Motivation is a fragile thing and we should give our peers all the courtesy
> we can. Let us make the community more enjoyable to everyone.
>
> sincerely,
> Tommi
>
> _______________________________________________
> Opensim-dev mailing list
> Opensim-dev at lists.berlios.de
> https://lists.berlios.de/mailman/listinfo/opensim-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://opensimulator.org/pipermail/opensim-dev/attachments/20090213/6a79698e/attachment-0001.html>