Skip to content

Commit 1d3d8aa

Browse files
committed
Fix several CMake dependencies
Signed-off-by: Paul Guyot <[email protected]>
1 parent e2f7842 commit 1d3d8aa

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

CMakeModules/BuildErlang.cmake

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ macro(pack_lib avm_name)
153153

154154
add_custom_command(
155155
OUTPUT ${avm_name}.avm
156-
DEPENDS ${pack_lib_${avm_name}_archive_targets} PackBEAM
156+
DEPENDS ${pack_lib_${avm_name}_archive_targets} ${pack_lib_${avm_name}_emu_archives} ${pack_lib_${avm_name}_archives} PackBEAM
157157
COMMAND ${CMAKE_BINARY_DIR}/tools/packbeam/PackBEAM -a ${INCLUDE_LINES} ${avm_name}.avm ${pack_lib_${avm_name}_emu_archives} ${pack_lib_${avm_name}_archives}
158158
COMMENT "Packing lib ${avm_name}.avm"
159159
VERBATIM
@@ -172,7 +172,7 @@ macro(pack_lib avm_name)
172172

173173
add_custom_command(
174174
OUTPUT ${avm_name}-${jit_target_arch_variant}.avm
175-
DEPENDS ${pack_lib_${avm_name}_archive_targets} PackBEAM
175+
DEPENDS ${pack_lib_${avm_name}_archive_targets} ${pack_lib_${avm_name}_jit_archives_${jit_target_arch_variant}} PackBEAM
176176
COMMAND ${CMAKE_BINARY_DIR}/tools/packbeam/PackBEAM -a ${INCLUDE_LINES} ${avm_name}-${jit_target_arch_variant}.avm ${pack_lib_${avm_name}_jit_archives_${jit_target_arch_variant}} ${pack_lib_${avm_name}_archives}
177177
COMMENT "Packing lib ${avm_name}-${jit_target_arch_variant}.avm"
178178
VERBATIM

tests/erlang_tests/CMakeLists.txt

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,10 @@ macro(jit_precompile module_name)
4242
endmacro()
4343

4444
function(compile_erlang module_name)
45+
# Parse optional DEPENDS argument
46+
set(multiValueArgs DEPENDS)
47+
cmake_parse_arguments(COMPILE_ERLANG "" "" "${multiValueArgs}" ${ARGN})
48+
4549
if(AVM_DISABLE_JIT)
4650
set(erlc_define -DAVM_DISABLE_JIT)
4751
else()
@@ -51,7 +55,7 @@ function(compile_erlang module_name)
5155
add_custom_command(
5256
OUTPUT ${module_name}.beam
5357
COMMAND erlc ${erlc_define} ${CMAKE_CURRENT_SOURCE_DIR}/${module_name}.erl
54-
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/${module_name}.erl
58+
DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/${module_name}.erl ${COMPILE_ERLANG_DEPENDS}
5559
COMMENT "Compiling ${module_name}.erl"
5660
)
5761
jit_precompile(${module_name})
@@ -566,12 +570,12 @@ compile_erlang(small_big_ext)
566570
compile_erlang(test_crypto)
567571

568572
compile_erlang(test_code_all_available_loaded)
569-
compile_erlang(test_code_load_binary)
573+
compile_erlang(test_code_load_binary DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/code_load/export_test_module_data.hrl)
570574
compile_erlang(test_code_load_abs)
571575
compile_erlang(test_code_ensure_loaded)
572-
compile_erlang(test_add_avm_pack_binary)
576+
compile_erlang(test_add_avm_pack_binary DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/code_load/code_load_pack_data.hrl)
573577
compile_erlang(test_add_avm_pack_file)
574-
compile_erlang(test_close_avm_pack)
578+
compile_erlang(test_close_avm_pack DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/code_load/code_load_pack_data.hrl)
575579

576580
compile_erlang(test_module_info)
577581
compile_erlang(erlang_module_loaded)

0 commit comments

Comments
 (0)