It has been observed that the target config-recursive for emulators/i386-wine-devel can cause bsd.port.mk to be included twice when run in i386 jails. This seems to be due to the fact that config-recursive ignores PKGNAMEPREFIX definition, which is used in emulators/wine-devel to test wether bsd.port.[pre|post.]mk needs to be included or bsd.port.mk has already been included by i386-wine-devel. We change the testing variable to WINE_SLAVE_BUILD, defined in i386-wine-devel/Makefile.i386.
- Run successfully make config-recursive in i386 jails.
- Run successfully poudriere options -a i386 i386-wine-devel.
- Build i386-wine-devel and wine-devel ports successfully on poudriere for all supported architectures (i386, amd64).
Here is what I wrote yesterday, but failed to hit "Submit":
Checking for _POSTMKINCLUDED looks like using a deep implementation detail - in general, variables whose name starts with an underscore indicate that. Is there another way you can think of? Perhaps one that obviates this
Here is what I write today (and I shall hit "Submit" ;-):
Approved by: gerald (maintainer, mentor)
Yes, and I hereby pre-approve (as both maintainer and mentor). Let's first get i386-wine-devel updated and out there for a week or two, and await any potential feedback from that end before backporting this change (and further ones to i386-wine-devel itself).