edestecd / puppet-clamav

Puppet Module for managing ClamAV
GNU General Public License v3.0
6 stars 59 forks source link

Hardcode clamav depenencies #57

Closed domgoodwin closed 9 months ago

domgoodwin commented 6 years ago

There is an issue with the clamav rpm package 0.99 in EPEL where the clamav-data dependency isn't hard coded. This means when puppet installs clamav v0.99 if it's using a repo with the new clamav v0.100 there will be mismatches between versions.

These changes add a dependencies class which install file system > data > lib using the param clamav_version so this issue isn't encountered.

edestecd commented 6 years ago

could you just set clamav_package?

class { 'clamav':
  manage_clamd   => true,
  clamav_package => ['clamav-filesystem'', 'clamav-data', 'clamav-lib'],
}

What are you setting clamav_version to ?

Maybe we can check for this case in params and just switch out the packages?

edestecd commented 6 years ago

Also what os are you using? CentOS 7 ?

domgoodwin commented 6 years ago

Setting clamav_package doesn't seem to work.

clamav_version is: 0.99.2-1.el7

Yes, CentOS7