From owner-cvs-all@FreeBSD.ORG Mon Jul 25 14:34:22 2011 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6169E10656D7; Mon, 25 Jul 2011 14:34:22 +0000 (UTC) (envelope-from pgollucci@p6m7g8.com) Received: from exhub015-2.exch015.msoutlookonline.net (exhub015-2.exch015.msoutlookonline.net [207.5.72.94]) by mx1.freebsd.org (Postfix) with ESMTP id 45D8F8FC08; Mon, 25 Jul 2011 14:34:22 +0000 (UTC) Received: from philip.hq.rws (174.79.184.239) by smtpx15.msoutlookonline.net (207.5.72.103) with Microsoft SMTP Server (TLS) id 8.2.254.0; Mon, 25 Jul 2011 07:34:21 -0700 Message-ID: <4E2D7EEC.8030600@p6m7g8.com> Date: Mon, 25 Jul 2011 14:34:20 +0000 From: "Philip M. Gollucci" Organization: P6M7G8 Inc. User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US; rv:1.9.1.16) Gecko/20110507 Thunderbird/3.0.11 MIME-Version: 1.0 To: Doug Barton References: <201107200252.p6K2qZRb053515@repoman.freebsd.org> <4E2B2B82.1020506@FreeBSD.org> In-Reply-To: <4E2B2B82.1020506@FreeBSD.org> X-Enigmail-Version: 1.0.1 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: "b. f." , cvs-all@FreeBSD.org, ports-committers@FreeBSD.org, Mario, riggs@rrr.de, bf1783@gmail.com, cvs-ports@FreeBSD.org, Ferreira 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 25 Jul 2011 14:34:22 -0000 On 07/23/11 20:13, Doug Barton wrote: > On 07/22/2011 23:07, b. f. wrote: > >> I must renew my plea to avoid these "fixes". When a user sets >> WITH/WITHOUT_FOO, he means precisely "build this port >> WITH/WITHOUT_FOO", and not "build this port WITH/WITHOUT_FOO, except >> if package X is installed, and, if it is, silently do something else", >> and we should not be adding code to do the latter. We should instead >> patch the broken distribution configure scripts and Makefiles, so that >> the problem does not occur. If a lack of time or another problem >> prevents this from being done initially, then as a temporary >> workaround, we should add something like: >> >> .if exists(${LOCALBASE}/lib/libass.so) && defined(WITHOUT_ASS) >> BROKEN= libass is installed, so broken mplayer scripts will ignore WITHOUT_ASS >> .endif >> >> so that the user will know that his choice of options cannot be used, >> and he can take the appropriate action -- and he will not be misled >> into thinking that his choice of options are used when they are not. > > +1 > Yes, I agree violently too. I even remember removing about 10 occurrences in the past. -- ------------------------------------------------------------------------ 1024D/DB9B8C1C B90B FBC3 A3A1 C71A 8E70 3F8C 75B8 8FFB DB9B 8C1C Philip M. Gollucci (pgollucci@p6m7g8.com) c: 703.336.9354 VP Infrastructure, Apache Software Foundation Committer, FreeBSD Foundation Consultant, P6M7G8 Inc. Sr. System Admin, Ridecharge Inc. Work like you don't need the money, love like you'll never get hurt, and dance like nobody's watching.