Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 26 Nov 2012 16:57:23 +0100
From:      Guido Falsi <madpilot@FreeBSD.org>
To:        Alexey Dokuchaev <danfe@FreeBSD.org>
Cc:        svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: svn commit: r307782 - head/audio/asunder
Message-ID:  <50B39163.2040309@FreeBSD.org>
In-Reply-To: <20121126135609.GC62928@FreeBSD.org>
References:  <201211261259.qAQCxdXD089022@svn.freebsd.org> <20121126135609.GC62928@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 11/26/12 14:56, Alexey Dokuchaev wrote:
> On Mon, Nov 26, 2012 at 12:59:39PM +0000, Guido Falsi wrote:
>> New Revision: 307782
>> URL: http://svnweb.freebsd.org/changeset/ports/307782
>>
>> -PORTVERSION=	2.0
>> -PORTREVISION?=	2
>> +PORTVERSION=	2.2
>
> Hmm, does this port have slaves?  PORTREVISION?= suggests it does, it this
> case it should be left as PORTREVISION?=0, otherwise it could be overridden
> by slave port(s).

I cannot find any slave ports. I'm inclined to think that the ?= was 
added by mistake in r289243.

>
>> +MONKEYAUDIO_DESC=	Monkey's Audio support
>
> We have standard knob for this one (APE_DESC).
>
>>   PORTDOCS=	AUTHORS COPYING ChangeLog INSTALL NEWS README TODO
>
> Normally we do not install COPYING as part of the docs, LICENSE should be
> set instead.  If software expects to find this file (e.g. for displaying in
> the about dialog), it should be redirected to share/licenses/${PKGNAME}
> instead (see `graphics/luminance' port for an example).
>

Thanks for these, I'll fix them and commit after maintainer approval.

-- 
Guido Falsi <madpilot@FreeBSD.org>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?50B39163.2040309>