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

Compilation issues under Archlinux using cmake #54

Closed
kbipinkumar opened this issue Jun 10, 2023 · 1 comment · Fixed by #58
Closed

Compilation issues under Archlinux using cmake #54

kbipinkumar opened this issue Jun 10, 2023 · 1 comment · Fixed by #58

Comments

@kbipinkumar
Copy link

i am trying to package mashmap for Archlinux distribution. using the compiling instruction with recommended cmake command as below,

$ cmake -B build \
        -DCMAKE_BUILD_TYPE='Release' \
        -DCMAKE_INSTALL_PREFIX='/usr' \
        -Wno-dev
$ cmake --build build

while running aforementioned command the compilation is failing with following error

-- The C compiler identification is GNU 13.1.1
-- The CXX compiler identification is GNU 13.1.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- CMAKE_BUILD_TYPE: Release
/build/mashmap/src/MashMap-3.0.4/scripts/generate_git_version.sh: line 3: git: command not found
-- Configuring done (0.5s)
-- Generating done (0.0s)
-- Build files have been written to: /build/mashmap/src/MashMap-3.0.4/build
[ 16%] Building CXX object CMakeFiles/mashmap.dir/src/common/utils.cpp.o
/build/mashmap/src/MashMap-3.0.4/src/common/utils.cpp: In function ‘int64_t mashmap::handy_parameter(const std::string&)’:
/build/mashmap/src/MashMap-3.0.4/src/common/utils.cpp:14:9: error: ‘uint64_t’ was not declared in this scope
   14 |         uint64_t str_len = value.length();
      |         ^~~~~~~~
compilation terminated due to -Wfatal-errors.
make[2]: *** [CMakeFiles/mashmap.dir/build.make:76: CMakeFiles/mashmap.dir/src/common/utils.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:85: CMakeFiles/mashmap.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

could someone take a look and tell what could be the issue?

bkille added a commit that referenced this issue Jul 9, 2023
@bkille bkille mentioned this issue Jul 10, 2023
@bkille
Copy link
Contributor

bkille commented Jul 10, 2023

Hi @kbipinkumar

Thanks for opening this issue! Seems like GNU 13 catches a missing #include when GNU 11 does not. I've fixed this case, but there still may be more. I'm in the process of updating our servers so that I can check with GNU 12 and 13 as well. Thanks again for letting us know about this.

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 a pull request may close this issue.

2 participants