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

Rebuild for libboost 1.84 #37

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

regro-cf-autotick-bot
Copy link
Contributor

This PR has been triggered in an effort to update boost1840.

Notes and instructions for merging this PR:

  1. Please merge the PR only after the tests have passed.
  2. Feel free to push to the bot's branch to update this PR if needed.

Please note that if you close this PR we presume that the feedstock has been rebuilt, so if you are going to perform the rebuild yourself don't close this PR until the your rebuild has been merged.


If this PR was opened in error or needs to be updated please add the bot-rerun label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase @conda-forge-admin, please rerun bot in a PR comment to have the conda-forge-admin add it for you.

This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/9103630188 - please use this URL for debugging.

@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@h-vetinari
Copy link
Member

osx running into

unknown location:0: fatal error: in "Serialization": memory access violation at address: 0x8: no mapping at fault address

I'm wondering if this is related to the libcxx ABI issues we've been seeing.

@traversaro
Copy link
Contributor

osx running into

unknown location:0: fatal error: in "Serialization": memory access violation at address: 0x8: no mapping at fault address

I'm wondering if this is related to the libcxx ABI issues we've been seeing.

Interesting, do you have any reference?

@h-vetinari
Copy link
Member

Interesting, do you have any reference?

Sure: conda-forge/libcxx-feedstock#148

@h-vetinari h-vetinari marked this pull request as draft May 29, 2024 23:46
@h-vetinari
Copy link
Member

Hm, libcxx issues look to be unrelated.

@traversaro
Copy link
Contributor

Hm, libcxx issues look to be unrelated.

Thanks for checking that out!

@johnwason
Copy link
Contributor

The vcpkg version of ompl is working fine with boost 1.84 and boost 1.85 on mac os both x64 and amd64. There are a few patches that might be helpful.

https://github.com/microsoft/vcpkg/tree/master/ports/ompl

@traversaro
Copy link
Contributor

