From owner-svn-src-head@FreeBSD.ORG Thu Sep 5 21:33:39 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 18333EC8; Thu, 5 Sep 2013 21:33:39 +0000 (UTC) (envelope-from andreast@FreeBSD.org) Received: from smtp.fgznet.ch (mail.fgznet.ch [81.92.96.47]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id AD9582208; Thu, 5 Sep 2013 21:33:37 +0000 (UTC) Received: from deuterium.andreas.nets (dhclient-91-190-14-19.flashcable.ch [91.190.14.19]) by smtp.fgznet.ch (8.13.8/8.13.8/Submit_SMTPAUTH) with ESMTP id r85LXI04007439; Thu, 5 Sep 2013 23:33:27 +0200 (CEST) (envelope-from andreast@FreeBSD.org) Message-ID: <5228F89E.9030102@FreeBSD.org> Date: Thu, 05 Sep 2013 23:33:18 +0200 From: Andreas Tobler User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 MIME-Version: 1.0 To: src-committers@FreeBSD.org Subject: Re: svn commit: r254273 - in head: . include lib lib/libc/iconv lib/libiconv_compat lib/libkiconv share/mk sys/sys tools/build/mk References: <201308130715.r7D7F1nu076335@svn.freebsd.org> <201309051013.35286.jhb@freebsd.org> <20130905201540.GA23637@devbox.vnode.local> <201309051654.13482.jhb@freebsd.org> In-Reply-To: <201309051654.13482.jhb@freebsd.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.64 on 81.92.96.47 Cc: Jilles Tjoelker , Peter Wemm , svn-src-all@FreeBSD.org, Dimitry Andric , Joel Dahl , gabor@FreeBSD.org, svn-src-head@FreeBSD.org, John Baldwin 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 21:33:39 -0000 On 05.09.13 22:54, John Baldwin wrote: > On Thursday, September 05, 2013 4:15:40 pm Joel Dahl wrote: >> 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 >>>>>>> >>>>>>>>>> Log: >>>>>>>>>> The iconv in libc did two things - implement the standard APIs, the GNU >>>>>>>>>> extensions and also tried to be link time compatible with ports libiconv. >>>>>>>>>> This splits that functionality and enables the parts that shouldn't >>>>>>>>>> interfere with the port by default. >>>>>>> >>>>>>>>>> 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 reasonably expect it >>>>>>>>>> to work. >>>>>>> >>>>>>>>>> I have tortured many machines over the last few days to try and reduce >>>>>>>>>> the possibilities of foot-shooting as much as I can. I've successfully >>>>>>>>>> recompiled to enable and disable the libiconv_compat modes, ports that use >>>>>>>>>> libiconv alongside system iconv etc. If you don't enable the >>>>>>>>>> WITH_LIBICONV_COMPAT switch, they don't share symbol space. >>>>>>> >>>>>>>>>> This is an extension of behavior on other system. iconv(3) is a standard >>>>>>>>>> libc interface and libiconv port expects to be able to run alongside it on >>>>>>>>>> systems that have it. >>>>>>> >>>>>>>>> Unfortunately I expect this will break many ports, when the libiconv >>>>>>>>> port is installed. A simple example is the following: >>>>>>>> >>>>>>> >>>>>>>> It also breaks installworld when /usr/src and /usr/obj are NFS exported >>>>>>>> read-only. >>>>>>> >>>>>>> I think it has to do with share/i18n/csmapper and share/i18n/esdb using >>>>>>> directories as make targets. This apparently causes these files to be >>>>>>> rebuilt at 'make installworld' time, which is always bad but is only >>>>>>> detected when /usr/obj is read-only. >>>>>>> >>>>>>> A hack that works is to enclose the four targets depending on ${SUBDIR} >>>>>>> in .if !make(install) . >>>>>>> >>>>>>> Unfortunately, the Makefiles were written to depend on the directories >>>>>>> as make targets fairly deeply, so a real fix is harder. >>>>>> >>>>>> 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 systems. >>>>> >>>>> FWIW, this is still broken. >>>> >>>> Again, this is still broken. >>> >>> Yeah, my laptop failed to build cups (required by ghostscript which is required >>> by emacs) because of this: >> >> Well, that's a different problem. Installworld is still broken on systems with >> readonly /usr/obj. > > It is the problem originally reported by dim@ at the start of this thread. :) > > My ports tree from today does build cups fine FWIW, but it seems there is still > a lot of fallout from this change. :( Jumping in an give my comment. This has been fixed in the past few days. A lot of has been done and I was also able to build my 600 pkg set w/o any iconv hickup. A week ago I had to change the mobo of my workstation and I thought this would be a good idea to rebuild every port I have installed. Bad idea, every iconv related one failed. Yesterday I retried..... I had only one issue when building qcad which relies on openNURBS. And the latter has to be compiled with -fPIC. But this is not related to the topic. So a big thanks to the ports people! Andreas