Re: cvs: php4(php_4_0_4) / configure.in /main php_version.h

On Sun, Dec 03, 2000 at 11:48:25PM +0200, Zeev Suraski wrote: > At 23:45 3/12/2000, Sterling Hughes wrote: > >I think that's why Sascha wanted the whitespace changes separated from the > >"actual" changes, so we could inspect the "actual" changes more > >carefully > >before they become a part of the release branch. It saves us from wading > >through the commit to see which changes are whitespace related and which > >changes are actually signifigant if the whitespace changes are removed. > > Hmm, well, I think that it's fairly easy to ignore lines that are identical > except for added spaces. i disagree. i bet that i can could you a ~100 line diff with mostly whitespace changes and even you will overlook something important. if it's not too late i think we should put it in CVS-RULES that whitespace-only changes should be committed seperately. (and marked as such) tc > > Zeev > > > -- > Zeev Suraski <zeev@zend.com> > CTO, Zend Technologies Ltd. http://www.zend.com/ > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, e-mail: php-cvs-unsubscribe@lists.php.net > For additional commands, e-mail: php-cvs-help@lists.php.net > To contact the list administrators, e-mail: php-list-admin@lists.php.net > >