Quoting Jon Stevens <[EMAIL PROTECTED]>:
> on 7/3/01 5:35 PM, "Jason van Zyl" <[EMAIL PROTECTED]> wrote:
>
> > All I was saying is that people making changes to help improve
> > our 'lot of turbine land' often don't have as much time to kill as I
> do :-)
> > Some mistakes will definitely be made. We'll figure it out.
> >
>
> Right, but in this case, changes were made to the build system after I had
> made the changes to fix it. Those changes totally ignored the changes that I
> had made and therefore broke the Scarab sandbox again. This is simply because
care was not taken to understand the changes that I had made in
> order to allow Scarab to call the Turbine build*.xml files from a different
> directory structure...
i did it!
i did not ignore anything ... but i didn't know that other projects depend on
our build system ...
what will happen if 20 other project copy scarabs build system?
do i have to test 20 apps after each change???
> All I'm asking for is that people take into consideration that changes they
> make affect other projects and that before you make a change you should have
> an understanding of what you are changing and why. Not only does this help
> reduce bugs, but it also makes for cleaner code.
i see your point and nobody wants to break anything, but if you decide to use
the sources from cvs instead of binaries it's primary your problem.
that's why most projects (e.g. jetspeed) use release builds.
our releases must be backward compatible .. but our internal build system?
> This is an attitude that I see Geir take with Velocity and I encourage
> everyone else to adopt it. He is very thorough about thinking about what he
> is doing and the side effects of the change before he commits code.
>
> I personally aspire to adopt that development attitude and kindly request
> that others do so as well. It is in all of our benefit to do so.
i'm sure every commiter is thinking about what he is doing ... but nobody is
perfect!
martin
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
