Bug 194678 - mail/policyd2 startup script throws warning
Summary: mail/policyd2 startup script throws warning
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: John Marino
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-29 08:50 UTC by reinier
Modified: 2014-11-13 22:03 UTC (History)
1 user (show)

See Also:


Attachments
Patch to fix this (300 bytes, patch)
2014-10-29 08:50 UTC, reinier
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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!