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

Add libicu-dev to package requirements in native_setup.md #316

Merged
merged 1 commit into from Feb 20, 2020

Conversation

lehmann-4178656ch
Copy link
Member

Following the instructions in native_setup.md in a freshly installed ubuntu 18.04.4 server vm it is not possible to compile qlever, this PR addresses this issue by adding libicu-dev to the list of required packages, the culprit was found by looking in the Dockerfile.

Issued commands:

vm@vm:~/qlever/build$ history
    1  sudo apt install git
    2  sudo apt install build-essential cmake libsparsehash-dev
    3  sudo apt install wget python3-yaml unzip curl
    4  git clone --recursive https://github.com/ad-freiburg/QLever.git qlever
    5  cd qlever
    6  mkdir build && cd build
    7  cmake -DCMAKE_BUILD_TYPE=Release .. && make -j $(nproc)
    8  history

cmake error:

...
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE  
-- The following ICU libraries were not found:
--   uc (required)
--   i18n (required)
CMake Error at /usr/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Failed to find all ICU components (missing: ICU_INCLUDE_DIR ICU_LIBRARY
  _ICU_REQUIRED_LIBS_FOUND) (Required is at least version "60")
Call Stack (most recent call first):
  /usr/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-3.10/Modules/FindICU.cmake:298 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:32 (find_package)


-- Configuring incomplete, errors occurred!
See also "/home/vm/qlever/build/CMakeFiles/CMakeOutput.log".
See also "/home/vm/qlever/build/CMakeFiles/CMakeError.log".

After running sudo apt install libicu-dev the build succeeds.

Copy link
Member

@niklas88 niklas88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks Axel!

@niklas88 niklas88 merged commit ea66d76 into ad-freiburg:master Feb 20, 2020
@lehmann-4178656ch lehmann-4178656ch deleted the libicu-docs branch February 20, 2020 10:41
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 this pull request may close these issues.

None yet

2 participants