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

conda-forge: Test failures on OS X #1771

Closed
wolfv opened this issue Nov 3, 2020 · 2 comments
Closed

conda-forge: Test failures on OS X #1771

wolfv opened this issue Nov 3, 2020 · 2 comments

Comments

@wolfv
Copy link

wolfv commented Nov 3, 2020

Hi cartographer team!

We are packaging cartographer on conda-forge and we have a test failure on OS X.

We already have working builds for Windows and Linux 🎉

Do you have any idea where this ILLEGAL error could come from? Is it an illegal instruction?

conda-forge/cartographer-feedstock#7
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=235112&view=results

@wolfv wolfv changed the title Test failures on OS X conda-forge: Test failures on OS X Nov 3, 2020
@Tobias-Fischer
Copy link

I had a quick look and the build gives away what the underlying issue is:

[80/333] Building CXX object CMakeFiles/cartographer.mapping.internal.2d.overlapping_submaps_trimmer_2d_test.dir/cartographer/mapping/internal/2d/overlapping_submaps_trimmer_2d_test.cc.o
In file included from ../cartographer/mapping/internal/2d/overlapping_submaps_trimmer_2d_test.cc:17:
In file included from ../cartographer/mapping/internal/2d/overlapping_submaps_trimmer_2d.h:20:
In file included from ../cartographer/common/port.h:22:
In file included from $BUILD_PREFIX/bin/../include/c++/v1/string:506:
In file included from $BUILD_PREFIX/bin/../include/c++/v1/string_view:175:
In file included from $BUILD_PREFIX/bin/../include/c++/v1/__string:57:
In file included from $BUILD_PREFIX/bin/../include/c++/v1/algorithm:643:
$BUILD_PREFIX/bin/../include/c++/v1/memory:2262:5: warning: delete called on 'cartographer::mapping::Grid2D' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]
    delete __ptr;
    ^
$BUILD_PREFIX/bin/../include/c++/v1/memory:2517:7: note: in instantiation of member function 'std::__1::default_delete<cartographer::mapping::Grid2D>::operator()' requested here
      __ptr_.second()(__tmp);
      ^
$BUILD_PREFIX/bin/../include/c++/v1/memory:2471:19: note: in instantiation of member function 'std::__1::unique_ptr<cartographer::mapping::Grid2D, std::__1::default_delete<cartographer::mapping::Grid2D> >::reset' requested here
  ~unique_ptr() { reset(); }
                  ^
../cartographer/mapping/2d/submap_2d.h:42:7: note: in instantiation of member function 'std::__1::unique_ptr<cartographer::mapping::Grid2D, std::__1::default_delete<cartographer::mapping::Grid2D> >::~unique_ptr' requested here
class Submap2D : public Submap {
      ^
$BUILD_PREFIX/bin/../include/c++/v1/memory:4291:26: note: in instantiation of function template specialization 'std::__1::__shared_ptr_emplace<const cartographer::mapping::Submap2D, std::__1::allocator<const cartographer::mapping::Submap2D> >::__shared_ptr_emplace<cartographer::mapping::proto::Submap2D &>' requested here
    ::new(__hold2.get()) _CntrlBlk(__a2, _VSTD::forward<_Args>(__args)...);
                         ^
../cartographer/mapping/internal/2d/overlapping_submaps_trimmer_2d_test.cc:72:15: note: in instantiation of function template specialization 'std::__1::make_shared<const cartographer::mapping::Submap2D, cartographer::mapping::proto::Submap2D &>' requested here
        {std::make_shared<const Submap2D>(submap_2d),
              ^
1 warning generated.

Similarly:

$BUILD_PREFIX/bin/../include/c++/v1/memory:2262:5: warning: delete called on 'cartographer::mapping::RangeDataInserterInterface' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor]

Tobias-Fischer added a commit to Tobias-Fischer/cartographer that referenced this issue Nov 3, 2020
Signed-off-by: Tobias Fischer <info@tobiasfischer.info>
@wolfv
Copy link
Author

wolfv commented Nov 5, 2020

Is fixed in master.

@wolfv wolfv closed this as completed Nov 5, 2020
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

No branches or pull requests

2 participants