Skip to content

Add: ec dependencies#3045

Merged
pascalholthaus merged 4 commits into
mainfrom
pholthaus/ec-deps
Jul 20, 2026
Merged

Add: ec dependencies#3045
pascalholthaus merged 4 commits into
mainfrom
pholthaus/ec-deps

Conversation

@pascalholthaus

@pascalholthaus pascalholthaus commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

What

Add: install-ec-dependencies.sh

Why

Add curl, graphviz and pycha.
These components are required for specific reports in the enterprise containers.

If someone knows something better as pip for pycha, please commit to this pr.

"libcurl4t64" gets updated during the curl installation; hopefully, that doesn't break any dependencies.

References

DEVOPS-2201

@pascalholthaus
pascalholthaus requested review from a team as code owners July 20, 2026 10:53
@greenbonebot
greenbonebot enabled auto-merge (rebase) July 20, 2026 10:53

@bjoernricks bjoernricks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just put into curl and graphviz into the runtime-dependencies.list if they are required for some report format. for which report format do we need pycha?

@y0urself

Copy link
Copy Markdown
Member

Just put into curl and graphviz into the runtime-dependencies.list if they are required for some report format. for which report format do we need pycha?

It looks, that pycha seems to be part of GOS ... I don't know if it is for a specific report-format.

https://github.com/greenbone/gsm-python3-pycha

When testing report formats on the OEC, it was noticed, that some libraries are missing.

@timopollmeier

timopollmeier commented Jul 20, 2026

Copy link
Copy Markdown
Member

for which report format do we need pycha?

It's used in the GSR-PDF and GXR-PDF report formats.

Comment thread .github/install-ec-dependencies.sh Outdated
Co-authored-by: Björn Ricks <bjoern.ricks@greenbone.net>
@pascalholthaus
pascalholthaus disabled auto-merge July 20, 2026 11:40
@pascalholthaus
pascalholthaus enabled auto-merge (squash) July 20, 2026 11:41
@pascalholthaus pascalholthaus changed the title Add: install-ec-dependencies.sh Add: ec dependencies Jul 20, 2026
@pascalholthaus
pascalholthaus disabled auto-merge July 20, 2026 11:41
@pascalholthaus
pascalholthaus enabled auto-merge (squash) July 20, 2026 11:42
@pascalholthaus
pascalholthaus merged commit 9016689 into main Jul 20, 2026
27 checks passed
@pascalholthaus
pascalholthaus deleted the pholthaus/ec-deps branch July 20, 2026 11: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.

4 participants