From owner-cvs-all@FreeBSD.ORG  Tue Jul 26 07:22:40 2011
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: by hub.freebsd.org (Postfix, from userid 1033)
	id 706CC1065670; Tue, 26 Jul 2011 07:22:40 +0000 (UTC)
Date: Tue, 26 Jul 2011 07:22:40 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Doug Barton <dougb@FreeBSD.org>
Message-ID: <20110726072240.GA12192@FreeBSD.org>
References: <201107200252.p6K2qZRb053515@repoman.freebsd.org>
	<CAGFTUwMZ5iAp=VvNZezX4fiGy2d5Ep6jaSWWUZp92KTobxkjsQ@mail.gmail.com>
	<4E2B2B82.1020506@FreeBSD.org> <4E2D7EEC.8030600@p6m7g8.com>
	<20110726025646.74097.qmail@exxodus.fedaykin.here>
	<4E2E320C.8050607@FreeBSD.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Disposition: inline
In-Reply-To: <4E2E320C.8050607@FreeBSD.org>
User-Agent: Mutt/1.4.2.1i
Cc: "b. f." <bf1783@googlemail.com>, cvs-all@FreeBSD.org,
	ports-committers@FreeBSD.org, bf1783@gmail.com, riggs@rrr.de,
	"Philip M. Gollucci" <pgollucci@p6m7g8.com>, cvs-ports@FreeBSD.org,
	Mario Sergio Fujikawa Ferreira <lioux@FreeBSD.org>
Subject: Re: cvs commit: ports/multimedia/mencoder Makefile
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: Tue, 26 Jul 2011 07:22:40 -0000

On Mon, Jul 25, 2011 at 08:18:36PM -0700, Doug Barton wrote:
> On 07/25/2011 19:56, Mario Sergio Fujikawa Ferreira wrote:
> > B) if multimedia/libass is installed:
> >   2. mplayer/mencoder only builds if we fix configure script and
> >      force port dependend on multimedia/libass because mplayer's
> >      configure is broken.
> > ---
> > 
> >   Is multimedia/libass easily available? Yeah, because multimedia/vlc
> > depends on it and lots (if not most) users building mplayer, should
> > have already built vlc. So this problem is pretty standard for many
> > users.
> 
> So the answer is simple. Fix whatever needs fixing and force the
> dependency.

Exactly.  If something cannot be switched off forcibly, then ask yourself
if users would rather have it or not.  For popular and/or lightweight stuff,
it's usually beneficial for everyone to hardcode dependency and help user to
worry less about all those OPTIONs. :-)

For controversial, specific, or heavy dependencies (or if they pull way too
many other stuff, like half of KDE4), it is desirable to fix the port and
make this dependency off by default.  If configure script (just an example)
is hard to fix, nag the user about their OPTIONs selection may not be
uphold.

./danfe