`_post-patch-finalize' is run for `checkpatch' as well breaking it's
idempotency.

This seems like an oversight in r1.1382 "next stage of PORTS_PRIVSEP".

OK?

Index: bsd.port.mk
===================================================================
RCS file: /cvs/ports/infrastructure/mk/bsd.port.mk,v
retrieving revision 1.1403
diff -u -p -r1.1403 bsd.port.mk
--- bsd.port.mk 21 May 2018 21:26:39 -0000      1.1403
+++ bsd.port.mk 23 May 2018 21:43:27 -0000
@@ -2626,8 +2626,8 @@ ${_PATCH_COOKIE}: ${_EXTRACT_COOKIE}
 .if target(post-patch)
        @${_PMAKE} post-patch
 .endif
-       @${_PMAKE} _post-patch-finalize
 .if ${PATCH_CHECK_ONLY:L} != "yes"
+       @${_PMAKE} _post-patch-finalize
        @${_PMAKE_COOKIE} $@
 .endif
 

Reply via email to