abroekhuis commented on a change in pull request #178: Updates the building info from ubuntu 14 to 18.04 URL: https://github.com/apache/celix/pull/178#discussion_r399699873
########## File path: documents/building/README.md ########## @@ -48,16 +46,26 @@ The following packages (libraries + headers) should be installed on your system: For debian based systems (apt), the following command should work: ```bash +#required for celix framework and default bundles sudo apt-get install -yq --no-install-recommends \ - build-essential \ - curl \ - git \ - libjansson-dev \ - libcurl4-openssl-dev \ - java \ - cmake \ - libffi-dev \ - libxml2-dev + build-essential \ + curl \ + uuid-dev \ + git \ + libjansson-dev \ + libcurl4-openssl-dev \ + default-jdk \ + cmake \ + libffi-dev \ + libxml2-dev + +#required if the ZMQ PubSubAdmin option (BUILD_PUBSUB_PSA_ZMQ) is enabled +sudo apt-get install -yq --no-install-recommends \ + libczmq-dev + +#required if the ENABLE_TESTING option is enabled +sudo apt-get install -yq --no-install-recommends \ + libcpputest-dev Review comment: Is the current solution for google test the desired one? Or should we use an installed version there as well? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
