On Mon Dec 18 16:33:20 2017, tmark wrote: > Ticket is ready for review. => the code fix is OK even it will be fine IMHO to change the no_dhcpd_xxx into have_dhcpd_xxx to reflect their real meanings. I have some concerns about the manual: - if by path you mean the result of dirname you should add "directory", i.e.: the absolute path and file name -> the absolute directory path and file name Note this stays against usage: this is named the absolute path. - in the precedence: lease-file-name -> lease-file-name configuration file statement -lf flag -> -lf command line flag - the requirement about the path can't be understood. More, I don't think it is true: the only constraint is the statement value must be an absolute path so it can be evaluated and opened after the chdir("/").