-
Notifications
You must be signed in to change notification settings - Fork 466
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
--with-libmongo-client option does not have auto choice. #83
Comments
Are we sure we want to accommodate distro packagers over individuals compiling syslog-ng? Wouldn't we need the same default for all submodules? Just asking, I have no objections if you have a strong opinion here. In case you do just go ahead and merge it. Thanks |
It actually helps individuals more than distro packagers (distro packagers have been using --with-libmongo-client=system since day one, pretty much), because they don't need to wonder why it's not picking up the system library. The change also makes the behaviour less confusing. And finally: promoting the use of system libraries is a Good Thing. I do agree that we should do something similar with the other submodules too, where we can. I'll submit a separate pull request that does it for ivykis. The third submodule we have is rabbitmq-c, where we don't support using the system library at all, so that'll remain as it is. |
Are we sure we want to accommodate distro packagers over individuals compiling syslog-ng? Wouldn't we need the same default for all submodules? Just asking, I have no objections if you have a strong opinion here. In case you do just go ahead and merge it. Thanks |
Make --with-libmongo-client support an "auto" setting, and make that the default. This fixes syslog-ng#83. Reported-by: Andras Mitzki <micek@balabit.hu> Signed-off-by: Gergely Nagy <algernon@balabit.hu>
And it's default parameter is internal(aka git submodule), which is not fortunate if you are a distro packager. (mostly you want to use the system libmongo-client library).
The text was updated successfully, but these errors were encountered: