search

edestecd / puppet-clamav

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

All Supported OSes for clamav milter #32

Open edestecd opened 7 years ago

ubellavance commented 7 years ago

I may try to get it to work with Ubuntu, but I don't really have access to Debian machines.

ubellavance commented 7 years ago

@Yamakasi, would you like to help with Ubuntu clamav-milter support? To test the module on Ubuntu, you'd have to comment these lines out: https://github.com/edestecd/puppet-clamav/blob/master/manifests/clamav_milter.pp#L7-L9.

ubellavance commented 7 years ago

@edestecd I have made the required changes to make the clamav-milter part compatible with Ubuntu 14.04 and 16.04. Do I need to test against Debian as well before submitting a PR?

edestecd commented 7 years ago

Debian is usually the same. I assume it is until otherwise noted. Its nice if you can test on Debian, but others will find the errors later if there are any.

ubellavance commented 7 years ago

I tested it with Debian 8.2 and the milter works fine. However clamd seems to be broken as it cannot start, but it's not related to my addition of clamav-milter.

ubellavance commented 7 years ago

I also tested Debian 7 so I'll prepare a PR soon.