Bug 230132 - shells/jailkit: fix plist if procmail installed (add PROCMAIL option)
Summary: shells/jailkit: fix plist if procmail installed (add PROCMAIL option)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Tobias Kortkamp
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-29 04:21 UTC by John Hein
Modified: 2018-11-15 08:56 UTC (History)
0 users

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


Attachments
[patch] add PROCMAIL option (1.13 KB, patch)
2018-07-29 04:21 UTC, John Hein
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Hein 2018-07-29 04:21:27 UTC
Created attachment 195575 [details]
[patch] add PROCMAIL option

If you try 'make stage check-plist' for shells/jailkit and mail/procmail is already installed, you will get:

===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: man/man8/jk_procmailwrapper.8.gz
Error: Orphaned: sbin/jk_procmailwrapper
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.


The configure script detects the procmail presence and builds the procmail wrapper which is not in the plist.

The attached patch adds a PROCMAIL option (default off) which explicitly disables the procmail wrapper (if the option is off) or registers a dependency on procmail (if on).
Comment 1 commit-hook freebsd_committer freebsd_triage 2018-11-15 08:56:30 UTC
A commit references this bug:

Author: tobik
Date: Thu Nov 15 08:56:15 UTC 2018
New revision: 485007
URL: https://svnweb.freebsd.org/changeset/ports/485007

Log:
  shells/jailkit: Avoid accidentally picking up procmail

  When configure sees procmail in the environment building of
  jk_procmailwrapper is enabled.  Add a new non-default PROCMAIL
  option to make it explicit.

  PR:		230132
  Submitted by:	John Hein <z7dr6ut7gs@snkmail.com>
  Approved by:	woodsb02 (maintainer timeout, 3.5 months)

Changes:
  head/shells/jailkit/Makefile
  head/shells/jailkit/pkg-plist