Bug 194678

Summary: mail/policyd2 startup script throws warning
Product: Ports & Packages Reporter: reinier
Component: Individual Port(s)Assignee: John Marino <marino>
Status: Closed FIXED    
Severity: Affects Many People CC: marino
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch to fix this none

Description reinier 2014-10-29 08:50:07 UTC
Created attachment 148756 [details]
Patch to fix this

During building the policyd port (at least 2.0.12_2), the shebang of /usr/local/bin/cbpolicyd is rewritten to the found perl-binary (in my and probably many cases /usr/local/bin/perl). The rc-script however is statically set to /usr/bin/perl. While this is a symlink to a copy of the same version of the perl binary as, rc.subr's _find_processes throws a warning when found interpreter doesn't match the given $command_interpreter:

/usr/local/etc/rc.d/policyd2: WARNING: $command_interpreter /usr/bin/perl != /usr/local/bin/perl

Attached is a proposed fix.
Comment 1 John Marino freebsd_committer freebsd_triage 2014-11-01 09:19:03 UTC
This port is unmaintained, so I'm moving this to patch ready.  the PORTREVISION needs to be bumped too.
Comment 2 commit-hook freebsd_committer freebsd_triage 2014-11-13 21:59:58 UTC
A commit references this bug:

Author: marino
Date: Thu Nov 13 21:59:50 UTC 2014
New revision: 372542
URL: https://svnweb.freebsd.org/changeset/ports/372542

Log:
  mail/policyd2: fix $command_interpreter warning

  While here, clean up a bit.  Use @sample keyword, wrap compound commands
  with parentheses, customize WWWDIR, specify PORTDOCS explicitly, and
  use MKDIR instead of INSTALL -d.

  PR:		194678
  Submitted by:	reiner (skoef.nl)

Changes:
  head/mail/policyd2/Makefile
  head/mail/policyd2/files/policyd2.in
  head/mail/policyd2/pkg-plist
Comment 3 John Marino freebsd_committer freebsd_triage 2014-11-13 22:03:52 UTC
thanks!