cvs commit: ports/net-im/centerim-devel Makefile pkg-plist

Alexey Dokuchaev danfe at FreeBSD.org
Wed Sep 29 08:22:15 UTC 2010


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 at 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


More information about the cvs-all mailing list