Created attachment 194084 [details]
svn diff from /usr/ports
Instructions at http://ipxe.org/crypto didn't work. First I tried patching the software to work with the base csplit but for the life of me I could not get it to work right so I just went ahead and added the dependency on gcsplit which works fabulously.
Also bumped the version to the latest.
Created attachment 194110 [details]
I can confirm the csplit problem, but it makes no sense to add a
coreutils build dependency to the ISO option. The certificate is
embdedded into every image net/ipxe builds not only the ISO.
We also only need this when CERT= is set in MAKE_ARGS, so let's
only add it in that case. Not everyone needs it.
Please try with the attached patch instead.
Yeah, that line being added only within ISO_* was an oversight and mistake. Your make-fu is much nicer with the conditional dependency; that's exactly what I wanted to do but I wasn't sure how. I was also unsure whether it was CERT or TRUST which triggered csplit, as I am using both. I've since tested your patch every different way I know how and can report it seems perfect to me. (I see you didn't need the extra patch-Makefile and I got rid of it first too.)
Thank you so much for the super quick turnaround on this, maintainer!
A commit references this bug:
Date: Sun Jun 10 10:02:56 UTC 2018
New revision: 472124
net/ipxe: Update to latest commit
Pull in gcsplit from sysutils/coreutils when users try to embed
certificates via CERT= in MAKE_ARGS. This currently fails because
iPXE expects csplit to behave like GNU csplit and uses constructs
that our csplit does not support.
Submitted by: Chad Jacob Milios <firstname.lastname@example.org> (based on)