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