Merge pull request atomvm#517 from fadushin/fix-install-target

Install target fix

This PR fixes the install target so that atomvm script gets properly deployed.

In addition, the Linux and macOS workflows now exercise the install target.

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

Author: bettio

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

@@ -83,3 +83,10 @@ jobs:
       working-directory: build
       run: |
         ./src/AtomVM ./tests/libs/alisp/test_alisp.avm
+
+    - name: "Install and smoke test"
+      working-directory: build
+      run: |
+        export PATH="/usr/local/opt/erlang@${{ matrix.otp }}/bin:$PATH"
+        sudo ninja install
+        atomvm examples/erlang/hello_world.avm

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

@@ -258,3 +258,9 @@ jobs:
       run: |
         ./src/AtomVM ./tests/libs/alisp/test_alisp.avm
         valgrind ./src/AtomVM ./tests/libs/alisp/test_alisp.avm
+
+    - name: "Install and smoke test"
+      working-directory: build
+      run: |
+        sudo PATH=${PATH} make install
+        atomvm examples/erlang/hello_world.avm

src/platforms/generic_unix/CMakeLists.txt

@@ -58,3 +58,8 @@ if (COVERAGE)
 endif()
 
 install(TARGETS AtomVM DESTINATION lib/atomvm)
+install(
+    FILES ${CMAKE_CURRENT_SOURCE_DIR}/atomvm
+    DESTINATION bin
+    PERMISSIONS OWNER_READ OWNER_WRITE OWNER_EXECUTE GROUP_READ GROUP_EXECUTE WORLD_READ WORLD_EXECUTE
+)

src/platforms/generic_unix/lib/CMakeLists.txt

@@ -61,12 +61,6 @@ endif()
 target_compile_definitions(libAtomVM${PLATFORM_LIB_SUFFIX} PUBLIC DYNLOAD_PORT_DRIVERS)
 target_link_libraries(libAtomVM${PLATFORM_LIB_SUFFIX} PUBLIC ${CMAKE_DL_LIBS})
 
-install(
-    FILES ${CMAKE_CURRENT_SOURCE_DIR}/atomvm
-    DESTINATION bin
-    PERMISSIONS OWNER_READ OWNER_WRITE OWNER_EXECUTE GROUP_READ GROUP_EXECUTE WORLD_READ WORLD_EXECUTE
-)
-
 if (COVERAGE)
     include(CodeCoverage)
     append_coverage_compiler_flags_to_target(libAtomVM${PLATFORM_LIB_SUFFIX})