Why does bsd.pkg.mk warn you when a patch was unsuccessful, and then *retry*
the patch?  Wouldn't it be saner to fail, so that the resulting files
(filename.orig, filename, filename.rej) reflect a usable set:  the original,
the file with successful patched hunks, and only the failed hunks?

I don't see any point to reapply a patch that partly failed.  After all,
patch(1) will bomb anyway because some of the hunks may already be applied.

-- 
-- Todd Vierling (tv@pobox.com)