From owner-svn-ports-all@FreeBSD.ORG Mon Jul 1 17:30:11 2013 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id B075DC24; Mon, 1 Jul 2013 17:30:11 +0000 (UTC) (envelope-from lippemail@gmail.com) Received: from mail-wi0-x22c.google.com (mail-wi0-x22c.google.com [IPv6:2a00:1450:400c:c05::22c]) by mx1.freebsd.org (Postfix) with ESMTP id C996D1784; Mon, 1 Jul 2013 17:30:10 +0000 (UTC) Received: by mail-wi0-f172.google.com with SMTP id c10so3329780wiw.5 for ; Mon, 01 Jul 2013 10:30:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; bh=UCB/2jw/V2cBIgxIL0vZKK5HuRPTLy/teyMvkjtxBfE=; b=YlRI1YDAF75K5ZMcXI5vhs5L01r5MiDQ6FZFisMdJvA0/xCW7uWB+HxPk7S2Rc51F/ SgiHPN+iNG4facMHo2u/kC+H6N6lJ6lPfzq2PGyEkla3IZnqYd2SPBhkhRg4TACFQcre YHGPo8jnMS2LDzcTPj9N6pn7i+jcYwSx/1UOjVi50q3sBIJ57C02pNj5sQjiRcBwfMnE 9QTQsh7LTd2G+RInjjrTTVbWfHcbNzPG/YDYwPagaerSQRkMxlBgNpuM4p+pMdWaZ5zb kFJ0xFha1zMw4d+3Uct+nmmoMzzMiuh3B/F1/MtfH3czLvdp7gLWAYdMGbM6Mr1EyQ9/ EAtg== MIME-Version: 1.0 X-Received: by 10.194.60.5 with SMTP id d5mr21022195wjr.26.1372699809773; Mon, 01 Jul 2013 10:30:09 -0700 (PDT) Sender: lippemail@gmail.com Received: by 10.180.7.98 with HTTP; Mon, 1 Jul 2013 10:30:09 -0700 (PDT) In-Reply-To: <20130701011908.GB9325@FreeBSD.org> References: <201306282045.r5SKjaZb090161@svn.freebsd.org> <20130701011908.GB9325@FreeBSD.org> Date: Mon, 1 Jul 2013 14:30:09 -0300 X-Google-Sender-Auth: ADpIL4t5ij3Xdtv-OxrSQi2Q0RU Message-ID: Subject: Re: svn commit: r321982 - in head/net: . hanstunnel hanstunnel/files From: Felippe de Meirelles Motta To: Alexey Dokuchaev Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.14 Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Jul 2013 17:30:11 -0000 Folks, My recent committs are the result of old and open PRs, where I tried to contact some submitters / maintainers and got no answer, so I decided to commit them the way they were. We have definitions of how to treat old PRs who already have new versions of software, changes to Makefile, any others changes, but the submitter and maintainer are no longer in contact? Sorry if I was wrong, but my thought was committ those open and apparently abandoned (for committers, maintainers and submitters) PRs. To minimize what you are characterizing as errors or failures in my committs, I'll open a PR for each of them with the necessary improvements. danfe@, bsam@ and others thanks for the points mentioned, some I had noticed, others I had really missed, but I just could not manipulate the outcome of the patches without proper approval of the maintainer, because this is your role. Thanks lippe@ On Sun, Jun 30, 2013 at 10:19 PM, Alexey Dokuchaev wrote: > On Fri, Jun 28, 2013 at 08:45:36PM +0000, Felippe de Meirelles Motta wrote: > > New Revision: 321982 > > URL: http://svnweb.freebsd.org/changeset/ports/321982 > > > > +# Created by: Alexander Panyushkin > > +# $FreeBSD$ > > + > > +PORTNAME= hans > > +PORTVERSION= 0.4.1 > > +CATEGORIES= net security > > +MASTER_SITES= ${MASTER_SITE_SOURCEFORGE} > > +MASTER_SITE_SUBDIR= /hanstunnel/source/ > > Submitter apparently does not how to use MASTER_SITES/MASTER_SITE_SUBDIR > combo propopery: > > make fetch DISTDIR=/tmp > ===> portaudit database exists, however, portaudit is not installed! > => hans-0.4.1.tar.gz doesn't seem to exist in /tmp/. > => Attempting to fetch > > http://nchc.dl.sourceforge.net/project//hanstunnel/source//hans-0.4.1.tar.gz > ... > > Note the bogus double slashes. Correct, modern way is to simple write: > > MASTER_SITES= SF/hanstunnel/source > > > +MAINTAINER= vsityz@gmail.com > > +COMMENT= Hans makes it possible to tunnel IPv4 through ICMP > > It's generally wrong to start COMMENT line with "Foobar is..." or alikes. > Better comment in this case would be smth. like "Program to tunnel IPv4 > through ICMP echo packets". > > > +LICENSE_PERMS_GPL3= yes > > Why not just "LICENSE= GPLv3" ?? > > > +OPTIONS_DEFINE= CLANG > > + > > +CLANG_DESC= Build with Clang instead of GCC > > + > > +OPTIONS_DEFAULT= CLANG > > I don't really see the point of such option. As there any noticeable > changes > which warrant this selection, or submitter just wanted to play with Clang a > bit? :-) Imagine a user (not a developer) building this port: such a > dialog > would probably puzzle him/her. > > > .if ${PORT_OPTIONS:MCLANG} > > .if ${OSVERSION} < 1000000 > > CC=clang > > CXX=clang++ > > .if ${OSVERSION} < 900033 > > BUILD_DEPENDS+= clang:${PORTSDIR}/lang/clang > > .endif > > .endif > > .else > > USE_GCC=4.2+ > > CC=gcc > > CXX=g++ > > CPP=gcpp > > .endif > > Bad indentation (lack there of). > > > +do-install: > > + @${INSTALL} -c -o ${BINOWN} -g ${BINGRP} -m ${BINMODE} > ${WRKSRC}/hans ${LOCALBASE}/bin > > Why not simply use INSTALL_PROGRAM here? > > ./danfe >