Commit 85210039 authored by Tom Finegan's avatar Tom Finegan

cmake: always create testdata targets.

Creation of the testdata target was previously guarded by
ENABLE_IDE_TEST_HOSTING. This change ensures the testdata
target is always available.

BUG=aomedia:1243

Change-Id: Ief47d4c16c3718b15505e8bc63b519d0c4dbc0ae
parent d0a3b9f1
......@@ -444,17 +444,6 @@ function (setup_aom_test_targets)
"AOM_UNIT_TEST_COMMON_INTRIN_NEON")
endif ()
if (NOT ENABLE_IDE_TEST_HOSTING)
if (MSVC OR XCODE)
# Skip creation of test data download and test run targets when generating
# for Visual Studio and Xcode unless the user explicitly requests IDE test
# hosting. This is done to make build cycles in the IDE tolerable when the
# IDE command for build project is used to build AOM. Default behavior in
# IDEs is to build all targets, and the test run takes hours.
return ()
endif ()
endif ()
make_test_data_lists("${AOM_UNIT_TEST_DATA_LIST_FILE}"
test_files test_file_checksums)
list(LENGTH test_files num_test_files)
......@@ -478,6 +467,17 @@ function (setup_aom_test_targets)
add_custom_target(testdata)
add_dependencies(testdata ${testdata_targets})
if (NOT ENABLE_IDE_TEST_HOSTING)
if (MSVC OR XCODE)
# Skip creation of test run targets when generating for Visual Studio and
# Xcode unless the user explicitly requests IDE test hosting. This is done
# to make build cycles in the IDE tolerable when the IDE command for build
# project is used to build AOM. Default behavior in
# IDEs is to build all targets, and the test run takes hours.
return ()
endif ()
endif ()
# Pick a reasonable number of targets (this controls parallelization).
ProcessorCount(num_test_targets)
if (num_test_targets EQUAL 0)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment