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

OpenVas-Scanner Adding information on missing package #463

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Toys0125
Copy link
Contributor

When I went over the documentation, I had the issue of missing Libcurl when trying to build from cmake. According to the online documentation, Ubuntu doesn't have a direct libcurl-dev package, and using libcurl4-openssl-dev fixes the issue.
additional post about Ubuntu 22.04 missing libcurl

What

I'm adding a warning that if libcurl is missing, here's the package if needed. It is also possible to just add the package to the requirements if the repo owner deems it necessary.

Why

It helps users when building, not having to research why libcurl is missing a package for CMake.

When I went over the documentation, I had the issue of missing Libcurl when trying to build from cmake. According to the online documentation, Ubuntu doesn't have a direct libcurl-dev package, and using libcurl4-openssl-dev fixes the issue.
@Toys0125 Toys0125 requested a review from a team as a code owner April 17, 2024 18:52
@greenbonebot greenbonebot enabled auto-merge (rebase) April 17, 2024 18:52
@bjoernricks
Copy link
Contributor

Hi, thanks for bringing this up. It seems that libcurl is a dependency that should be installed by default. Therefore we are going to update the docs for installing libcurl.

@cfi-gb
Copy link
Member

cfi-gb commented May 23, 2024

@bjoernricks @Toys0125 @greenbone/docs-maintainers It looks like this has been already included in #464 (with the libcurl4-gnutls-dev package added / available here https://greenbone.github.io/docs/latest/22.4/source-build/index.html#openvas-scanner) and is thus not required anymore.

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.

3 participants