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