From owner-svn-src-head@FreeBSD.ORG Thu Sep 5 14:46:54 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 04CDDC0B; Thu, 5 Sep 2013 14:46:54 +0000 (UTC) (envelope-from baptiste.daroussin@gmail.com) Received: from mail-we0-x22a.google.com (mail-we0-x22a.google.com [IPv6:2a00:1450:400c:c03::22a]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 9D716248E; Thu, 5 Sep 2013 14:46:52 +0000 (UTC) Received: by mail-we0-f170.google.com with SMTP id w62so773825wes.15 for ; Thu, 05 Sep 2013 07:46:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=FTO6llb1YZ0oXUYb1RUTODxS1h+5H+w+z6CciUP/iZQ=; b=fvpn7tGWsPOwZYtpC8hH5AGuOPC6qKUeYmd5vwQQ5eqpef8y1/nIJqIqeH0oyVjWEP TIboGz28XYxLitHLFOnBI+A6IYL5ooeIBd/LByoGPGAc18TsMPKliwlIsjcxCgQ05/0O s9tU+SZf59fFfwVeIU5RbI1Xlh3NNZXGeEhS4FLEEzZ7xKJOxdKjTXdJj44Eit2EFCij Zhu4GtF82/fSQVGDG0abDXMY+98LFcWKVIzOteHBMeQYQSYyTSPWi66iPGjS1ZsBiqv8 n9di9ogSxqZPX+UQfEo5lZbmIfK27/FBpioxGxf2daHXmc610v//b29ju5PYpha7hhTN KNfA== X-Received: by 10.180.109.35 with SMTP id hp3mr6630227wib.52.1378392410870; Thu, 05 Sep 2013 07:46:50 -0700 (PDT) Received: from ithaqua.etoilebsd.net (ithaqua.etoilebsd.net. [37.59.37.188]) by mx.google.com with ESMTPSA id v7sm12247563wiy.11.1969.12.31.16.00.00 (version=TLSv1 cipher=RC4-SHA bits=128/128); Thu, 05 Sep 2013 07:46:50 -0700 (PDT) Sender: Baptiste Daroussin Date: Thu, 5 Sep 2013 16:46:48 +0200 From: Baptiste Daroussin To: John Baldwin Subject: Re: svn commit: r254273 - in head: . include lib lib/libc/iconv lib/libiconv_compat lib/libkiconv share/mk sys/sys tools/build/mk Message-ID: <20130905144647.GO82066@ithaqua.etoilebsd.net> References: <201308130715.r7D7F1nu076335@svn.freebsd.org> <20130822155835.GA52789@devbox.vnode.local> <20130903195241.GA93218@devbox.vnode.local> <201309051013.35286.jhb@freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Qa0ccP92Gc0Ko3P8" Content-Disposition: inline In-Reply-To: <201309051013.35286.jhb@freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-Mailman-Approved-At: Thu, 05 Sep 2013 15:44:02 +0000 Cc: src-committers@freebsd.org, Jilles Tjoelker , Peter Wemm , svn-src-all@freebsd.org, Dimitry Andric , Joel Dahl , gabor@freebsd.org, svn-src-head@freebsd.org, Peter Wemm X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Sep 2013 14:46:54 -0000 --Qa0ccP92Gc0Ko3P8 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Sep 05, 2013 at 10:13:34AM -0400, John Baldwin wrote: > On Tuesday, September 03, 2013 3:52:41 pm Joel Dahl wrote: > > On Thu, Aug 22, 2013 at 05:58:35PM +0200, Joel Dahl wrote: > > > On Sun, Aug 18, 2013 at 03:51:25PM -0700, Peter Wemm wrote: > > > > On 8/18/13 3:42 PM, Jilles Tjoelker wrote: > > > > > On Sun, Aug 18, 2013 at 09:53:04PM +0200, Joel Dahl wrote: > > > > >> On Sun, Aug 18, 2013 at 12:34:30AM +0200, Dimitry Andric wrote: > > > > >>> On Aug 13, 2013, at 09:15, Peter Wemm wrote: > > > > >>>> Author: peter > > > > >>>> Date: Tue Aug 13 07:15:01 2013 > > > > >>>> New Revision: 254273 > > > > >>>> URL: http://svnweb.freebsd.org/changeset/base/254273 > > > > >=20 > > > > >>>> Log: > > > > >>>> The iconv in libc did two things - implement the standard API= s, the GNU > > > > >>>> extensions and also tried to be link time compatible with por= ts libiconv. > > > > >>>> This splits that functionality and enables the parts that sho= uldn't > > > > >>>> interfere with the port by default. > > > > >=20 > > > > >>>> WITH_ICONV (now on by default) - adds iconv.h, iconv_open(3) = etc. > > > > >>>> WITH_LIBICONV_COMPAT (off by default) adds the libiconv_open = etc API, linker > > > > >>>> symbols and even a stub libiconv.so.3 that are good enough to= be able > > > > >>>> to 'pkg delete -f libiconv' on a running system and reasonabl= y expect it > > > > >>>> to work. > > > > >=20 > > > > >>>> I have tortured many machines over the last few days to try a= nd reduce > > > > >>>> the possibilities of foot-shooting as much as I can. I've su= ccessfully > > > > >>>> recompiled to enable and disable the libiconv_compat modes, p= orts that use > > > > >>>> libiconv alongside system iconv etc. If you don't enable the > > > > >>>> WITH_LIBICONV_COMPAT switch, they don't share symbol space. > > > > >=20 > > > > >>>> This is an extension of behavior on other system. iconv(3) i= s a standard > > > > >>>> libc interface and libiconv port expects to be able to run al= ongside it on > > > > >>>> systems that have it. > > > > >=20 > > > > >>> Unfortunately I expect this will break many ports, when the lib= iconv > > > > >>> port is installed. A simple example is the following: > > > > >> > > > > >=20 > > > > >> It also breaks installworld when /usr/src and /usr/obj are NFS e= xported > > > > >> read-only. > > > > >=20 > > > > > I think it has to do with share/i18n/csmapper and share/i18n/esdb= using > > > > > directories as make targets. This apparently causes these files t= o be > > > > > rebuilt at 'make installworld' time, which is always bad but is o= nly > > > > > detected when /usr/obj is read-only. > > > > >=20 > > > > > A hack that works is to enclose the four targets depending on ${S= UBDIR} > > > > > in .if !make(install) . > > > > >=20 > > > > > Unfortunately, the Makefiles were written to depend on the direct= ories > > > > > as make targets fairly deeply, so a real fix is harder. > > > >=20 > > > > I was looking at this yesterday, but was tied up with other things.= I'll > > > > take a look at it today after getting a few other things done. It = should be > > > > easy enough to replicate by changing /usr/obj to readonly on test s= ystems. > > >=20 > > > FWIW, this is still broken. > >=20 > > Again, this is still broken. >=20 > Yeah, my laptop failed to build cups (required by ghostscript which is re= quired > by emacs) because of this: >=20 > =3D=3D=3D> FreeBSD 10 autotools fix applied to /usr/ports/print/cups-im= age/work/cups-1.5.4/configure > Configuring CUPS with options: > ... > configure: WARNING: unrecognized options: --disable-pdftops > ... > checking iconv.h usability... yes > checking iconv.h presence... yes > checking for iconv.h... yes > checking for library containing iconv_open... none required > ... > Linking texttops... > cc -L../cgi-bin -L../cups -L../filter -L../ppdc -L../scheduler -L/usr/loc= al/lib -Wl,-rpath=3D/usr/lib:/usr/local/lib -Wl,-R/usr/local/lib -Wall -W= no- > format-y2k -Wunused -fPIC -Os -g -fstack-protector -Wno-tautological-comp= are -o texttops texttops.o textcommon.o common.o -lcups -lssl -lcrypto -l= z=20 > -pthread -lcrypt -lm -lssp_nonshared=20 > ../cups/libcups.so: undefined reference to `libiconv' > ../cups/libcups.so: undefined reference to `libiconv_close' > ../cups/libcups.so: undefined reference to `libiconv_open' > cc: error: linker command failed with exit code 1 (use -v to see invocati= on) > gmake[9]: *** [commandtops] Error 1 > ... >=20 > Having cups broken will probably break all of kde, etc. Are we seeing th= is > in HEAD package builds yet? (Maybe the world we are using doesn't have > this change yet?) Note that this is just a clean build of HEAD and a fre= sh > build of ports from scratch. >=20 This should work properly now, the ports tree received motification yesterd= ay and converters/libiconv is not installed anymore now. regards, Bapt --Qa0ccP92Gc0Ko3P8 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) iEYEARECAAYFAlIomVcACgkQ8kTtMUmk6EwPOgCgj0LeXhe56NfFHtfP0acKhKjl 6OwAoJATDcqMXj5lkU71Rj5fiH5nL/iS =kP58 -----END PGP SIGNATURE----- --Qa0ccP92Gc0Ko3P8--