Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 28 Nov 2012 16:08:25 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Alex Kozlov <ak@FreeBSD.org>
Cc:        svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: svn commit: r307899 - head/benchmarks/super-smack
Message-ID:  <20121128160825.GB53862@FreeBSD.org>
In-Reply-To: <20121128130417.GA31569@ravenloft.kiev.ua>
References:  <201211281109.qASB9dPm010874@svn.freebsd.org> <20121128115359.GA53862@FreeBSD.org> <20121128130417.GA31569@ravenloft.kiev.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Nov 28, 2012 at 03:04:17PM +0200, Alex Kozlov wrote:
> On Wed, Nov 28, 2012 at 11:53:59AM +0000, Alexey Dokuchaev wrote:
> > On Wed, Nov 28, 2012 at 11:09:39AM +0000, Alex Kozlov wrote:
> > > New Revision: 307899
> > > URL: http://svnweb.freebsd.org/changeset/ports/307899
> > > 
> > > +PGSQL_DESC=	Enable PostgreSQL support
> > > +MYSQL_DESC=	Enable MySQL support
> > > 
> > In this case, generic descriptions should have been used.
> > 
> No, specific descriptions are still better.

They are essentially the same modulo "enable" word, which was decided to
remove some time ago.  In fact, "generic" descriptions are better since they
mention that is it *database* support.  Doug removed the word "support" in
r305288, which many folks disagree with, but it will be backed out one we
will have dialog4ports committed, which will allow proper support for
checklists and radiobuttons.  You can leave them now, but after r305288 will
be backed out, I will ask you again to drop them.

> This was impossible to catch in vim, it should be automated like with WWW.
> If this tag is obsolete, then portlint should display a warning. I can
> implement the patch, but before that can you please point me to documentation
> where it stated? Thanks.

It is not technically obsolete; sometimes it is the only attribution we
can provide.  If there is actual WWW page, it should be dropped.

./danfe



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