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

Update supported platforms. #50

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

jhumlick
Copy link

Remove references to CentOs, add AlmaLinux, and update versions and architectures.

CLAM-2739

Remove references to CentOs, add AlmaLinux, and update versions and architectures.

CLAM-2739
@jhumlick jhumlick added the documentation Improvements or additions to documentation label Feb 13, 2025
@jhumlick jhumlick requested a review from val-ms February 13, 2025 00:38
@jhumlick jhumlick self-assigned this Feb 13, 2025
src/Introduction.md Outdated Show resolved Hide resolved
Copy link
Contributor

@val-ms val-ms left a comment

Choose a reason for hiding this comment

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

Mostly looking good, just some minor recommendations.

@@ -355,7 +355,7 @@ Endian Firewall Community (EFW) is a turn-key Linux security distribution that c

### ClamTK | ClamAV

ClamTk is a GUI front-end for ClamAV using gtk2-perl. It is designed to be an easy-to-use, on-demand scanner for Linux systems. ClamTk has been ported to Fedora, Debian, RedHat, openSUSE, ALT Linux, Ubuntu, CentOS, Gentoo, Archlinux, Mandriva, PCLinuxOS, FreeBSD, and others.
Copy link
Contributor

Choose a reason for hiding this comment

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

I wouldn't change this one. I think ClamTK is no longer supported. I'm okay to let it be.

@@ -7,7 +7,7 @@ The following are instructions to build ClamAV *version 0.104 and newer* using C
- [Building ClamAV with CMake (v0.104 and newer)](#building-clamav-with-cmake-v0104-and-newer)
- [Install prerequisites](#install-prerequisites)
- [Alpine](#alpine)
- [Redhat / Centos / Fedora](#redhat--centos--fedora)
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't need to remove references to Redhat just because we don't do automated testing on Redhat.

Comment on lines -56 to -58
### Redhat / Centos / Fedora
### AlmaLinux / Fedora

*For RHEL 8 or Centos Stream*, you will probably need to run this to enable EPEL & PowerTools.
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above, no need to remove Redhat. IDK about mentioning Centos Stream.

I don't recall if you need to install EPEL and PowerTools for Fedora. Did you try this?

Comment on lines -110 to -112
### EPEL: Fedora, RHEL, and CentOS
### EPEL: AlmaLinux and Fedora

EPEL creates ClamAV packages for Fedora (as well as EPEL-enabled CentOS and RHEL). For more information on EPEL, [visit their wiki](https://fedoraproject.org/wiki/EPEL).
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd keep RHEL in here.


To enable EPEL for CentOS:
To enable EPEL for Fedora/AlmaLinux:
```bash
dnf install -y epel-release
Copy link
Contributor

Choose a reason for hiding this comment

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

It may be appropriate for RHEL and AlmaLinux (not sure). I just tested with fedora:latest and epel-release doesn't exist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants