CESNET / netopeer2

NETCONF toolset
BSD 3-Clause "New" or "Revised" License
301 stars 189 forks source link

Add support for running netopeer2-server in foreground with syslog #1405

Closed troglobit closed 1 year ago

troglobit commented 1 year ago

This patch adds support for running netopeer2-server under init systems like Finit, which supervise processes running in the foreground. Similar to -d, but still using syslog.

troglobit commented 1 year ago

Will do, thanks for the swift review!

troglobit commented 1 year ago

Okay, this can be merged after some changes. Also, change the base branch to devel, we do not accept PRs to master.

Fixed.

troglobit commented 1 year ago

Almost there but I have realized that you should probably also change the man page doc/netopeer2-server.8.

Of course, will fix that too. New update coming later tonight (CET).

troglobit commented 1 year ago

There, finally. This last changeset actually doesn't look half bad. Note, I decided to move the case and group it closer to -d, which the new option has a lot in common with. Because I noticed all other non-argument options were sorted first.

troglobit commented 1 year ago

Right, code changes are really simple now so just this one last detail and it should be good to go.

Aaaaand done :smiley::+1: