]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
Prevent eapol_test from running repeatedly (again)
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Thu, 6 Jul 2017 14:57:15 +0000 (10:57 -0400)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Thu, 6 Jul 2017 14:57:15 +0000 (10:57 -0400)
This was broken during the conversion to boilermake

src/tests/eapol_test/all.mk

index 2d399f0c0efbd78791a6b8cb60f6c994516b78c0..ca8a6b5450f0ea512ee0a2b2a7691b2cd721c5de 100644 (file)
@@ -167,8 +167,18 @@ $(OUTPUT_DIR)/%.ok: $(DIR)/%.conf | radiusd.kill $(CONFIG_PATH)/radiusd.pid
 tests.eap: $(EAPOL_OK_FILES)
        ${Q}$(MAKE) radiusd.kill
 else
+#
+#  Build rules and the make file get evaluated at different times
+#  if we don't touch the test skipped file immediately, users can
+#  cntrl-c out of the build process, and the skip file never gets
+#  created as the tests.eap target is evaluated much later in the
+#  build process.2
+#
+ifneq (,$(findstring test,$(MAKECMDGOALS)))
+$(shell touch "$(OUTPUT_DIR)/eapol_test.skip")
+endif
+
 tests.eap: $(OUTPUT_DIR)
        ${Q}echo "Skipping EAP tests due to previous build error"
        ${Q}echo "Retry with: $(MAKE) clean.$@ && $(MAKE) $@"
-       ${Q}touch "$(OUTPUT_DIR)/eapol_test.skip"
 endif