Hi Jan,

Jan Klemkow <j.klem...@wemelug.de> writes:

> ping
>
> On Wed, May 27, 2015 at 01:27:44AM +0200, Jan Klemkow wrote:
>> ping
>> 
>> On Sat, May 16, 2015 at 01:30:39AM +0200, Jan Klemkow wrote:
>> > hi,
>> > 
>> > this new port expense net/ucspi-tcp with TLS and SOCKSv5 support.  This
>> > version also includes regression tests.  It contains SOCKSv5 client and
>> > TLS server and client programs.
>> > 
>> > Just tell me, if any thing is wrong.  I will fix it.

I have noticed a couple of nits which I have addressed in the updated
tarball attached.  Here's the diff:

diff -pruN ucspi-tools.orig/Makefile ucspi-tools/Makefile
--- ucspi-tools.orig/Makefile   Thu Jun  4 13:41:45 2015
+++ ucspi-tools/Makefile        Thu Jun  4 13:51:41 2015
@@ -3,13 +3,11 @@
 COMMENT=       socks proxy and tls support for ucspi
 
 DISTNAME=      ucspi-tools-1.1
-REVISION=      0
 CATEGORIES=    net
 
 GH_ACCOUNT=    younix
 GH_PROJECT=    ucspi
 GH_COMMIT=     cf8c863045c83b19ce04f6b1ff5a64094ea1026e
-WRKDIST=       ${WRKDIR}/ucspi-${GH_COMMIT}
 
 HOMEPAGE=      https://github.com/younix/ucspi
 MAINTAINER=    Jan Klemkow <j.klem...@wemelug.de>
@@ -17,11 +15,10 @@ MAINTAINER= Jan Klemkow <j.klem...@wemelug.de>
 # ISC
 PERMIT_PACKAGE_CDROM=  Yes
 
-WANTLIB += c crypto ssl
+WANTLIB += c crypto ssl tls
+
 FAKE_FLAGS=    DESTDIR=""
 
 RUN_DEPENDS=   net/ucspi-tcp
-
-NO_TEST=       No
 
 .include <bsd.port.mk>
diff -pruN ucspi-tools.orig/pkg/DESCR ucspi-tools/pkg/DESCR
--- ucspi-tools.orig/pkg/DESCR  Thu Jun  4 13:41:45 2015
+++ ucspi-tools/pkg/DESCR       Thu Jun  4 13:48:42 2015
@@ -1,3 +1,3 @@
 This package contains additional tools for the UCSPI tool chains.  It brings
-support for using socks proxys on client side and tls on server and client
+support for using socks proxies on client side and tls on server and client
 side.
diff -pruN ucspi-tools.orig/pkg/PLIST ucspi-tools/pkg/PLIST
--- ucspi-tools.orig/pkg/PLIST  Thu Jun  4 13:41:45 2015
+++ ucspi-tools/pkg/PLIST       Thu Jun  4 13:49:58 2015
@@ -1,7 +1,5 @@
 @comment $OpenBSD$
 @bin bin/sockc
 @bin bin/tlsc
-man/
-man/man1/
 @man man/man1/sockc.1
 @man man/man1/tlsc.1

I think it's fine to import as is, but I'd like to point out that since
your first submission you changed the git commit without changing the
version.  Please consider using git tags and GH_TAGNAME instead, with
tags considered as fixed.  Also, this is using libtls and your first
submission didn't build anymore.  Are you ok with maintaining a port
that depends on a somewhat fast-moving target such as libtls?

Looking for opinions from other porters too.

Cheers,

Attachment: ucspi-tools.tgz
Description: Binary data

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to