Skip to content

Commit 127ba9e

Browse files
committed
Merge pull request #1981 from pguyot/w46/fix-cmake-dependencies-and-remove-build-other
Fix several CMake dependencies and remove armv4t cross build These changes are made under both the "Apache 2.0" and the "GNU Lesser General Public License 2.1 or later" license terms (dual license). SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later
2 parents 831972d + 1a32750 commit 127ba9e

File tree

3 files changed

+10
-179
lines changed

3 files changed

+10
-179
lines changed

.github/workflows/build-and-test-other.yaml

Lines changed: 0 additions & 173 deletions
This file was deleted.

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)