From owner-freebsd-ports@FreeBSD.ORG Thu Aug 26 09:34:08 2010 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6308A10656AA for ; Thu, 26 Aug 2010 09:34:08 +0000 (UTC) (envelope-from jdc@koitsu.dyndns.org) Received: from qmta10.westchester.pa.mail.comcast.net (qmta10.westchester.pa.mail.comcast.net [76.96.62.17]) by mx1.freebsd.org (Postfix) with ESMTP id AE1648FC18 for ; Thu, 26 Aug 2010 09:34:07 +0000 (UTC) Received: from omta22.westchester.pa.mail.comcast.net ([76.96.62.73]) by qmta10.westchester.pa.mail.comcast.net with comcast id yxYx1e0011ap0As5Axa7b4; Thu, 26 Aug 2010 09:34:07 +0000 Received: from koitsu.dyndns.org ([98.248.41.155]) by omta22.westchester.pa.mail.comcast.net with comcast id yxa51e0063LrwQ23ixa62S; Thu, 26 Aug 2010 09:34:07 +0000 Received: by icarus.home.lan (Postfix, from userid 1000) id 708A49B425; Thu, 26 Aug 2010 02:34:04 -0700 (PDT) Date: Thu, 26 Aug 2010 02:34:04 -0700 From: Jeremy Chadwick To: Emanuel Haupt Message-ID: <20100826093404.GA84386@icarus.home.lan> References: <20100817060114.GA86738@icarus.home.lan> <20100824153157.GA63745@icarus.home.lan> <20100824211420.3ecced97.ehaupt@FreeBSD.org> <20100826102356.b5e71239.ehaupt@critical.ch> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100826102356.b5e71239.ehaupt@critical.ch> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: Alex@freebsd.org, Garrett Wollman , freebsd-ports@freebsd.org, Emanuel Haupt , Goncharov Subject: Re: shells/bash and the libiconv dependency mess X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 26 Aug 2010 09:34:08 -0000 On Thu, Aug 26, 2010 at 10:23:56AM +0200, Emanuel Haupt wrote: > Emanuel Haupt wrote: > > Jeremy Chadwick wrote: > > > On Mon, Aug 16, 2010 at 11:01:14PM -0700, Jeremy Chadwick wrote: > > > > Let me explain what transpired in chronological order: > > > > > > > > On 2010/05/11, ehaupt committed the following patch: > > > > > > > > http://www.freebsd.org/cgi/cvsweb.cgi/ports/shells/bash/files/patch-Makefile.in > > > > > > > > And bumped PORTREVISION (from 0 to 1) in the Makefile. This > > > > unconditionally made bash require libiconv, and the only > > > > justification is "fix statically linked version". > > > > > > > > Those of us who use WITHOUT_NLS or who do not have libiconv > > > > already on their systems (from another port) immediately notice > > > > the problem (bash will no longer build): > > > > > > > > http://www.freebsd.org/cgi/query-pr.cgi?pr=147747 > > > > http://www.freebsd.org/cgi/query-pr.cgi?pr=148329 > > > > http://www.freebsd.org/cgi/query-pr.cgi?pr=149218 > > > > > > > > Three months goes by and finally something is committed to fix the > > > > problem on 2010/08/06. Except the fix doesn't make any sense; all > > > > it does is make libiconv a mandatory dependency (USE_ICONV): > > > > > > > > http://www.freebsd.org/cgi/cvsweb.cgi/ports/shells/bash/Makefile#rev1.123 > > > > > > > > This, of course, means that WITHOUT_NLS is broken and doesn't work > > > > as it's supposed to, since libiconv is now a mandatory requirement > > > > (it doesn't need to be): > > > > > > > > # make WITHOUT_NLS=true all-depends-list > > > > /usr/ports/devel/bison > > > > /usr/ports/converters/libiconv > > > > /usr/ports/devel/m4 > > > > /usr/ports/devel/libtool22 > > > > > > > > Why was this done the way it was? patch-Makefile.in should be > > > > removed and instead replaced with a REINPLACE_CMD that handles the > > > > conditionals (WITH_STATIC_BASIC, WITHOUT_NLS, etc.) in a more > > > > clean manner. > > > > > > > > And where are the details of the supposed "statically linked > > > > version" problem? > > > > > > > > Sorry if I sound angry, but this whole situation is a mess, and > > > > shells/bash is a very important port. If someone wants me to put > > > > my money where my mouth is and go + clean it up I'll be happy to. > > > > Testing all the different quirk combinations really isn't that > > > > complex. > > > > > > Since I didn't really get any answers regarding this predicament, I > > > went ahead and did the necessary effort. Below are my QA notes. > > > The patch is available here: > > > > > > http://jdc.parodius.com/freebsd/bash-without-nls.patch > > > > The patch works for me. One minor thing, PORTREVISION should be > > bumped. > > > > > Please note the required removal of files/patch-Makefile.in. > > > > > > I can file a PR for all this if need be, just let me know. > > > > Please do that, that way I can commit the patch after a 14 day > > maintainer timeout. > > I created: > > http://www.freebsd.org/cgi/query-pr.cgi?pr=149981 Thank you for doing this, and my apologies. I had a molar removed yesterday afternoon so I've pretty much been tending that + sleeping, otherwise I would've gotten to it. :-) Thanks again! -- | Jeremy Chadwick jdc@parodius.com | | Parodius Networking http://www.parodius.com/ | | UNIX Systems Administrator Mountain View, CA, USA | | Making life hard for others since 1977. PGP: 4BD6C0CB |