I have added an entry to Changes.rst, as requested. I have changed the // comment in openvpn/manage.h to /* */ as pointed out by Matthias.
Code-wise, this is fairly straightforward (as it's mostly just "code inside #ifdef"), except for my nicely fixed-up plugin code :-) - but that one is good, too (and even had a comment on why it's "tls_final()"). NOTE: multi_process_incoming_link() now has an extra set of {} with indentation, due to the old construct of having an "else" as the last statement in the #endif (around line 3200) - we should fix this with a followup patch. I have lightly tested this server/client side - I have no setup that actually makes use of pf, so nothing breaks :-) - but all the rest seems to work just fine. Your patch has been applied to the master branch. commit 1308ccec9f85418b088723bb870855c0a933311b Author: Antonio Quartulli Date: Fri Aug 27 21:00:14 2021 +0200 Remove support for PF (Packet Filter) Signed-off-by: Antonio Quartulli <a...@unstable.cc> Acked-by: Arne Schwabe <a...@rfc2549.org> Message-Id: <20210827190014.12640-...@unstable.cc> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg22780.html Signed-off-by: Gert Doering <g...@greenie.muc.de> -- kind regards, Gert Doering _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel