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

All Supported OSes for clamav milter #32

Open
edestecd opened this issue Jan 14, 2017 · 6 comments · May be fixed by #48
Open

All Supported OSes for clamav milter #32

edestecd opened this issue Jan 14, 2017 · 6 comments · May be fixed by #48

Comments

@edestecd
Copy link
Owner

No description provided.

@ubellavance
Copy link

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

@ubellavance
Copy link

@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
Copy link

@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
Copy link
Owner Author

edestecd commented Jul 28, 2017

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
Copy link

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
Copy link

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

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 a pull request may close this issue.

2 participants