There were some problems with libcxx hardening mode being enabled in release builds (see conda-forge/libcxx-feedstock#162), they have been fixed, so let's try to re-render.

@traversaro
Copy link
Contributor

@conda-forge-admin please rerender

@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@traversaro
Copy link
Contributor

@conda-forge-admin please rerender

@traversaro
Copy link
Contributor

fyi @h-vetinari I removed the recipe/conda_build_config.yaml, I guess it was a debugging leftover.

@traversaro
Copy link
Contributor

There were some problems with libcxx hardening mode being enabled in release builds (see conda-forge/libcxx-feedstock#162), they have been fixed, so let's try to re-render.

If that is the problem, it may be nevertheless useful to report the test failures upstream.

@h-vetinari
Copy link
Member

fyi @h-vetinari I removed the recipe/conda_build_config.yaml, I guess it was a debugging leftover.

Yes, absolutely! We shouldn't ever publish builds based on libcxx_dev!

If that is the problem, it may be nevertheless useful to report the test failures upstream.

AFAIR the problems here precede the publication of libcxx 18, so they are unlikely to be related. But it may be worth checking out the vcpkg patches and see if we can pick up something missing.

@traversaro
Copy link
Contributor

If that is the problem, it may be nevertheless useful to report the test failures upstream.

AFAIR the problems here precede the publication of libcxx 18, so they are unlikely to be related. But it may be worth checking out the vcpkg patches and see if we can pick up something missing.

I quickly checked them and they seem all to be related to CMake packaging, so I am not sure if they are related. Note that anyhow vcpkg does not run the projects' test suite as part of their CI, so (unless someone run those test somewhere else) we are not sure if they do not have those failures as well.

@traversaro
Copy link
Contributor

traversaro commented Jul 31, 2024

Indeed, this is still failing:

2024-07-31T07:35:55.5165650Z Test project $SRC_DIR/build
2024-07-31T07:35:55.5168480Z       Start  1: test_adjacency_list
2024-07-31T07:35:55.7051120Z  1/21 Test  #1: test_adjacency_list ..............   Passed    0.22 sec
2024-07-31T07:35:55.7052420Z       Start  2: test_heap
2024-07-31T07:35:55.7102250Z  2/21 Test  #2: test_heap ........................   Passed    0.00 sec
2024-07-31T07:35:55.7103270Z       Start  3: test_grid
2024-07-31T07:35:55.7154270Z  3/21 Test  #3: test_grid ........................   Passed    0.00 sec
2024-07-31T07:35:55.7154970Z       Start  4: test_gridb
2024-07-31T07:35:55.7202300Z  4/21 Test  #4: test_gridb .......................   Passed    0.00 sec
2024-07-31T07:35:55.7204290Z       Start  5: test_nearestneighbors
2024-07-31T07:35:57.2147820Z  5/21 Test  #5: test_nearestneighbors ............   Passed    1.49 sec
2024-07-31T07:35:57.2174560Z       Start  6: test_pdf
2024-07-31T07:35:57.3756180Z  6/21 Test  #6: test_pdf .........................   Passed    0.16 sec
2024-07-31T07:35:57.3757870Z       Start  7: test_random
2024-07-31T07:35:57.8814090Z  7/21 Test  #7: test_random ......................   Passed    0.50 sec
2024-07-31T07:35:57.8814760Z       Start  8: test_halton_sampling
2024-07-31T07:35:57.8938910Z  8/21 Test  #8: test_halton_sampling .............   Passed    0.01 sec
2024-07-31T07:35:57.8939750Z       Start  9: test_state_operations
2024-07-31T07:35:59.1558330Z  9/21 Test  #9: test_state_operations ............   Passed    1.26 sec
2024-07-31T07:35:59.1558990Z       Start 10: test_state_spaces
2024-07-31T07:35:59.2248960Z 10/21 Test #10: test_state_spaces ................   Passed    0.07 sec
2024-07-31T07:35:59.2330060Z       Start 11: test_state_storage
2024-07-31T07:35:59.2402180Z 11/21 Test #11: test_state_storage ...............   Passed    0.01 sec
2024-07-31T07:35:59.2402740Z       Start 12: test_ptc
2024-07-31T07:35:59.7243990Z 12/21 Test #12: test_ptc .........................   Passed    0.45 sec
2024-07-31T07:35:59.7245870Z       Start 13: test_planner_data
2024-07-31T07:37:51.0929170Z 13/21 Test #13: test_planner_data ................***Failed  111.40 sec
2024-07-31T07:37:51.0930390Z Running 6 test cases...
2024-07-31T07:37:51.0931800Z Error:   Failed to load PlannerData: input stream error
2024-07-31T07:37:51.0932510Z          at line 140 in $SRC_DIR/src/ompl/base/src/PlannerDataStorage.cpp
2024-07-31T07:37:51.0933600Z $SRC_DIR/tests/base/planner_data.cpp:530: �[1;31;49merror: in "Serialization": check data2.numVertices() == states.size() has failed [0 != 1000]�[0;39;49m
2024-07-31T07:37:51.0934570Z $SRC_DIR/tests/base/planner_data.cpp:531: �[1;31;49merror: in "Serialization": check data2.numEdges() == num_edges_to_add has failed [0 != 10000]�[0;39;49m
2024-07-31T07:37:51.0935550Z $SRC_DIR/tests/base/planner_data.cpp:534: �[1;31;49merror: in "Serialization": check data2.numStartVertices() == 3 has failed�[0;39;49m
2024-07-31T07:37:51.0936330Z $SRC_DIR/tests/base/planner_data.cpp:535: �[1;31;49merror: in "Serialization": check data2.numGoalVertices() == 2 has failed�[0;39;49m
2024-07-31T07:37:51.0937230Z $SRC_DIR/tests/base/planner_data.cpp:536: �[1;31;49merror: in "Serialization": check data2.isStartVertex(0) has failed�[0;39;49m
2024-07-31T07:37:51.0937880Z $SRC_DIR/tests/base/planner_data.cpp:537: �[1;31;49merror: in "Serialization": check data2.isStartVertex(states.size()/2) has failed�[0;39;49m
2024-07-31T07:37:51.0938770Z $SRC_DIR/tests/base/planner_data.cpp:538: �[1;31;49merror: in "Serialization": check data2.isStartVertex(states.size()-1) has failed�[0;39;49m
2024-07-31T07:37:51.0939550Z $SRC_DIR/tests/base/planner_data.cpp:539: �[1;31;49merror: in "Serialization": check data2.isGoalVertex(1) has failed�[0;39;49m
2024-07-31T07:37:51.0940330Z $SRC_DIR/tests/base/planner_data.cpp:540: �[1;31;49merror: in "Serialization": check data2.isGoalVertex(states.size()-2) has failed�[0;39;49m
2024-07-31T07:37:51.0941350Z unknown location:0: �[4;31;49mfatal error: in "Serialization": memory access violation at address: 0x8: no mapping at fault address�[0;39;49m
2024-07-31T07:37:51.0942150Z $SRC_DIR/tests/base/planner_data.cpp:544: �[1;36;49mlast checkpoint�[0;39;49m
2024-07-31T07:37:51.0942470Z 
2024-07-31T07:37:51.0942860Z �[1;31;49m*** 10 failures are detected in the test module "PlannerData"
2024-07-31T07:37:51.0943150Z �[0;39;49m
2024-07-31T07:37:51.0943350Z 
2024-07-31T07:37:51.0943600Z       Start 14: test_2denvs_geometric
2024-07-31T07:38:43.0818300Z 14/21 Test #14: test_2denvs_geometric ............   Passed   51.99 sec
2024-07-31T07:38:43.0820750Z       Start 15: test_2dmap_geometric_simple
2024-07-31T07:38:43.4365790Z 15/21 Test #15: test_2dmap_geometric_simple ......   Passed    0.35 sec
2024-07-31T07:38:43.4369140Z       Start 16: test_2dmap_ik
2024-07-31T07:38:43.4626550Z 16/21 Test #16: test_2dmap_ik ....................   Passed    0.03 sec
2024-07-31T07:38:43.4628650Z       Start 17: test_2dcircles_opt_geometric
2024-07-31T07:39:23.3169980Z 17/21 Test #17: test_2dcircles_opt_geometric .....   Passed   39.85 sec
2024-07-31T07:39:23.3170700Z       Start 18: test_2dpath_simplifying
2024-07-31T07:39:23.4120130Z 18/21 Test #18: test_2dpath_simplifying ..........   Passed    0.09 sec
2024-07-31T07:39:23.4121670Z       Start 19: test_constraint_sphere
2024-07-31T07:39:53.3258440Z 19/21 Test #19: test_constraint_sphere ...........   Passed   29.91 sec
2024-07-31T07:39:53.3259380Z       Start 20: test_2dmap_control
2024-07-31T07:39:58.9493010Z 20/21 Test #20: test_2dmap_control ...............   Passed    5.62 sec
2024-07-31T07:39:58.9493920Z       Start 21: test_planner_data_control
2024-07-31T07:41:34.6173900Z 21/21 Test #21: test_planner_data_control ........***Failed   95.67 sec
2024-07-31T07:41:34.6175330Z Running 5 test cases...
2024-07-31T07:41:34.6176530Z Debug:   Storing 10000 PlannerDataEdgeControl objects
2024-07-31T07:41:34.6177580Z Error:   Failed to load PlannerData: input stream error
2024-07-31T07:41:34.6178270Z          at line 112 in $SRC_DIR/src/ompl/control/src/PlannerDataStorage.cpp
2024-07-31T07:41:34.6179500Z $SRC_DIR/tests/control/planner_data.cpp:549: �[1;31;49merror: in "Serialization": check data2.numVertices() == states.size() has failed [0 != 1000]�[0;39;49m
2024-07-31T07:41:34.6181390Z $SRC_DIR/tests/control/planner_data.cpp:550: �[1;31;49merror: in "Serialization": check data2.numEdges() == num_edges_to_add has failed [0 != 10000]�[0;39;49m
2024-07-31T07:41:34.6183110Z unknown location:0: �[4;31;49mfatal error: in "Serialization": memory access violation at address: 0x8: no mapping at fault address�[0;39;49m
2024-07-31T07:41:34.6184330Z $SRC_DIR/tests/control/planner_data.cpp:555: �[1;36;49mlast checkpoint�[0;39;49m
2024-07-31T07:41:34.6184960Z 
2024-07-31T07:41:34.6186270Z �[1;31;49m*** 3 failures are detected in the test module "PlannerDataControl"
2024-07-31T07:41:34.6187800Z �[0;39;49m
2024-07-31T07:41:34.6188400Z 
2024-07-31T07:41:34.6188780Z 
2024-07-31T07:41:34.6189580Z 90% tests passed, 2 tests failed out of 21
2024-07-31T07:41:34.6200550Z 
2024-07-31T07:41:34.6201560Z Total Test time (real) = 339.13 sec
2024-07-31T07:41:34.6203550Z 
2024-07-31T07:41:34.6204230Z The following tests FAILED:
2024-07-31T07:41:34.6206710Z 	 13 - test_planner_data (Failed)
2024-07-31T07:41:34.6207880Z 	 21 - test_planner_data_control (Failed)

Let's open a debug PR to understand if this is actually related to boost 1.84 .

@traversaro
Copy link
Contributor

Let's open a debug PR to understand if this is actually related to boost 1.84 .

Actually we have the answer, #38 that builds with boost 1.83 the tests are passing fine.

@h-vetinari
Copy link
Member

Ah right, I recall now that this was failing the tests somehow;

13/21 Test #13: test_planner_data ................***Failed  109.39 sec
Running 6 test cases...
Error:   Failed to load PlannerData: input stream error
         at line 140 in $SRC_DIR/src/ompl/base/src/PlannerDataStorage.cpp
$SRC_DIR/tests/base/planner_data.cpp:530: error: in "Serialization": check data2.numVertices() == states.size() has failed [0 != 1000]
$SRC_DIR/tests/base/planner_data.cpp:531: error: in "Serialization": check data2.numEdges() == num_edges_to_add has failed [0 != 10000]
$SRC_DIR/tests/base/planner_data.cpp:534: error: in "Serialization": check data2.numStartVertices() == 3 has failed
$SRC_DIR/tests/base/planner_data.cpp:535: error: in "Serialization": check data2.numGoalVertices() == 2 has failed
$SRC_DIR/tests/base/planner_data.cpp:536: error: in "Serialization": check data2.isStartVertex(0) has failed
$SRC_DIR/tests/base/planner_data.cpp:537: error: in "Serialization": check data2.isStartVertex(states.size()/2) has failed
$SRC_DIR/tests/base/planner_data.cpp:538: error: in "Serialization": check data2.isStartVertex(states.size()-1) has failed
$SRC_DIR/tests/base/planner_data.cpp:539: error: in "Serialization": check data2.isGoalVertex(1) has failed
$SRC_DIR/tests/base/planner_data.cpp:540: error: in "Serialization": check data2.isGoalVertex(states.size()-2) has failed
unknown location:0: fatal error: in "Serialization": memory access violation at address: 0x8: no mapping at fault address
$SRC_DIR/tests/base/planner_data.cpp:544: last checkpoint

*** 10 failures are detected in the test module "PlannerData"

Given the following warning in the implementation

[68/278] Building CXX object src/ompl/CMakeFiles/ompl.dir/base/src/PlannerData.cpp.o
$SRC_DIR/src/ompl/base/src/PlannerData.cpp:82:18: warning: variable 'count' set but not used [-Wunused-but-set-variable]
    unsigned int count = 0;
                 ^
1 warning generated.

it could be perhaps that there's some UB going on that happens to pass on linux/win, but not on newer clang anymore?

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.

4 participants