From owner-svn-ports-head@FreeBSD.ORG Tue Feb 3 10:01:00 2015 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 50674B04; Tue, 3 Feb 2015 10:01:00 +0000 (UTC) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 23E82CB; Tue, 3 Feb 2015 10:00:59 +0000 (UTC) Received: from [10.31.9.208] (unknown [213.225.137.129]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id 7694543BB4; Tue, 3 Feb 2015 04:00:52 -0600 (CST) Message-ID: <54D09C51.5020001@marino.st> Date: Tue, 03 Feb 2015 11:00:49 +0100 From: John Marino Reply-To: marino@freebsd.org User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Tijl Coosemans , Kubilay Kocak Subject: Re: svn commit: r378316 - head/devel/libhtp References: <201502021841.t12IfvP1021156@svn.freebsd.org> <54D01223.7020703@FreeBSD.org> <54D015B7.2080408@marino.st> <54D01B67.4010406@FreeBSD.org> <20150203105553.3bbbeb6c@kalimero.tijl.coosemans.org> In-Reply-To: <20150203105553.3bbbeb6c@kalimero.tijl.coosemans.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, marino@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Feb 2015 10:01:00 -0000 On 2/3/2015 10:55, Tijl Coosemans wrote: > On Tue, 03 Feb 2015 11:50:47 +1100 Kubilay Kocak wrote: >> On 3/02/2015 11:26 AM, John Marino wrote: >>> I don't know if "LIBS" is standardized but it's a flag that's missing, >>> not a library. >> >> I'll pull tijl@ in the explain this one if needed and I'll leave this >> commit reference here as well: >> >> https://svnweb.freebsd.org/ports?view=revision&revision=357486 > > Both -L and -l should go into LIBS if possible. LIBS appears later on > the command line and -L${LOCALBASE}/lib should appear after any -L flags > specified by upstream. > >> And just to note, there was this related and "simple" change that broke >> Python completely: >> >> https://lists.freebsd.org/pipermail/freebsd-python/2014-July/007129.html > > I think you misremember this. It turned out to be a misconfiguration in > the user's make.conf. > >>>> Further, if it is indeed the case that iconv:translit adds -liconv to >>>> LDFLAGS, wouldn't adding -L${LOCALBASE}/lib to LDFLAGS be better solved >>>> in Uses/iconv.mk when that case is true? >>> >>> I would think so. >>> Most of the time this doesn't pop up because another dependency brings >>> in -L/usr/local/lib so it works by accident very often. >>> >>> At the very least it should have an ${LDFLAGS_ICONV} option that could >>> be added to LDFLAGS. It would be a more correct solution. >> >> If you can sort out a proper fix in the right place that would be great. >> You have approval from me to revert/change devel/libhtp as necessary. > > You can use -L${ICONV_PREFIX}/lib (and -I${ICONV_PREFIX}/include). The > reason USES=iconv doesn't add -L/-l to LDFLAGS or LIBS (or -I to > CPPFLAGS) is because not all ports support these variables and the > ordering of the flags may be important. Only the port Makefile knows > all the flags that need to be added and in which order. Okay, so to summarize: 1) it was correct to add the fix to the port makefile 2) Uses/iconv.mk doesn't have to be updated 3) I'll test moving the -L flag into LIBS, and if it still builds fine I'll make the change (I don't think it makes much difference in this case, but it seems to be preferred). Thanks tijl, John