literal name_enable wherever possible, and ${name}_enable
when it's not, to prepare for the demise of set_rcvar().
In cases where I had to hand-edit unusual instances also
modify formatting slightly to be more uniform (and in
some cases, correct). This includes adding some $FreeBSD$
tags, and most importantly moving rcvar= to right after
name= so it's clear that one is derived from the other.
We have not checked for this KEYWORD for a long time now, so this
is a complete noop, and thus no PORTREVISION bump. Removing it at
this point is mostly for pedantic reasons, and partly to avoid
perpetuating this anachronism by copy and paste to future scripts.
infastructure of openradius uses the ARCH macro in order to run ar(1).
As FreeBSD's make(1) predefines ARCH to the architecture, this broke
compilation. Fix this by (ab)using the AR macro instead.
As the maintainer timed out on fixing this by about half a year, and I
need the port right now, I take the liberty to ignore the maintainer
bit here.