Created attachment 174049 [details]
regenerated patch file
I've adjusted the Tunnelbick XOR patch to compile with the new version. Attached is the full patch to overwrite the current file. The reference commit in the OPNsense ports is below.
A commit references this bug:
Date: Sat Aug 27 01:17:24 UTC 2016
New revision: 420956
Update Tunnelblick XOR patch.
Submitted by: Franco Fichtner
Triggered by an inquiry of mat@ why I bumped PORTREVISION:
it appears the attached regenerated patch and the Github commite are not the same, so I am reopening bug, have requested Franco's clarification by e-mail.
I don't completely understand the question.
First of all the revision bump wasn't necessary because the build for the TUNNELBLICK option wasn't working in the first place. It's not in this diff.
Second of all mat@ fixed this (thanks) hours after I opened this issue. It's an unfortunate development of proactive committing of a non-maintainer on a non-default option.
As for a clarification, https://tunnelblick.net/cOpenvpn_xorpatch.html does not have an updated patch, so I manually rebuilt the patch using "makepatch" and reassembled the file. As you can see from the diff, the only conflict from 2.3.11 to 2.3.12 was the removal of the maxsize argument.
The question I have is how mat@ found this and whether or not TUNNELBLICK could be a default option given the apparent interest in this. ;)
Tunnelblick will not be a FreeBSD port default option unless upstream turns it into one.
If mat@ forgets he has committed without letting me know, this is a non-issue. Closing again, and Mathieu now needs to cope with not getting proper traces here or heads-ups to me that he changed my port... ;-)
Fair enough, thanks. :)