git: ec6682ed6c1d - main - atf.test: fix installation of python test scripts
Date: Tue, 19 Dec 2023 16:24:08 UTC
The branch main has been updated by kp:
URL: https://cgit.FreeBSD.org/src/commit/?id=ec6682ed6c1d7304521bad39a1ad7ceae76aaca3
commit ec6682ed6c1d7304521bad39a1ad7ceae76aaca3
Author: Kristof Provost <kp@FreeBSD.org>
AuthorDate: 2023-12-19 13:15:48 +0000
Commit: Kristof Provost <kp@FreeBSD.org>
CommitDate: 2023-12-19 16:23:54 +0000
atf.test: fix installation of python test scripts
Python test scripts get processed (to add the `#! /usr/libexec/
atf_pytest_wrapper` shebang line), into a .xtmp file, and installed from
there. However, as there was no dependency of this .xtmp file on the
original file we kept reinstalling the .xtmp file, even if the original
had been edited already.
This could cause great confusion when debugging python test scripts.
Reviewed by: markj
Sponsored by: Rubicon Communications, LLC ("Netgate")
Differential Revision: https://reviews.freebsd.org/D43118
---
share/mk/atf.test.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/share/mk/atf.test.mk b/share/mk/atf.test.mk
index 4b8ac8dedc27..f7696641ffb0 100644
--- a/share/mk/atf.test.mk
+++ b/share/mk/atf.test.mk
@@ -143,7 +143,7 @@ CLEANFILES+= ${_T} ${_T}.tmp
# this proves to be useful within the tests.
ATF_TESTS_PYTEST_SED_${_T}?= # empty
ATF_TESTS_PYTEST_SRC_${_T}?= ${.CURDIR}/${_T:S,.xtmp$,,}
-${_T}:
+${_T}: ${_T:S/.xtmp//}
echo "#! /usr/libexec/atf_pytest_wrapper -P ${TESTSBASE}" > ${.TARGET}.tmp
.if empty(ATF_TESTS_PYTEST_SED_${_T})
cat ${ATF_TESTS_PYTEST_SRC_${_T}} >>${.TARGET}.tmp