Skip to content

Conversation

@timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented Jan 15, 2026

What

libcrypt-dev is added to the list of build dependencies.

Why

libcrypt-dev is required to build gvm-libs.

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA d5b5c90.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@greenbonebot
Copy link
Member

greenbonebot commented Jan 15, 2026

Scanning the following files:

.github/build-dependencies.list
.github/runtime-dependencies.oldstable.list
.github/runtime-dependencies.stable.list
.github/runtime-dependencies.testing.list

Scan: '.github/build-dependencies.list'

Nothing detected in .github/build-dependencies.list
Scan took 0.00 seconds

Scan: '.github/runtime-dependencies.oldstable.list'

Nothing detected in .github/runtime-dependencies.oldstable.list
Scan took 0.00 seconds

Scan: '.github/runtime-dependencies.stable.list'

Nothing detected in .github/runtime-dependencies.stable.list
Scan took 0.00 seconds

Scan: '.github/runtime-dependencies.testing.list'

Nothing detected in .github/runtime-dependencies.testing.list
Scan took 0.00 seconds

@timopollmeier timopollmeier changed the title Fix: Add doxygen to build-dependencies.list Fix: Add doxygen and graphviz to build-dependencies.list Jan 15, 2026
@timopollmeier timopollmeier changed the title Fix: Add doxygen and graphviz to build-dependencies.list Fix: Add missing build dependencies Jan 15, 2026
Copy link
Contributor

@bjoernricks bjoernricks left a comment

Choose a reason for hiding this comment

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

Lets just get the debian testing container image build running again

Use exact package name for `libgcrypt-dev` package which is
`libgcrypt20-dev`. `libgcrypt20-dev` just *Provides* `libgcrypt-dev`.
Also require libcrypt1 because we link against it. By default it is
already installed because `passwd` depends on it. But we should ensure
that it is still installed if something in Debian will change in future.
@bjoernricks bjoernricks marked this pull request as ready for review January 16, 2026 08:47
@bjoernricks bjoernricks requested review from a team as code owners January 16, 2026 08:47
@bjoernricks bjoernricks self-requested a review January 16, 2026 08:48
@bjoernricks bjoernricks enabled auto-merge (rebase) January 16, 2026 08:49
@bjoernricks bjoernricks merged commit eef7cd8 into main Jan 16, 2026
23 checks passed
@bjoernricks bjoernricks deleted the doxygen-build-dependency branch January 16, 2026 08:49
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.

4 participants