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

Fix: added PostgreSQL 16 to the list of known versions in FindPostgre… #2181

Merged
merged 2 commits into from
May 2, 2024
Merged

Fix: added PostgreSQL 16 to the list of known versions in FindPostgre… #2181

merged 2 commits into from
May 2, 2024

Conversation

easymoney322
Copy link
Contributor

…SQL.cmake

The gvm compiles only when used with PostgreSQL (Debian 16.2-1). The older version of PSQL (PSQL 15) isn't in Kali's rolling repo anymore.

What

This fixes cmake when compiling from source on Kali 2024.1, as the older version of PSQL no longer present in the APT's repository. Without it, cmake isn't able to find the PSQL-16. After the change was made, I was able to compile gvmd.

Why

PSQL-16 was released at 2023-09-14, and its only a matter of time untill it will become default version on LTS builds also.
You also can't use the apt-package version of the greenborne on kali, as it's broken, so compiling seems to be the only way.
The fix doesn't seem to break the build process*, so I'd say suggest bumping the version.

Tests

The only thing to consider, is that I was following the guide, that seems to download source of an older version - 23.2.0

@easymoney322 easymoney322 requested a review from a team as a code owner April 25, 2024 15:12
Copy link

github-actions bot commented Apr 25, 2024

Conventional Commits Report

Type Number
Bug Fixes 2

🚀 Conventional commits found.

easymoney322 and others added 2 commits May 2, 2024 08:58
…SQL.cmake

The gvm compiles only when used with PostgreSQL (Debian 16.2-1). The older version of PSQL (PSQL 15) isn't in Kali's rolling repo anymore.
@bjoernricks bjoernricks enabled auto-merge (squash) May 2, 2024 07:00
@bjoernricks bjoernricks disabled auto-merge May 2, 2024 07:03
@bjoernricks bjoernricks merged commit 6373712 into greenbone:main May 2, 2024
4 of 8 checks passed
@easymoney322 easymoney322 deleted the patch-1 branch May 2, 2024 12:53
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