16:58 jpaetzel 

Remove net/socks5 from the ports tree.

There are numerous redistribution and legal issues surrounding
this piece of software.

net/socks5

08:57 dougb 

In the rc.d scripts, change assignments to rcvar to use the
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.

04:26 mi 

A long-standing issue in the vendor's reimplementation of vsnprintf()
was not only inefficient (writing out to a file and then reading back!),
but also had a bug in its handling of va_args.

The most obvious ill-effect was the daemon crashing shortly after
start-up on amd64. The inefficientcy was less obvious, but still there
on all platforms.

Although log.c contained HAVE_VSNPRINTF ifdef-s, the configure script
does not check for the function's presence and never sets the define.

Add the -DHAVE_VSNPRINTF as a simple work-around... Bump PORTREVISION.

Approved by: maintainer

While here, ditch the ancient socks5.sh.sample in favor of the proper
USE_RC_SUBR script based on the maintainer-supplied code.