Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add github actions. #1962

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

lygstate
Copy link

Disable check sign off first.

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]

@lygstate lygstate force-pushed the github-actions-upstream branch 3 times, most recently from f5eba67 to d039e33 Compare January 31, 2021 06:43
IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
Also fixes building with cmake 3.x

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
Testing MSVC generator and setup CONFIG_TYPE properly
Add BUILD_BYPRODUCTS to ExternalProject_Add for building with Ninja

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
target_link_libraries implies dependencies.

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
iotjs_add_flags works on list instead of string.
Also fixes flags not support MSVC

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
@lygstate lygstate force-pushed the github-actions-upstream branch 3 times, most recently from dc23a76 to 54c32f9 Compare February 9, 2021 18:03
Use if DEFINED to make sure the flag still setup when it's defined and value to NO
separate_arguments(EXTRA_JERRY_CMAKE_PARAMS) at the beginning.

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
==3059== Memcheck, a memory error detector
==3059== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==3059== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==3059== Command: /root/work_space/iotjs/build/x86_64-linux/debug/bin/iotjs /root/work_space/iotjs/test/run_pass/test_net_8.js
==3059==
AssertionError: {"actual":"1","expected":"12","operator":"=="}
AssertionError: {"actual":1,"expected":0,"operator":"=="}
==3059==
==3059== HEAP SUMMARY:
==3059==     in use at exit: 0 bytes in 0 blocks
==3059==   total heap usage: 131 allocs, 131 frees, 314,345 bytes allocated
==3059==

According to this, the server on close event triggered multiple times by close function

wait all four connection to end

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
Getting /usr/i686-linux-gnu/lib in LD_LIBRARY_PATH when cross i6868

Signed-off-by: Yonggang Luo <[email protected]>
2021-02-05T14:59:40.0078677Z ==3560== Command: /root/work_space/iotjs/build/x86_64-linux/debug/bin/iotjs /root/work_space/iotjs/test/run_pass/test_iotjs_promise.js
2021-02-05T14:59:40.0080006Z ==3560==
2021-02-05T14:59:40.0080983Z AssertionError: {"expected":"Resolved","operator":"=="}
Use consistence CMAKE_TOOLCHAIN_FILE

Signed-off-by: Yonggang Luo <[email protected]>
@lygstate lygstate force-pushed the github-actions-upstream branch 2 times, most recently from 655b965 to 4a0f4a2 Compare February 10, 2021 08:32
Disable check sign off first.
Rename to ci_script.py

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]
@lygstate lygstate force-pushed the github-actions-upstream branch from 4a0f4a2 to 7505a41 Compare February 10, 2021 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant