We'd like to be able to use the eapol_test test program. Here's a patch to add an option for this.
Created attachment 213663 [details] patch
The other patches don't appear to be related to EAPOL_TEST. What are they for?
Maintainer approved. Fill your boots.
Comment on attachment 213663 [details] patch ^Triage: Set attachment maintainer-approval flag (to +) to signify maintainer approval.
(In reply to Cy Schubert from comment #2) I should have at least commented about the changes to files/*. Apparently turning on the CONFIG_EAPOL_TEST .config option causes additional code to be compiled resulting in errors: ../src/utils/Packet32.c:131:14: error: passing 'int8_t *' (aka 'signed char *') to parameter of type 'const char *' converts between pointers to integer types with different sign [-Werror,-Wpointer-sign] if (strncmp(iface, "\\Device\\NPF_", 12) == 0) ^~~~~ [...] ../src/drivers/driver_ndis.c:508:22: error: format specifies type 'int' but the argument has type 'size_t' (aka 'unsigned long') [-Werror,-Wformat] __func__, oid, len); ^~~ [etc.] I just retest adding the patches one at a time and it looks like the changes to Packet32.c are not required.
Thanks. Commit it without the Packet32.c patch then.
A commit references this bug: Author: leres Date: Wed Apr 22 03:37:46 UTC 2020 New revision: 532442 URL: https://svnweb.freebsd.org/changeset/ports/532442 Log: security/wpa_supplicant: Add EAPOL_TEST option Add an option option to builds and installs the eapol_test program. Update/patch to solve new compilation errors that result with the option enabled. Note: Leaving the changes to Packet32.c out of the PR patch set as it is not needed. PR: 245809 Approved by: cy (maintainer) Changes: head/security/wpa_supplicant/Makefile head/security/wpa_supplicant/files/ntddndis.h head/security/wpa_supplicant/files/patch-src_drivers_driver__ndis.c head/security/wpa_supplicant/pkg-plist
Created attachment 213700 [details] patch Here's a quick patch to change the @comment logic for the new EAPOL_TEST option to use OPTIONS_SUB instead. (I thought this feature would exist but didn't easily find it yesterday; today it occurred to me to search /usr/ports/Mk/* for @comment... and of course now that I know what it's named I see it in the porters handbook.)
Go for it.
A commit references this bug: Author: leres Date: Wed Apr 22 19:54:39 UTC 2020 New revision: 532510 URL: https://svnweb.freebsd.org/changeset/ports/532510 Log: security/wpa_supplicant: Simplify @comment logic by using OPTIONS_SUB PR: 245809 Approved by: cy (maintainer) Changes: head/security/wpa_supplicant/Makefile
This was wrapped up last month.
Indeed it was. Thanks for committing it.