Sat, Mar 30
Mar 19 2024
Mar 16 2024
Feb 27 2024
thanks for the feedback!
Feb 20 2024
That's the way how DragonFlyBSD devs solved the problem
https://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/451640b7cf6bcf7826b901ac9a51647442adb96b
Feb 18 2024
Corrected line break.
Correcting section reference to Sx instead of Cm.
Feb 16 2024
Feb 7 2024
Jan 16 2024
Resigning from this; I tried to provide feedback over IRC, but that was seemingly not well-received (and questions unanswered) and I'm not interested in reviewing this as-is. I'd much prefer splitting it into two scripts, one with, e.g., verbs, that manages wireguard interfaces and then the rc script that simply drives that in an obvious way. The last objection I heard was that there's too much state to pass around, but it's not at all clear why unless this is trying to mix way too much rc.conf configuration in with wg config.
Jan 15 2024
Dec 29 2023
Dec 12 2023
No manual page to review, yet manpages is a group reviewer. Did a file get accidentally left out?
Oct 16 2023
This is my first review here, I hope to not be stepping on any toes. If I have, please correct me so I might do better next time. Overall your wireguard startup mechanism looks good and my suggestions are strictly cosmetic in nature. I would not be at all disappointed if this patch were committed unaltered.
Sep 26 2023
address @yurpv's feedback.
Sep 25 2023
Sep 24 2023
LGTM but I would like to know the opinion of an src committer before this lands.
Sep 22 2023
i think this is ready to land now.
Sep 21 2023
Sep 19 2023
add more reviewers
Sep 18 2023
address review from @andrew_tao173.riddles.org.uk
Sep 16 2023
Sep 15 2023
Sep 14 2023
thank you for the review @pstef!
This update addresses your comments.
Sep 13 2023
Aug 31 2023
Please _mail_ me the proper patch, github provides something broken like From: =?UTF-8?q?Mina=20Gali=C4=87?= <freebsd@igalic.co>.
Also there needs to be 'Differential revision' and 'MFC' taglines.
note that i don't have commit, so i can't commit / MFC this.