Skip to content
This repository has been archived by the owner on Jun 30, 2021. It is now read-only.

Improvement syslog class #40

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

Conversation

jewelnuruddin
Copy link

Because enabling auidt log into syslog might be causing the system overhead.
So make sure if someone really want to enable this.

#39

@kemra102
Copy link
Owner

kemra102 commented May 9, 2018

I've thought about this for a while. I'm not sure I understand what you;re trying to achieve here. If you don;t want syslog, don't include it - it isn't required or included by default.

@jewelnuruddin
Copy link
Author

jewelnuruddin commented May 24, 2018

It just my opinion as I said details in issue "#39"
It seems good to avoid something hard coded define like active = true

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants