build: drop build-certs as a test-run dependency

After adding it as a test executables dependency, it run twice in
MSBuild jobs. Also there is little reason to try building them in both
build and run tests targets.

(The reason MSBuild building it twice, is our use of
`TrackFileAccess=false` to improve build performance.)

https://github.com/curl/curl/actions/runs/14156797251/job/39662914155?pr=16840#step:15:31

Follow-up to 68609f0e33 #16866
Follow-up to 0c1ad21f97 #16845

Closes #16876
This commit is contained in:
Viktor Szakats 2025-03-30 22:34:26 +02:00
parent 3e2860f66d
commit fb4dbbac4a
No known key found for this signature in database
GPG Key ID: B5ABD165E2AEF201
2 changed files with 9 additions and 11 deletions

View File

@ -41,7 +41,7 @@ function(curl_add_runtests _targetname _test_flags)
if(NOT BUILD_LIBCURL_DOCS) if(NOT BUILD_LIBCURL_DOCS)
string(APPEND _test_flags " !documentation") string(APPEND _test_flags " !documentation")
endif() endif()
set(_depends "build-certs") set(_depends "")
# Skip walking through dependent targets before running tests in CI. # Skip walking through dependent targets before running tests in CI.
# This avoids: GNU Make doing a slow re-evaluation of all targets and # This avoids: GNU Make doing a slow re-evaluation of all targets and
# skipping them, MSBuild doing a re-evaluation, and actually rebuilding them. # skipping them, MSBuild doing a re-evaluation, and actually rebuilding them.

View File

@ -160,30 +160,28 @@ perlcheck:
build-certs: build-certs:
(cd certs && $(MAKE) build-certs) (cd certs && $(MAKE) build-certs)
runtests-pre: perlcheck build-certs test: perlcheck all
test: runtests-pre all
$(TEST) $(TFLAGS) $(TEST) $(TFLAGS)
quiet-test: runtests-pre all quiet-test: perlcheck all
$(TEST) $(TEST_Q) $(TFLAGS) $(TEST) $(TEST_Q) $(TFLAGS)
am-test: runtests-pre all am-test: perlcheck all
$(TEST) $(TEST_AM) $(TFLAGS) $(TEST) $(TEST_AM) $(TFLAGS)
ci-test: runtests-pre all ci-test: perlcheck all
$(TEST) $(TEST_CI) $(TFLAGS) $(TEST) $(TEST_CI) $(TFLAGS)
full-test: runtests-pre all full-test: perlcheck all
$(TEST) $(TEST_F) $(TFLAGS) $(TEST) $(TEST_F) $(TFLAGS)
nonflaky-test: runtests-pre all nonflaky-test: perlcheck all
$(TEST) $(TEST_NF) $(TFLAGS) $(TEST) $(TEST_NF) $(TFLAGS)
torture-test: runtests-pre all torture-test: perlcheck all
$(TEST) $(TEST_T) $(TFLAGS) $(TEST) $(TEST_T) $(TFLAGS)
event-test: runtests-pre all event-test: perlcheck all
$(TEST) $(TEST_E) $(TFLAGS) $(TEST) $(TEST_E) $(TFLAGS)
default-pytest: ci-pytest default-pytest: ci-pytest