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

Error compiling - hash<std::filesystem::path #229

Closed
drtebi opened this issue May 27, 2022 · 4 comments
Closed

Error compiling - hash<std::filesystem::path #229

drtebi opened this issue May 27, 2022 · 4 comments
Labels
Milestone

Comments

@drtebi
Copy link

drtebi commented May 27, 2022

Unfortunately I am getting an error when compiling on Arch Linux (freshly upgraded system):
In file included from /home/drtebi/Downloads/lms-git/src/lms-git/src/libs/utils/impl/Path.cpp:20: /home/drtebi/Downloads/lms-git/src/lms-git/src/libs/utils/include/utils/Path.hpp:44:16: error: redefinition of ‘struct std::hash<std::filesystem::__cxx11::path>’ 44 | struct hash<std::filesystem::path> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from /usr/include/c++/12.1.0/filesystem:45, from /home/drtebi/Downloads/lms-git/src/lms-git/src/libs/utils/include/utils/Path.hpp:22: /usr/include/c++/12.1.0/bits/fs_path.h:1422:10: note: previous definition of ‘struct std::hash<std::filesystem::__cxx11::path>’ 1422 | struct hash<filesystem::path> | ^~~~~~~~~~~~~~~~~~~~~~ make[2]: *** [src/libs/utils/CMakeFiles/lmsutils.dir/build.make:202: src/libs/utils/CMakeFiles/lmsutils.dir/impl/Path.cpp.o] Error 1 make[1]: *** [CMakeFiles/Makefile2:1656: src/libs/utils/CMakeFiles/lmsutils.dir/all] Error 2 make: *** [Makefile:146: all] Error 2

I thought it might be a new issues and tried installing 3.28 from source, but there I am getting the same error.

gcc/g++ version 12.1.0

@epoupon
Copy link
Owner

epoupon commented May 27, 2022

Thanks for reporting this, I thought it was already corrected...

@epoupon epoupon added the bug label May 27, 2022
@epoupon epoupon added this to the v3.29.1 milestone May 27, 2022
@epoupon
Copy link
Owner

epoupon commented May 28, 2022

Should be fine on develop branch, will make a hotfix soon if you can wait

@epoupon epoupon closed this as completed May 28, 2022
@drtebi
Copy link
Author

drtebi commented May 28, 2022

Fantastic. I will probably try out the develop branch, why not... Thanks for the quick fix.

@epoupon
Copy link
Owner

epoupon commented May 28, 2022

(Now integrated in master/3.29.1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants