cvs commit: src/usr.sbin/pkg_install/lib deps.c
    Maxim Sobolev 
    sobomax at FreeBSD.org
       
    Wed Jun 20 08:47:17 UTC 2007
    
    
  
Kris Kennaway wrote:
> So the answer to my question is ... ?  I'm still not sure you
> understood it.  If you are making no functional changes then the md5
> of the resulting binary will be unchanged, and this is in fact a good
> way to check whether you are introducing functional changes or just
> style/whitespace changes.
Well, perhaps I did not make myself entirely clear. There were in fact 
some small functional changes (such as replacing fprintf(stderr...) with 
warnx() for consistency with the rest if the code and using abort(3) 
instead of exit(3) if sh.t happens) so that the checksum will not be the 
same. If you take a look at the patch it should be quite obvious what I 
am talking about.
-Maxim
    
    
More information about the cvs-src
mailing list