From owner-svn-src-head@FreeBSD.ORG Thu Sep 5 20:56:40 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 53D1FED7; Thu, 5 Sep 2013 20:56:40 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from bigwig.baldwin.cx (bigwig.baldwin.cx [IPv6:2001:470:1f11:75::1]) (using TLSv1 with cipher ADH-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 0E56F2FD6; Thu, 5 Sep 2013 20:56:40 +0000 (UTC) Received: from jhbbsd.localnet (unknown [209.249.190.124]) by bigwig.baldwin.cx (Postfix) with ESMTPSA id 247BFB97A; Thu, 5 Sep 2013 16:56:38 -0400 (EDT) From: John Baldwin To: Joel Dahl Subject: Re: svn commit: r254273 - in head: . include lib lib/libc/iconv lib/libiconv_compat lib/libkiconv share/mk sys/sys tools/build/mk Date: Thu, 5 Sep 2013 16:54:13 -0400 User-Agent: KMail/1.13.5 (FreeBSD/8.2-CBSD-20110714-p28; KDE/4.5.5; amd64; ; ) References: <201308130715.r7D7F1nu076335@svn.freebsd.org> <201309051013.35286.jhb@freebsd.org> <20130905201540.GA23637@devbox.vnode.local> In-Reply-To: <20130905201540.GA23637@devbox.vnode.local> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201309051654.13482.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (bigwig.baldwin.cx); Thu, 05 Sep 2013 16:56:38 -0400 (EDT) Cc: src-committers@freebsd.org, Jilles Tjoelker , Peter Wemm , svn-src-all@freebsd.org, Dimitry Andric , 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 20:56:40 -0000 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. :( -- John Baldwin