Open Zhaojun-Liu opened 2 weeks ago
@Zhaojun-Liu the error is on my part, I have been changing the bit_set
repo and not ported back some of the API changes to dependent repos like dctl
. I will try and get this working again but it might take a week or two.
@Zhaojun-Liu the error is on my part, I have been changing the
bit_set
repo and not ported back some of the API changes to dependent repos likedctl
. I will try and get this working again but it might take a week or two.
Okay, thank you for the quickly reply. 😃
@Zhaojun-Liu can you please open an issue on dctl as well?
@Zhaojun-Liu can you please open an issue on dctl as well?
bit_set itself builds cleanly and I've exposed all necessary methods for dependent project (add()
and pop()
members had been disabled). Closing this issue, but please check the dctl issue.
@Zhaojun-Liu I have pushed a new commit removing some C++23 constructs currently not supported by MSVC. Could you please update your copy of this repo and do the canonical CMake steps as follows:
cd bit_set
mkdir build
cd build
cmake ..
cmake --build .
ctest -E bench
Once this works succesfully, I can take a look at the DCTL repo again.
Hi @rhalbersma, Let me first talk about the test results: after I updated bit_set to the latest commit (4e951b0), the DCTL build was fine, but when I followed your steps to build bit_set itself, it failed, you can check the attachments.
For details (with VS2022 x64 Native Tools command): The full steps I followed to build DCTL are as follows, and now it built fine.
Steps to build bit_set , based on your steps, I just updated the cmake step and added the extra 3 packages using vcpkg as follows, it failed and I attached the build log "cmake_build_bs.log", I also tried to build with msbuild, that is replace step 6 with msbuild /m /p:Platform=x64 /p:Configuration=Release bit_set.sln /t:Rebuild 2>&1 >msbuild_bs.log
, the log has also been uploaded as an attachment.
bit_set build logs (same failures in the 2 logs): cmake_build_bs.log msbuild_bs.log
@Zhaojun-Liu sorry, I missed a few size_t literals (just grepped for 0 and 1, but there were a few more) in the unit tests. Should be fixed now. Thanks for testing this so thoroughly. Note that you might also have to link to Google Benchmark (not sure how if it's within vcpkg yet) to build all the benchmarks.
@Zhaojun-Liu sorry, I missed a few size_t literals (just grepped for 0 and 1, but there were a few more) in the unit tests. Should be fixed now. Thanks for testing this so thoroughly. Note that you might also have to link to Google Benchmark (not sure how if it's within vcpkg yet) to build all the benchmarks.
Hi @rhalbersma, I installed Google Benchmark with vcpkg, it is in vcpkg, you can see https://github.com/microsoft/vcpkg/blob/master/ports/benchmark/vcpkg.json, but with latest commit(0c182bb), it still failed, please see the attachment. I'm not sure if there's anything else I need to configure? cmake_build_bs2.log
At this point I'll have to bite the bullet and install msvc I think. The remaining errors in the test are benign and related to function template argument deduction in combination with named concepts and or the typename
keyword, I suspect. I have some ideas to work around it. Will take the weekend probably.
@Zhaojun-Liu I managed to successfully compile and run most of the unit tests. Only the benchmarks and the sieve tests are not compiling. MSVC doesn't grok the proxy references in all instances, and I had to write out a 2-legged range call to std::ranges::equal
to the 4-legged iterator version.
You should be able to compile and then run `ctest -E "benchmark|sieve" and get 100%. For now this is good enough. I might try and investigate the other errors later.
Hi, We build dctl with msvc on Windows, it failed due to the below errors, it looks a regression issue from https://github.com/rhalbersma/bit_set/commit/06cd028ec3c197b27a6d94a121f21ef1b819793a, can you help look this issue? Thanks.
More info: the related repos and commit I used as follows: dctl commit: de565a1 bit_set commit: c91bdd5 Tabula commit: ab6451c Xstd commit: 4a1625d
Repro steps:
Repro file: divide.txt
By the way, I found that for the bit_set repository, all commits after commit c91bdd5 cause the git clone command to fail on Windows with the following error message: