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.
propogated by copy and paste.
1. Primarily the "empty variable" default assignment, which is mostly
${name}_flags="", but fix a few others as well.
2. Where they are not already documented, add the existence of the _flags
(or other deleted empties) option to the comments, and in some cases add
comments from scratch.
3. Replace things that look like:
prefix=%%PREFIX%%
command=${prefix}/sbin/foo
to just use %%PREFIX%%. In many cases the $prefix variable is only used
once, and in some cases it is not used at all.
4. In a few cases remove ${name}_flags from command_args
5. Remove a long-stale comment about putting the port's rc.d script in
/etc/rc.d (which is no longer necessary).
No PORTREVISION bumps because all of these changes are noops.
- While I'm here, remove extra empty line in distinfo
PR: ports/113383
Submitted by: rafan
Approved by: VANHULLEBUS Yvan <yvan.vanhullebus at netasq.com> (maintainer)
- introduce OPTIONS to enable/disable features
- add more features to the OPTION dialog
- choose reasonable defaults for OPTIONS (disabled patented stuff)
- remove usesless WRKSRC line
- move LDFLAGS to the place where it is necessary
- extend CONFIGURE_ARGS to set the directory for the adminport socket
* Note: racoonctl is useless without adminport enabled
* create the socket dir in post-install
- bump PORTREVISION that users notice the changes
- finally: remove one item from the TODO list on top of the Makefile ;)
pkg-descr:
- shortened by one line to please portlint
pkg-plist:
- slight reorganization
- add create instructions for the adminport socket dir for the
"installing from package" case
- add removal instructions for the adminport socket dir
files/patch-configure:
- add some stuff to detect libiconv when the GSSAPI OPTION is enabled
- iconv is required for GSSAPI but the library is not added to LIBS
in the original configure script
- Why not use USE_AUTOTOOLS and patch configure.ac?
* because the leads to configure running twice which is useless
overhead. I found no way to avoid this so we patch configure.
files/racoon.sh.in:
- add the adminsocket directory to required_dirs
PR: ports/96368
Submitted by: Joerg Pulz <Joerg.Pulz@frm2.tum.de>
Approved by: maintainer
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.
ports/security/ipsec-tools rc.d script defaults to 'enabled'
It also installs its own versions of setkey and libipsec.so
which seems redundant as they are part of the base system
and should be used in preference.
Submitted by: Vivek Khera <vivek@khera.org>
PR: ports/91317
bringing it in line with the old security/racoon port and the handbook [1]
- Make use of USE_RC_SUBR instead of home-grown substitution and install
- Prevent installation of some intermediate sample configuration files
PR: ports/89273 [1]
Submitted by: Angelo Turetta <aturetta@bestunion.it> [1]
Approved by: VANHULLEBUS Yvan <vanhu@netasq.com> (maintainer)
is the only one which is maintained and have lots of new features.
PR: 85544
Submitted by: VANHULLEBUS Yvan <vanhu@netasq.com>
Approved by: perky (mentor)