From owner-svn-ports-head@freebsd.org Thu Aug 20 15:26:16 2015 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0FE1D9BFE38; Thu, 20 Aug 2015 15:26:16 +0000 (UTC) (envelope-from freebsd.contact@marino.st) 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 DB061107E; Thu, 20 Aug 2015 15:26:15 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: from [192.168.0.23] (unknown [130.255.19.36]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id 9BC3743BBF; Thu, 20 Aug 2015 10:26:06 -0500 (CDT) Reply-To: marino@freebsd.org Subject: Re: svn commit: r394871 - in head: Mk mail/thunderbird mail/thunderbird/files www/firefox www/firefox-esr www/firefox-esr/files www/firefox/files www/libxul www/libxul/files www/seamonkey www/seamonkey... References: <201508201316.t7KDGDPs042842@repo.freebsd.org> <55D5D3FC.9060001@marino.st> To: Jan Beich Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org, marino@freebsd.org From: John Marino X-Enigmail-Draft-Status: N1110 Message-ID: <55D5F186.5030807@marino.st> Date: Thu, 20 Aug 2015 17:25:58 +0200 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 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: Thu, 20 Aug 2015 15:26:16 -0000 On 8/20/2015 5:15 PM, Jan Beich wrote: > John Marino writes: >> By switching the comparison from < to >=, I think this just broke >> DragonFly which does not support jemalloc on any version. > > Ahem, OSVERSION=9999999 in DPorts' bsd.port.mk isn't sane either. > Either emulate it (along with OSREL) via USES=alias, derive from > __DragonFly_version or force breakage by not defining it. I disagree. Having it work 90% of the time by default is better than 0% of the time by default. OSVERSION only works for FreeBSD, period. there is no emulation possible. We have DFLYVERSION analog but that's not known in the stock version. With correct usage, OSVERSION is never used without OPSYS. Those are coupled data points. I "get" the argument that some people have that OPSYS should be assumed to be "FreeBSD" and thus doesn't need to be checked, but obviously that is not conducive to DragonFly support. > > --enable-jemalloc is a toggle for bundled jemalloc. Without it the build > defaults to system malloc. configure also checks if system malloc > supports jemalloc non-standard API. The API is mainly used for > about:memory. However, lack of malloc_usable_size() may impact > performance but it is usually available elsewhere (musl, glibc, etc). > > files/extra-patch-bug1125514 should be a nop for DragonFly, FreeBSD 9.x > and FreeBSD 10.x. It can cause garbage in memory stats for bundled jemalloc > but otherwise explicitly separated by make(1) conditional. > > My intent is to make DragonFly use bundled jemalloc like older releases > of FreeBSD. Try applying below commit under ${WRKSRC}/memory/jemalloc. Why? jemalloc has never worked on DF. Ever port with jemalloc support, bundled or otherwise fails. > To avoid undefined behavior OPSYS check has to be added twice for (large > and small) OSVERSION. Or maybe inverse it like or just the entire section in "If FreeBSD". If you don't want to adjust then we'll just have to carry another patch in the overlay. I'm not going to push the issue. Thanks, John