Skip to content

docs: Add dnf installation step#206

Open
gursewak1997 wants to merge 1 commit intomainfrom
add-bcvk-installation-docs
Open

docs: Add dnf installation step#206
gursewak1997 wants to merge 1 commit intomainfrom
add-bcvk-installation-docs

Conversation

@gursewak1997
Copy link
Collaborator

Now that we have bcvk in testing/stable for Fedora 42+ and EPEL9/10, we can add the installation process with dnf.
Package: https://koji.fedoraproject.org/koji/packageinfo?packageID=43081

Signed-off-by: gursewak1997 <gursmangat@gmail.com>
@gursewak1997 gursewak1997 enabled auto-merge (squash) February 12, 2026 22:57
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a helpful section to the installation documentation for users who can install bcvk via dnf. My review includes a suggestion to add a note about runtime prerequisites to ensure users have a smooth setup process.

Comment on lines +3 to +10
## From Packages

bcvk is available in Fedora 42+ and EPEL 9/10:

```bash
sudo dnf install bcvk
```

Choose a reason for hiding this comment

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

medium

This is a great addition to the documentation. To improve the user experience, it would be helpful to explicitly mention that there are runtime prerequisites that need to be installed separately, as the dnf package does not seem to include them as dependencies. This will help prevent users from running into issues after installation.

Suggested change
## From Packages
bcvk is available in Fedora 42+ and EPEL 9/10:
```bash
sudo dnf install bcvk
```
## From Packages
bcvk is available in Fedora 42+ and EPEL 9/10:
```bash
sudo dnf install bcvk

After installing, ensure you have the necessary runtime prerequisites like QEMU/KVM and Podman.

Copy link
Collaborator

Choose a reason for hiding this comment

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

After installing, ensure you have the necessary runtime prerequisites like QEMU/KVM and Podman.

Well we messed this up, we should add those to the spec file...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah well, open a new PR for that:/

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.

2 participants