On Mon, May 21, 2018 at 03:03:28PM +0200, Klemens Nanni wrote:
> Index: bsd.port.mk
> ===================================================================
> RCS file: /cvs/ports/infrastructure/mk/bsd.port.mk,v
> retrieving revision 1.1402
> diff -u -p -r1.1402 bsd.port.mk
> --- bsd.port.mk 17 May 2018 09:59:36 -0000 1.1402
> +++ bsd.port.mk 21 May 2018 12:29:32 -0000
> @@ -3406,7 +3406,7 @@ repackage:
> @${_MAKE} package
>
> rebuild:
> - @rm -f ${_BUILD_COOKIE}
> + @${_PBUILD} rm -f ${_BUILD_COOKIE}
> @${_MAKE} build
>
> uninstall deinstall:
sthen noticed rebuild, he's going to commit it
> @@ -3415,10 +3415,10 @@ uninstall deinstall:
>
> peek-ftp:
> @echo "DISTFILES=${DISTFILES}"
> - @install -d ${DISTDIR_MODE} ${FULLDISTDIR}; \
> + @${_PFETCH} install -d ${DISTDIR_MODE} ${FULLDISTDIR}; \
> cd ${FULLDISTDIR}; echo "cd ${FULLDISTDIR}"; \
> for i in ${MASTER_SITES:Mftp*}; do \
> - echo "Connecting to $$i"; ${FETCH_CMD} $$i ; break; \
> + echo "Connecting to $$i"; ${_PFETCH} ${FETCH_CMD} $$i ; break; \
> done
>
This part is good
> show-required-by:
> @@ -3434,17 +3434,17 @@ show:
> # du fails if it can't access everything
> show-size:
> @if du -ks ${WRKDIR} 2>/dev/null >${WRKDIR}/wrkdir-size; then \
> - cat ${WRKDIR}/wrkdir-size && rm -f ${WRKDIR}/wrkdir-size; \
> + cat ${WRKDIR}/wrkdir-size && ${_PBUILD} rm -f
> ${WRKDIR}/wrkdir-size; \
> else \
> - chmod -R u+rX ${WRKDIR}; \
> + ${_PBUILD} chmod -R u+rX ${WRKDIR}; \
> du -ks ${WRKDIR}; \
> fi
>
> show-fake-size:
> @if du -ks ${WRKINST} 2>/dev/null >${WRKINST}/wrkdir-size; then \
> - cat ${WRKINST}/wrkdir-size && rm -f ${WRKINST}/wrkdir-size; \
> + cat ${WRKINST}/wrkdir-size && ${_PBUILD} rm -f
> ${WRKINST}/wrkdir-size; \
> else \
> - chmod -R u+rX ${WRKINST}; \
> + ${_PBUILD} chmod -R u+rX ${WRKINST}; \
> du -ks ${WRKINST}; \
> fi
>
Those should definitely be done differently... Namely adding a file under
WRKDIR seems like overkill, stuff could go to a tmp file
and also it's probably more useful to always go _PBUILD for the du instead of
chmod'ing.
