Compile error: "error: invalid initialization of reference of type ‘std::move_iterator<const long unsigned int*>::reference {aka long unsigned int&&}’ from expression of type ‘std::remove_reference<const long unsigned int&>::type {aka const long unsigned int}’" about datasketches-cpp HOT 3 CLOSED

Thanks for reporting this, I don't think anyone has tried it on 4.8.5 for a while. And I don't recall seeing it as an option in github actions, so we can't get it into our CI build. Operating on the idea that "untested == broken" I guess we should have expected this!

No ETA on this yet, but with luck we'll be able to look into it soon -- although there's a change the answer ends up being that we add a minimum gcc version.

from datasketches-cpp.

I updated to devtools-7 with g++ (GCC) 7.3.1 20180303 (Red Hat 7.3.1-5) but still get an error. This would suggest it's somewhere else in my build environment - most likely libc/glibc since it's in an allocator, but would like to check or upgrade the full stack if needed. Can you let me know what release/version of components are used for CI?

EDIT: I deployed an AWS instance with latest Amazon Linux 2.0, installed build tools cmake3, gcc-c++, and git, and compiled without issue. So it might be useful to know what the CI environment is also, but clearly it's a version issue on my side.

from datasketches-cpp.

It seems to me that the problem has to do with a recently introduced wrapped_compact_theta_sketch. I would think that if you are not using this particular class in your code, it should continue to compile and work. If you would like to compile all unit tests, I would suggest commenting out the offending test in theta_a_not_b_test.cpp lines 170-190 for now.

from datasketches-cpp.