search

linuxserver / docker-syslog-ng

GNU General Public License v3.0
51 stars 9 forks source link

Add the paho-mqtt-c library as required by the syslog-ng documentation #13

Closed mmattel closed 1 year ago

mmattel commented 1 year ago

linuxserver.io

Description:

Referencing: Prerequisites to using the mqtt() destination from the syslog-ng documentation.

Benefits of this PR and context:

By adding the required library referenced by syslog-ng documentation, no additional container tweaking is neccessary.

How Has This Been Tested?

Source / References:

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/syslog-ng/3.38.1-r0-pkg-5f07a0ec-pr-13/index.html https://ci-tests.linuxserver.io/lspipepr/syslog-ng/3.38.1-r0-pkg-5f07a0ec-pr-13/shellcheck-result.xml

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/syslog-ng/3.38.1-r0-pkg-5f07a0ec-pr-13/index.html https://ci-tests.linuxserver.io/lspipepr/syslog-ng/3.38.1-r0-pkg-5f07a0ec-pr-13/shellcheck-result.xml

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/syslog-ng/3.38.1-r0-pkg-5f07a0ec-pr-13/index.html https://ci-tests.linuxserver.io/lspipepr/syslog-ng/3.38.1-r0-pkg-5f07a0ec-pr-13/shellcheck-result.xml

mmattel commented 1 year ago

The sort order is fixed now though it was not correct before this PR... Note that I have double checked if there is a syslog-ng package possibly missing related to mqtt. The command used was :apk list -a | grep syslog-ng and had no mqtt related results. Changing this PR into review state.

@Roxedus could you re-review?