Created attachment 145764 [details] patch for security/bro The default bro user does not have a valid shell so this pkg-plist line doesn't work: %%BROCTL%%@exec su %%BROUSER%% -c '%D/bin/broctl install' It fails with this error: ===> Creating users and/or groups. Using existing group 'bro'. Using existing user 'bro'. [2/2] Reinstalling bro-2.3_1: 100% This account is currently not available. The patch adds -fm to su. It also removes some duplicate pkg-plist lines.
Comment on attachment 145764 [details] patch for security/bro Here's a redports build log for this patch: https://redports.org/~leres/20140814025522-50812-233864/bro-2.3_2.log
FYI: =======================<phase: check-sanity >============================ /!\ bro-2.3_2: Makefile warnings, please consider fixing /!\ USE_PYTHON is deprecated, please use USES=python
please submit a new patch fixing these errors (verify with make check-plist) =========================================================================== ====> Running Q/A tests (stage-qa) ====> Checking for pkg-plist issues (check-plist) ===> Parsing plist ===> Checking for items in STAGEDIR missing from pkg-plist Error: Orphaned: %%ETCDIR%%ctl.cfg Error: Orphaned: etc/networks.cfg Error: Orphaned: etc/node.cfg ===> Checking for directories owned by MTREEs ===> Checking for directories handled by dependencies ===> Checking for items in pkg-plist which are not in STAGEDIR ===> Error: Plist issues found. *** Error code 1 Stop. make: stopped in /usr/ports/security/bro While you are there, you can fix sanity-check warning too I guess.
Created attachment 145793 [details] patch for security/bro Revised patch that fixes: - USE_PYTHON - make check-plist
Comment on attachment 145793 [details] patch for security/bro Also, redports log: https://redports.org/~leres/20140814230414-81772-234103/bro-2.3_2.log
A commit references this bug: Author: marino Date: Fri Aug 15 00:05:18 UTC 2014 New revision: 364876 URL: http://svnweb.freebsd.org/changeset/ports/364876 Log: security/bro: Add su flags so pkg initialization works PR: 192646 Submitted by: maintainer (Craig Leres) Changes: head/security/bro/Makefile head/security/bro/pkg-plist
Thanks!