From owner-svn-ports-all@FreeBSD.ORG Tue Feb 3 09:55:57 2015 Return-Path: Delivered-To: svn-ports-all@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 96A9B9FA; Tue, 3 Feb 2015 09:55:57 +0000 (UTC) Received: from mailrelay103.isp.belgacom.be (mailrelay103.isp.belgacom.be [195.238.20.130]) by mx1.freebsd.org (Postfix) with ESMTP id 330C5FB5; Tue, 3 Feb 2015 09:55:55 +0000 (UTC) X-Belgacom-Dynamic: yes X-Cloudmark-SP-Filtered: true X-Cloudmark-SP-Result: v=1.1 cv=ZT6Ny9SzQUL0U4sOn6xM6rLbT25dGdPOAe/imaXRw80= c=1 sm=2 a=kj9zAlcOel0A:10 a=6I5d2MoRAAAA:8 a=mVx6q1jHy28WIcjNDtYA:9 a=CjuIK1q_8ugA:10 a=m9jJNuv2fyIA:10 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A2BhBgCjmtBU/xOHsFtagwZSTsUIDIVvAoEYRAEBAQEBfYQMAQEBAwEBAjccIwULCw4KCSUPGREeBhMbiAoMAQjVZQEBAQEBAQEBAQEBAQEBAQEBG4oOhWoHhCkFkleCDYErgh6BTpEPIoNvPTEBgkEBAQE Received: from 19.135-176-91.adsl-dyn.isp.belgacom.be (HELO kalimero.tijl.coosemans.org) ([91.176.135.19]) by relay.skynet.be with ESMTP; 03 Feb 2015 10:55:54 +0100 Received: from kalimero.tijl.coosemans.org (kalimero.tijl.coosemans.org [127.0.0.1]) by kalimero.tijl.coosemans.org (8.14.9/8.14.9) with ESMTP id t139trvQ001671; Tue, 3 Feb 2015 10:55:53 +0100 (CET) (envelope-from tijl@FreeBSD.org) Date: Tue, 3 Feb 2015 10:55:53 +0100 From: Tijl Coosemans To: Kubilay Kocak Subject: Re: svn commit: r378316 - head/devel/libhtp Message-ID: <20150203105553.3bbbeb6c@kalimero.tijl.coosemans.org> In-Reply-To: <54D01B67.4010406@FreeBSD.org> References: <201502021841.t12IfvP1021156@svn.freebsd.org> <54D01223.7020703@FreeBSD.org> <54D015B7.2080408@marino.st> <54D01B67.4010406@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII 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-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 Feb 2015 09:55:57 -0000 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.