[Kolab-devel] CVS commit messages and testing.

Bernhard Reiter bernhard at intevation.de
Thu Jan 5 11:18:39 CET 2006


Hi Richard,

I just stumbled about a comment in your CVS commit and noticed:

a) CVS comment are not the right place to add comments that might 
change over time or contain questions, general hints or things todo.
That you have not tested it well, is not a eternal property of the changes
between the two files. :)

b) I would of course be really cool, if you could just keep an OpenPKG based
system around to test your changes at least a little bit. We will do some 
testing at some time, but we might notice bugs quite a bit later then.
Currently I guess the OpenPKG is still the main system for Kolab Servers
and we need to keep it in shape or loose important users.

Thanks again for your and Marcus' great work!
This email is purely meant contstructive to even improve. 
Bernhard 

> Author: richard

> Log Message:
> * configure.ac: Added AC_SUBST(WITHOPENPKG)
> * dist_conf/common: Added CHKCONFIG, WITHOPENPKG and backupfiles
> * dist_conf/suse: Added assignment for backupfiles
> * kolab_bootstrap.in: Added one time backup support for configuration
>   files that are provided by the distributor
>   Removed a redundant @ldapserver_dir@ statement
>   Changed removal of *.pem files and the @ldapserver_dir@/* files
>   Check the existance of files and directories, before moving
>   Change @sysconfdir@/rc.conf for openpkg based distributions
>
>
> It works very well on my (non openpkg) system.  Please test it on a
> openpkg system carefully too!




More information about the devel mailing list