Created attachment 172035 [details]
Simple patch to fix the problem
Error found when X11 option is enabled:
====> Running Q/A tests (stage-qa)
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: libexec/fetchmailconf.py
===> Error: Plist issues found.
*** Error code 1
make: stopped in /usr/ports/mail/fetchmail
====>> Error: check-plist failures detected
Created attachment 172444 [details]
Submitter's patch fixes the case where X11 is enabled, but causes poudriere testport failures when X11 is disabled. This version passes poudriere testport for X11 enabled as well as disabled.
Btw, at this point I'm rather confused about how this hasn't been broken with X11 *disabled* since this commit:
After that commit, libexec/fetchmailconf.py definitely wasn't in the plist, but definitely was being installed. I've re-run poudriere testport using the version from HEAD several times, including after wiping my poudriere data dir, and it's consistently there.
In any case, the previous version was wrong and the attached patch does fix it. I'd appreciate it if someone could explain it to me though. :)
(In reply to Corey Halpin from comment #2)
Your patch is correct. Thanks.
Forgetting to list installed files in pkg-plist doesn't stop the build by default, but listing non-existent files in pkg-plist does.
A commit references this bug:
Date: Sun Jul 31 14:16:56 UTC 2016
New revision: 419379
Fix pkg-plist when X11 option is on.
Submitted by: Ting-Wei Lan <email@example.com>
Approved by: Corey Halpin <firstname.lastname@example.org> (maintainer)