On Sun, Jul 31, 2016 at 5:46 AM, Pedro Giffuni <p...@freebsd.org> wrote:
>>> Index: sbin/resolvconf/Makefile
>>> ===================================================================
>>> --- sbin/resolvconf/Makefile    (revision 303557)
>>> +++ sbin/resolvconf/Makefile    (working copy)
>>> @@ -30,6 +30,7 @@
>>>                 -e 's:@LIBEXECDIR@:${FILESDIR}:g' \
>>>                 -e 's:@VARDIR@:${VARDIR}:g' \
>>>                 -e 's:@RESTARTCMD \(.*\)@:${RESTARTCMD}:g' \
>>> +               -e 's:@RESTARTCMD@:${RESTARTCMD}:g' \
>>>                 -e 's:@RCDIR@:${RCDIR}:g' \
>>>                 -e 's: vpn : ng[0-9]*&:g' \
>>>                 ${DIST}/$@.in > $@
> And the underscore was not a typo.
> Thanks Guy!
> Pedro.


I meant for the RESTARTCMD_= statement to be added too.
I should have sent a patch file.

Issue could be seen during boot if using dhcpcd which invokes resolvconf
or by running 'resolvconf -r SERVICENAME'

With r303567, RESTARTCMD is now empty in the script so
dynamic detection for service command is done.

Attaching new patch.

Renaming RESTARTCMD, CMD1, CMD2 to _WITH_ARG since it is only
used by pdns_recursor.in.
The other files have all moved to use $RESTARTCMD passed from resolvconf.

I will send a patch upstream to do the same for pdns_recursor.in.
The ugly _WITH_ARG vars can be removed when pdns_recursor.in
is updated to new style.

Renaming RESTARTCMD_ to RESTARTCMD and adding the lines for
new CMD1, CMD2, RESTARTCMD statements.


Attachment: resolvconf_RESTARTCMD.patch
Description: Binary data

freebsd-current@freebsd.org mailing list
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Reply via email to