mbox series

[0/3] package/sysklogd: version bump, some fixes, and more menuconfig

Message ID 20210122154333.1927190-1-troglobit@gmail.com
Headers show
Series package/sysklogd: version bump, some fixes, and more menuconfig | expand

Message

Joachim Wiberg Jan. 22, 2021, 3:43 p.m. UTC 
Hi!

This series bumps sysklogd to the latest version, fixes an issue with
fetching from GitHub (release tarballs exist), and finally adds some
menuconfig options enabling the native logger tool and controlling
the back-off time for unreachable remote syslog servers.

Cheers
 /Joachim

Joachim Wiberg (3):
  package/sysklogd: bump to verson 2.2.0
  package/sysklogd: add menuconfig support for enabling native logger
  package/sysklogd: menuconfig support for remote syslog retry delay

 ...RCE_required_for_O_CLOEXEC_on_uClibc.patch | 44 -------------------
 package/sysklogd/Config.in                    | 30 +++++++++++++
 package/sysklogd/sysklogd.hash                |  2 +-
 package/sysklogd/sysklogd.mk                  | 12 ++---
 package/sysklogd/syslogd.service              |  3 +-
 5 files changed, 40 insertions(+), 51 deletions(-)
 delete mode 100644 package/sysklogd/0001-Define-_GNU_SOURCE_required_for_O_CLOEXEC_on_uClibc.patch

Comments

Yann E. MORIN Jan. 24, 2021, 9:40 a.m. UTC | #1 
Joachim, All,

On 2021-01-22 16:43 +0100, Joachim Wiberg spake thusly:
> Hi!
> 
> This series bumps sysklogd to the latest version, fixes an issue with
> fetching from GitHub (release tarballs exist), and finally adds some
> menuconfig options enabling the native logger tool and controlling
> the back-off time for unreachable remote syslog servers.
> 
> Cheers
>  /Joachim
> 
> Joachim Wiberg (3):
>   package/sysklogd: bump to verson 2.2.0
>   package/sysklogd: add menuconfig support for enabling native logger
>   package/sysklogd: menuconfig support for remote syslog retry delay

All three patches applied to master, thanks.

Please review my comments and rework on patch 1. I also slightly
reworked patch 2 to use the more traditional ifew-else-endif block
rather than the $(if) construct.

Thanks!

Regards,
Yann E. MORIN.

>  ...RCE_required_for_O_CLOEXEC_on_uClibc.patch | 44 -------------------
>  package/sysklogd/Config.in                    | 30 +++++++++++++
>  package/sysklogd/sysklogd.hash                |  2 +-
>  package/sysklogd/sysklogd.mk                  | 12 ++---
>  package/sysklogd/syslogd.service              |  3 +-
>  5 files changed, 40 insertions(+), 51 deletions(-)
>  delete mode 100644 package/sysklogd/0001-Define-_GNU_SOURCE_required_for_O_CLOEXEC_on_uClibc.patch
> 
> -- 
> 2.25.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Joachim Wiberg Jan. 24, 2021, 3:25 p.m. UTC | #2 
On Sun, Jan 24, 2021 at 10:40, "Yann E. MORIN" <yann.morin.1998@free.fr> wrote:
> On 2021-01-22 16:43 +0100, Joachim Wiberg spake thusly:
>>   package/sysklogd: bump to verson 2.2.0
>>   package/sysklogd: add menuconfig support for enabling native logger
>>   package/sysklogd: menuconfig support for remote syslog retry delay
> All three patches applied to master, thanks.
> Please review my comments and rework on patch 1. I also slightly
> reworked patch 2 to use the more traditional ifew-else-endif block
> rather than the $(if) construct.

Looks good, much better with your rework.  Only issue, as I mentioned in
the previous thread, is the install path for the logger tool, which I'm
posting another patch for.

Thank you! :)

Regards
 /Joachim