Davanum Srinivas wrote:
IMHO, You can start with a gentle push and if you don't get "enough attention" then do a -1. IMHO, No one can expect a total control of changes to a class that was just checked in a few hours ago. That's preposterous. Unless *ALL* details were discussed prior on the mailing list and the checkin was just a formality. IMHO, the situation was not serious enough to warrant a -1.
In that case, I agree fully. In any case, Deepal apologized for it and I thought we're past that story but I guess not!
Yes, the process is if there is a -1 then the code needs to be reverted to the previous version as soon as possible, unless there is build break in our test harness.
There were several -1s against a bunch of changes involved with folks removing existing functionality because "they didn't see it being used". Sorry but that's -1 material .. if someone just removes some features because they can't see it being used then that's a problem and the onus is on that person to *ask*. If they can't be bothered to ask then -1 is a way of telling "this is not accepted behavior" too.
Sorry, IMHO, spate of -1's that i see means a "My way or the Highway" approach is taking root and i personally don't like it.
This is too strong Dims- everyone needs to realize that commits cannot remove existing functionality once we've published it in a release. Example: Glen just removed another feature :) .. see Thilina's mail a bit ago. Once we release some API we CANNOT remove it in the next version .. best we can do is deprecate. We can remove in a future release!
Sanjiva. -- Sanjiva Weerawarana, Ph.D. Founder & Director; Lanka Software Foundation; http://www.opensource.lk/ Founder, Chairman & CEO; WSO2, Inc.; http://www.wso2.com/ Director; Open Source Initiative; http://www.opensource.org/ Member; Apache Software Foundation; http://www.apache.org/ Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/ --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
