From owner-cvs-all@FreeBSD.ORG  Wed Sep 29 08:22:15 2010
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 1033)
	id 723021065674; Wed, 29 Sep 2010 08:22:15 +0000 (UTC)
Date: Wed, 29 Sep 2010 08:22:15 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Andrej Zverev <az@freebsd.org>
Message-ID: <20100929082215.GB97597@FreeBSD.org>
References: <201009290707.o8T77b2s073828@repoman.freebsd.org>
	<AANLkTinWUC-6i3JgRBT0a7Uc3+ViYChZ1hE4wKwKTbap@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
In-Reply-To: <AANLkTinWUC-6i3JgRBT0a7Uc3+ViYChZ1hE4wKwKTbap@mail.gmail.com>
User-Agent: Mutt/1.4.2.1i
Cc: cvs-ports@freebsd.org, Baptiste Daroussin <bapt@freebsd.org>,
	cvs-all@freebsd.org, ports-committers@freebsd.org
Subject: Re: cvs commit: ports/net-im/centerim-devel Makefile pkg-plist
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: **OBSOLETE** CVS commit messages for the entire tree
	<cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 29 Sep 2010 08:22:15 -0000

On Wed, Sep 29, 2010 at 11:39:56AM +0400, Andrej Zverev wrote:
> On Wed, Sep 29, 2010 at 11:07 AM, Baptiste Daroussin <bapt@freebsd.org>wrote:
> > bapt        2010-09-29 07:07:37 UTC
> >
> >  FreeBSD ports repository
> >
> >  Modified files:
> >    net-im/centerim-devel Makefile pkg-plist
> >  Log:
> >  - fix plist
> >  - added run depend on perl and python
> >  - bump port revision
> >
> >  Reported by:    QAT
> 
> Are you sure the necessary dependencies for Perl and python?
> Maybe these scripts are not for the program, but simply in order to be able
> to build the some reports, while P5 & Python is clearly not needed.

I also want to express my concern with recent (over last few years)
trend of simple adding necessary dependencies because "QAT complained",
while in many cases these deps are bogus (used only during brain-damaged
configure script is one common case).  Not to pick on Baptiste, but I
encourage us to check more thoroughly why particular dependency is
required and reflecting the reason in commit log (except some simple
obvious cases).

> Also typo PORTRESION -> PORTREVISION.

Ugh, very embarrassing typo.

./danfe