From owner-svn-ports-all@freebsd.org Fri Mar 26 11:28:57 2021 Return-Path: Delivered-To: svn-ports-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 0A26E5BB34E; Fri, 26 Mar 2021 11:28:57 +0000 (UTC) (envelope-from dbaio@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [IPv6:2610:1c1:1:606c::24b:4]) (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-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4F6KVc6xSbz4XZq; Fri, 26 Mar 2021 11:28:56 +0000 (UTC) (envelope-from dbaio@FreeBSD.org) Received: from t480.local (unknown [177.132.127.71]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) (Authenticated sender: dbaio/mail) by smtp.freebsd.org (Postfix) with ESMTPSA id DE5282691B; Fri, 26 Mar 2021 11:28:55 +0000 (UTC) (envelope-from dbaio@FreeBSD.org) Date: Fri, 26 Mar 2021 08:28:51 -0300 From: "Danilo G. Baio" To: Gleb Popov Cc: ports-committers , svn-ports-all , svn-ports-head Subject: Re: svn commit: r569233 - head/lang/ghc Message-ID: <20210326112851.pivtavly5kbvsvqr@t480.local> References: <202103260044.12Q0idbc009254@repo.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="63cxukkc2him3g63" Content-Disposition: inline In-Reply-To: X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.34 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: Fri, 26 Mar 2021 11:28:57 -0000 --63cxukkc2him3g63 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Mar 26, 2021 at 11:06:32AM +0300, Gleb Popov wrote: > On Fri, Mar 26, 2021 at 3:44 AM Danilo G. Baio wrote: >=20 > > Author: dbaio > > Date: Fri Mar 26 00:44:39 2021 > > New Revision: 569233 > > URL: https://svnweb.freebsd.org/changeset/ports/569233 > > > > Log: > > lang/ghc: Generate PORTDOCS pkg-plist dynamically > > > > This is the best approach for ports building its documentation with > > Sphinx in > > html and will save time on future Sphinx updates. > > > > Approved by: portmgr blanket > > > > Modified: > > head/lang/ghc/Makefile > > head/lang/ghc/pkg-plist > > > > Modified: head/lang/ghc/Makefile > > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > > --- head/lang/ghc/Makefile Fri Mar 26 00:40:44 2021 (r56923= 2) > > +++ head/lang/ghc/Makefile Fri Mar 26 00:44:39 2021 (r56923= 3) > > @@ -84,6 +84,8 @@ BOOT_LLVM_VERSION=3D 90 > > > > PLIST_SUB=3D GHC_VERSION=3D${GHC_VERSION} > > CABAL_VERSION=3D${CABAL_VERSION} > > > > +PORTDOCS=3D * > > + > > .include > > > > # This version of ncurses is needed by bootstrap compiler > > >=20 > I spent quite a time to make the pkg-plist fully static, and now you make > such a change without consulting the port's maintainer. >=20 > I was thinking that static pkg-plist's are generally favored over > dynamically-generated ones. Why usual "make makeplist" didn't work for yo= u? > Why didn't you ask me to help you with Sphinx upgrade? Note that this is not a plist recommendation in general, just for documentation html and Sphinx. I've been doing this kind of changes since I started maintaining Sphinx and it's becoming easier to maintain/update Sphinx and all ports that use it over time. In this case there was an issue reported on Twitter, lang/ghc wasn't building with the older Sphinx, I think this kind of issue isn't nice because the port clearly works with both versions (yes I could set a Sphinx version >=3D 3.5.2), but this is a ports/plist/sphinx issue. The improvement here was tested on my set of poudriere jails with all options enabled, all options disabled, mixed values on Sphinx 3.5.2 and 3.4.3, I consider this an improvement for lang/ghc, which was made in others important ports as well. In the future, I'll surely ask for your help, thank you. Regards. --=20 Danilo G. Baio (dbaio) --63cxukkc2him3g63 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQKTBAABCgB9FiEEORj0UTsjzCy+enIkmpN7LfMuiNcFAmBdxW9fFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDM5 MThGNDUxM0IyM0NDMkNCRTdBNzIyNDlBOTM3QjJERjMyRTg4RDcACgkQmpN7LfMu iNc1txAAskeeohV8LOBvJ19gDaCIsZvwD/e5LmsqPKt/nLGPMFNc4h/WgmMPsHps hZNXwp+yOoxpp7cPZ+K3vZz+7viM4Iw5FLogsvryXHGb49gteFN0yYa42dmX4G49 WB2rH6BOrFNbgNQLEdVaqfRoGzNpMjdGzSn81fK8+qWhVoNY7Ows/8a0GL/j35zA eI0Knm+RIptVsLPGIn9Kgo67ujAOXUz+3XaLrV4qCSQtYQo3I3tmUSGGQf5FGe8Q RZvPAndLdH4fZ0e+2K6QGWvjmNN2IcrPYLuyogNltsV/gcHDiHKaUWy9M1HD+Usw SeamY8EPjqb+t5gXCNNI4rSQ8a1DcWJmaC7LL0V1UxGBcAmMZUjDK+69HSJfOwwM s403Zn7N5QTcGZSVWHyJujQU6ygO4D+J5Ivg196tNBvAUorDStMOuH1l5GgKtNNR ox10uHfYazSvkTT75IgXK732qZe+DSb8g542JEyblYbYvCvJWbWUU7UCG/2RYrNB TkWYm5k8l0pzZE2Niu0RPhqqYUjEzcrqV8b4aoM95KVYwUpuav98xWFs5lvaNPtL sM2Lfwm2Zoq2DnqE9B7iqEcD3T+UGSaCThM0VXqm+u9sOuh+xSiR/xrGtplsK1Fh PfXcgyMsne9AQv4pEty56AE6t6vLvhZe3RQSugOPJ/xVpD5+65k= =CKab -----END PGP SIGNATURE----- --63cxukkc2him3g63--