Skip to content

Conversation

@tuhaihe
Copy link
Member

@tuhaihe tuhaihe commented Oct 20, 2025

libssh2-devel is introduced as a dependency package for the new feature in the PR #1226.

libssh2-devel is included in rocky-release-hpc, so we need to rocky-release-hpc before installing libssh2-devel:

dnf install rocky-release-hpc
dnf install libssh2-devel

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


Copy link
Contributor

@leborchuk leborchuk left a comment

Choose a reason for hiding this comment

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

I should also take it into account in #1378

libssh2-devel is introduced as a dependency package for the new feature
in the PR apache#1226.

libssh2-devel is included in rocky-release-hpc, so we need to
rocky-release-hpc before installing libssh2-devel:

```
dnf install rocky-release-hpc
dnf install libssh2-devel
```
Copy link

@xtangcode xtangcode left a comment

Choose a reason for hiding this comment

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

After installing rocky-release-hpc, it’s good practice to refresh DNF’s metadata to ensure the new repository is recognized immediately. Consider adding "dnf clean all && dnf makecache" between the two commands.

@tuhaihe
Copy link
Member Author

tuhaihe commented Oct 21, 2025

After installing rocky-release-hpc, it’s good practice to refresh DNF’s metadata to ensure the new repository is recognized immediately. Consider adding "dnf clean all && dnf makecache" between the two commands.

Hi @xtangcode thanks for your feedback! These commands have been in the file. So there is no need to add them again.

@tuhaihe tuhaihe merged commit 85c2f26 into apache:main Oct 21, 2025
49 of 50 checks passed
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