Bug 213597 - net/netatalk3: rc.d script incorrectly substituted after r423537
Summary: net/netatalk3: rc.d script incorrectly substituted after r423537
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Joe Marcus Clarke
Depends on:
Reported: 2016-10-18 17:56 UTC by Dimitry Andric
Modified: 2016-10-18 18:31 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (marcus)


Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer 2016-10-18 17:56:26 UTC
In r423537 net/netatalk3 was updated to use the new options framework (see also bug 213192).  Unfortunately, the update has broken the substitution of %%ZEROCONF%% in the rc.d script:

$ make showconfig
===> The following configuration options are available for netatalk3-3.1.10,1:
     ACL=off: Enable ACL support
     DBUS=off: D-Bus IPC system support
     KERBEROS=off: Kerberos support
     KERBEROS5=off: Kerberos 5 support
     LDAP=off: LDAP protocol support
     LIBWRAP=off: TCP wrapper support
     MYSQL=off: Enable MySQL CNID backend support
     PAM=off: Pluggable authentication module support
     SENDFILE=off: Enable Sendfile support
====> Zeroconf(Bonjour) support: you can only select none or one of them
     AVAHI=off: Zeroconf support via Avahi
     MDNSRESPONDER=on: Zeroconf support via mDNSResponder
===> Use 'make config' to modify these settings
$ make
... build build build ...
===> Staging rc.d startup script(s)
$ head /wkrdirs/usr/ports/net/netatalk3/work/stage/usr/local/etc/rc.d/netatalk
# $FreeBSD: head/net/netatalk3/files/netatalk.in 398560 2015-10-04 01:23:32Z marcus $
# PROVIDE: netatalk
# KEYWORD: shutdown
# AFP fileserver for Mac clients.  Add the following to /etc/rc.conf to
# enable:

I also tried with ZEROCONF completely off, but it results in the same issue.

However, I am unsure how to get the functionality of the earlier Makefile, which had:

SUB_LIST+=              ZEROCONF=""

in case of zeroconf being turned off,

SUB_LIST+=              ZEROCONF="avahi_daemon"

in case of avahi, and

SUB_LIST+=              ZEROCONF="mdnsd"

in case of mdnsresponder.
Comment 1 commit-hook freebsd_committer 2016-10-18 18:30:28 UTC
A commit references this bug:

Author: marcus
Date: Tue Oct 18 18:30:24 UTC 2016
New revision: 424199
URL: https://svnweb.freebsd.org/changeset/ports/424199

  * Properly expand %%ZEROCONF%% in the rc.d script
  * While here, ensure zeroconf support is disabled at configure time if neither Avahi or mDNSResponder
    has been chosen

  PR:		213597

Comment 2 Joe Marcus Clarke freebsd_committer 2016-10-18 18:31:11 UTC
Fixed in 3.1.0_2, as well as making sure to disable Zeroconf support is so requested.