From nobody Fri Oct 14 18:58:33 2022 X-Original-To: dev-commits-ports-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Mpwdw4wXmz4g498; Fri, 14 Oct 2022 18:58:44 +0000 (UTC) (envelope-from pkubaj@anongoth.pl) Received: from mail.anongoth.pl (mail.anongoth.pl [46.248.190.61]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X448 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "anongoth.pl", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Mpwds5sMXz3TNW; Fri, 14 Oct 2022 18:58:41 +0000 (UTC) (envelope-from pkubaj@anongoth.pl) Received: from anongoth.pl (unknown [192.168.1.15]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) (Authenticated sender: pkubaj@anongoth.pl) by mail.anongoth.pl (Postfix) with ESMTPSA id 8F23D1BFE01; Fri, 14 Oct 2022 20:58:34 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=anongoth.pl; s=ANONGOTH; t=1665773914; bh=8eASaFQ0bruTScPur/wzVbTddTb8fiuvd+lDKPxZSMY=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=ZQyvuAnAMsjVxXelQP0dQbZeHKFBk19Sp3Rz0BwGHPgsmAAGvdnd3OFqDWrakV2qx Wxhi/WbJG9zTMApqs2BIncE1/l/0/Xnorp2WA42Lut2Evi256MKHz7TpP5CgdtlPHR 8WGwX4pvevtokXw9KncAk7Uzw2hwLpdoezMhFgCuJGKh87KQwf2dxgYm/nof3+DWm0 Q4ZPt6DdEw5/21Zx6+V0SMy5IX+wTK6a8WLf1WrdZ41/sosX2/lnkhpp0UszS1XTUN H9mg1iwH8AUqz00pR1ABCazt0HwFlPlm6JaZwQRxup4ux39n9waEbfBjtw7x7kVHwV HAI6eGODtCG9Q== Date: Fri, 14 Oct 2022 20:58:33 +0200 From: Piotr Kubaj To: Adam Weinberger Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 9d241a90370c - main - Mk: Add USES=luajit Message-ID: References: <202210141707.29EH7i08013647@gitrepo.freebsd.org> List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="YQOvFm5CyfdOppTU" Content-Disposition: inline In-Reply-To: <202210141707.29EH7i08013647@gitrepo.freebsd.org> X-Rspamd-Queue-Id: 4Mpwds5sMXz3TNW X-Spamd-Bar: ----- Authentication-Results: mx1.freebsd.org; dkim=pass header.d=anongoth.pl header.s=ANONGOTH header.b=ZQyvuAnA; dmarc=pass (policy=reject) header.from=anongoth.pl; spf=pass (mx1.freebsd.org: domain of pkubaj@anongoth.pl designates 46.248.190.61 as permitted sender) smtp.mailfrom=pkubaj@anongoth.pl X-Spamd-Result: default: False [-5.04 / 15.00]; SIGNED_PGP(-2.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_SHORT(-1.00)[-1.000]; MID_RHS_NOT_FQDN(0.50)[]; DMARC_POLICY_ALLOW(-0.50)[anongoth.pl,reject]; NEURAL_HAM_MEDIUM(-0.44)[-0.443]; R_SPF_ALLOW(-0.20)[+ip4:46.248.190.61]; MIME_GOOD(-0.20)[multipart/signed,text/plain]; R_DKIM_ALLOW(-0.20)[anongoth.pl:s=ANONGOTH]; MIME_TRACE(0.00)[0:+,1:+,2:~]; MLMMJ_DEST(0.00)[dev-commits-ports-all@freebsd.org,dev-commits-ports-main@freebsd.org]; ASN(0.00)[asn:47544, ipnet:46.248.160.0/19, country:PL]; FROM_EQ_ENVFROM(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; RCVD_COUNT_TWO(0.00)[2]; FREEFALL_USER(0.00)[pkubaj]; ARC_NA(0.00)[]; DKIM_TRACE(0.00)[anongoth.pl:+]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; RCPT_COUNT_THREE(0.00)[4]; TO_MATCH_ENVRCPT_ALL(0.00)[]; RCVD_TLS_ALL(0.00)[] X-ThisMailContainsUnwantedMimeParts: N --YQOvFm5CyfdOppTU Content-Type: text/plain; charset=utf-8; format=flowed Content-Disposition: inline On 22-10-14 17:07:44, Adam Weinberger wrote: >The branch main has been updated by adamw: > >URL: https://cgit.FreeBSD.org/ports/commit/?id=9d241a90370c9410d997be37db5e3a5657bdb4ca > >commit 9d241a90370c9410d997be37db5e3a5657bdb4ca >Author: Adam Weinberger >AuthorDate: 2022-10-14 16:50:39 +0000 >Commit: Adam Weinberger >CommitDate: 2022-10-14 17:07:39 +0000 > > Mk: Add USES=luajit > > USES=luajit centralizes luajit dependencies via bsd.default-versions.mk > (defaulting to luajit-devel). > > Ports currently depend on: > lang/luajit: 42 > lang/luajit-devel: 0 > lang/luajit-openresty: 51 > > The difficulty here is that: > - They all conflict with each other > - Upstream is very clear about not intending for users to use the > version in lang/luajit at all, and wants all luajit users to use > lang/luajit-devel > - luajit-openresty is very aggressive about being drop-in compatible > with luajit-devel (though not the other direction) > > For the vast majority of situations, USES=luajit will be sufficient. > Ports that use openresty's API additions can specify > USES=luajit:openresty. > > I'm adding this into head very early in the 2022Q4 cycle so that there's > ample time to address the default. It's a difficult decision. osa, the > maintainer of luajit and luajit-devel, prefers that the default be > luajit, but I went the other direction because it's in direct conflict > with upstream's clear stated intention. It's not a simple decision; see > the review below for a little discussion on it. > > The TL;DR here is that this defaults to the one luajit version that no > port currently depends on. However, this version is what OpenBSD, > Homebrew, Ubuntu, Debian, Fedora, MacPorts, and Arch default to (and > none of them even provide the stale version lang/luajit provides). > This will bring FreeBSD in line with nearly every other OS package > system, and with upstream's intention. > > See also PRs 225342, 257051. > > Reviewed by: bapt, tcberner > Approved by: portmgr (bapt, tcberner) > Differential Revision: https://reviews.freebsd.org/D36947 >--- > Mk/Uses/luajit.mk | 27 +++++++++++++++++++++++++++ > Mk/bsd.default-versions.mk | 9 ++++++--- > 2 files changed, 33 insertions(+), 3 deletions(-) > >diff --git a/Mk/Uses/luajit.mk b/Mk/Uses/luajit.mk >new file mode 100644 >index 000000000000..75e54407d484 >--- /dev/null >+++ b/Mk/Uses/luajit.mk >@@ -0,0 +1,27 @@ >+# Select your favorite luajit runtime >+# Feature: luajit >+# Usage: USES= luajit[:version] >+# MAINTAINER: adamw@FreeBSD.org >+ >+.if !defined(_INCLUDE_USES_LUAJIT_MK) >+_INCLUDE_USES_LUAJIT_MK=yes >+ >+.if !empty(luajit_ARGS) >+LUAJIT_VER= ${luajit_ARGS} >+.else >+LUAJIT_VER= ${LUAJIT_DEFAULT} >+.endif >+ >+# When adding a version, please keep the comment in >+# Mk/bsd.default-versions.mk in sync >+VALID_LUAJIT_VER= luajit luajit-devel luajit-openresty >+ >+.if empty(VALID_LUAJIT_VER:M${LUAJIT_DEFAULT}) >+IGNORE= Invalid luajit default version ${LUAJIT_DEFAULT}: valid versions are ${VALID_LUAJIT_VER} >+.elif empty(VALID_LUAJIT_VER:M${LUAJIT_VER}) >+IGNORE= Invalid luajit version ${LUAJIT_VER}: valid versions are ${VALID_LUAJIT_VER} >+.endif >+ >+LIB_DEPENDS= libluajit-5.1.so:lang/${LUAJIT_VER} >+ >+.endif >diff --git a/Mk/bsd.default-versions.mk b/Mk/bsd.default-versions.mk >index bbf3d5880c70..5a36e0b9c4d0 100644 >--- a/Mk/bsd.default-versions.mk >+++ b/Mk/bsd.default-versions.mk >@@ -17,9 +17,10 @@ _INCLUDE_BSD_DEFAULT_VERSIONS_MK= yes > > LOCALBASE?= /usr/local > >-. for lang in APACHE BDB COROSYNC EMACS FIREBIRD FORTRAN FPC GCC GHOSTSCRIPT GL GO \ >- IMAGEMAGICK JAVA LAZARUS LIBRSVG2 LINUX LLVM LUA MONO MYSQL NINJA NODEJS PERL5 \ >- PGSQL PHP PYTHON PYTHON2 PYTHON3 RUBY RUST SAMBA SSL TCLTK VARNISH >+. for lang in APACHE BDB COROSYNC EMACS FIREBIRD FORTRAN FPC GCC \ >+ GHOSTSCRIPT GL GO IMAGEMAGICK JAVA LAZARUS LIBRSVG2 LINUX LLVM \ >+ LUA LUAJIT MONO MYSQL NINJA NODEJS PERL5 PGSQL PHP PYTHON \ >+ PYTHON2 PYTHON3 RUBY RUST SAMBA SSL TCLTK VARNISH > . if defined(${lang}_DEFAULT) > ERROR+= "The variable ${lang}_DEFAULT is set and it should only be defined through DEFAULT_VERSIONS+=${lang:tl}=${${lang}_DEFAULT} in /etc/make.conf" > . endif >@@ -86,6 +87,8 @@ LLVM_DEFAULT?= 90 > . endif > # Possible values: 5.1, 5.2, 5.3, 5.4 > LUA_DEFAULT?= 5.2 >+# Possible values: luajit, luajit-devel, luajit-openresty >+LUAJIT_DEFAULT?= luajit-devel > # Possible values: 5.10, 5.20, 6.8 > MONO_DEFAULT= 5.10 > # Possible values: 5.6, 5.7, 8.0, 10.3m, 10.4m, 10.5m, 5.7p, 5.7w Could you make USES=luajit default to openresty on powerpc64 and powerpc64le? lang/luajit and lang/luajit-devel don't support those, but openresty fork does. I believe aarch64 is in similar situation, except that it's also supported by luajit-devel. --YQOvFm5CyfdOppTU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQJkBAABCgBOFiEEycyIeNkkgohzsoorelmbhSCDnJ0FAmNJsVkwFIAAAAAAFQAS cGthLWFkZHJlc3NAZ251cGcub3JncGt1YmFqQGFub25nb3RoLnBsAAoJEHpZm4Ug g5ydo6kP/0fNSZzyoPHM5zGIwbrkZLtP0Mu8gW7OEZMqkShVQxtdVAGR1v7GJuzK xJAtdTIBpCgoIfhZmFaqY+S8SuQjj1L0B9rtoT1IFBthIFaxFp8Vd2Yle0w6e6qs kiIrIhvWLApIcubzz1/XDB9UGlyiOxf8kRQYxAUY+N28LlWWrdhSyY5rXerl+hUi mm2E5f6/wgAiFXxLTSw4ZcsrGEG7sZ0Ukoi3aJog+q0TQ3kY99Sp7xzzLkBckV1c cVLFudKdRjioer4+YcblltB19WUB5Kk508U9TmD9hdmOvwkn2fmD2WHhcQ/+P8qB DK3WshABZumSZi5jAInTHLArl4YWctgcxvnkuK/7Ca2sSUFuX7ZjknXakDOqqMh0 5M8+gB5FLejSfGxiLCV4KUeqrWWzabaE+TFiXko2HjvIbbZwsJH3wH8D2mJtRiVL Be8nQ2rnqkLXHM1e76PFJYB+23O6NCs1xZxdmrqbAVhRSsTJ1vDvpBtq67E/Jlpg z0M9fiml8QjVZo6EIj2wsbHN/wJXsZtkY6NyvuJTYTqgMEQIy3MmRlf8GezbkQqI xi8pvM/A6ABlRGR6ftSWH9fF2SbJqsg8VNNI3++cXBVQisoVimDsYpzy3LCirIl7 wFKD+cZDBFvxl/ZCUZbZD73Avrxer2Mdqawei36qNRrmaOfuQW6q =t1Md -----END PGP SIGNATURE----- --YQOvFm5CyfdOppTU--