radvd-project / radvd

radvd | Official repository: https://github.com/radvd-project/radvd
https://radvd.litech.org/
Other
203 stars 106 forks source link

Support blank config files #224

Closed audreylace closed 3 months ago

audreylace commented 11 months ago

I found it odd that radvd considers blank files to be invalid. Main use case for empty files is to allow other services in the system to simply write changes to the configuration file and simply tell radvd reload. Right now they can do this but it requires re-starting radvd as the process quits out.

Suggested changes, do know this would require fixing the poll bug.

Sample changes: https://github.com/radvd-project/radvd/compare/master...audreylace:radvd-stinput:support-blank-conffile?expand=1

audreylace commented 11 months ago

Few places that invoke scripts that could be used to dynamically change and reload the configuration file:

robbat2 commented 11 months ago

Agree on the bug; not sure about your branch, it still exits if the file is empty?

Neustradamus commented 11 months ago

@audreylace: Have you seen @robbat2 comment?