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 conda-based CI on Windows, macOS and Linux #566

Merged
merged 3 commits into from
Aug 3, 2021
Merged

Conversation

traversaro
Copy link
Member

@traversaro traversaro commented Jul 29, 2021

With some logic taken from robotology/idyntree-yarp-tools#17 to compile YARP from source in devel branch, at least for now.

@traversaro
Copy link
Member Author

Compilation works. The ControlBoardControlTest tests seems to be flaky on Conda/Linux, as it happens with Apt-provided Gazebo.

macOS tests are failing instead with:

Test project /Users/runner/work/gazebo-yarp-plugins/gazebo-yarp-plugins/build
    Start 1: ControlBoardControlTest
1/2 Test #1: ControlBoardControlTest ..........***Exception: SegFault 13.70 sec
[==========] Running 4 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 4 tests from PhysicsEngines/ControlBoardControlTest
[ RUN      ] PhysicsEngines/ControlBoardControlTest.PluginTest/0

    Start 2: RobotInterfaceTest
2/2 Test #2: RobotInterfaceTest ...............***Exception: SegFault  0.45 sec
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from PhysicsEngines/RobotInterfaceTest
[ RUN      ] PhysicsEngines/RobotInterfaceTest.PluginTest/0


0% tests passed, 2 tests failed out of 2

Total Test time (real) =  14.18 sec

The following tests FAILED:
	  1 - ControlBoardControlTest (SEGFAULT)
	  2 - RobotInterfaceTest (SEGFAULT)
Errors while running CTest

While on Windows the tests hangs:

Test project D:/a/gazebo-yarp-plugins/gazebo-yarp-plugins/build
    Start 1: ControlBoardControlTest

Given this, I think it is ok to disable tests for now on macOS and Windows.

@traversaro traversaro requested a review from ste93 July 29, 2021 10:13
@traversaro
Copy link
Member Author

@ste93 I added this so we are sure that all the future refactoring are not going to create compilation problems.

@traversaro traversaro requested a review from Nicogene August 3, 2021 08:50
@traversaro traversaro merged commit 72fea65 into devel Aug 3, 2021
@traversaro traversaro deleted the add-conda-ci branch August 3, 2021 14:06
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.

2 participants