From owner-dev-commits-ports-main@freebsd.org Wed Jul 7 07:00:49 2021 Return-Path: Delivered-To: dev-commits-ports-main@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 991E164FB65; Wed, 7 Jul 2021 07:00:49 +0000 (UTC) (envelope-from mat@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 4GKVgj3hfDz3KYN; Wed, 7 Jul 2021 07:00:49 +0000 (UTC) (envelope-from mat@freebsd.org) Received: from mail.j.mat.cc (owncloud.cube.mat.cc [IPv6:2a01:678:4:1::228]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mail.mat.cc", Issuer "R3" (verified OK)) (Authenticated sender: mat/mail) by smtp.freebsd.org (Postfix) with ESMTPSA id 4B292207BC; Wed, 7 Jul 2021 07:00:49 +0000 (UTC) (envelope-from mat@freebsd.org) Received: from aching.in.mat.cc (unknown [IPv6:2a01:678:42:0:9675:34d6:9907:ada4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: mat@mat.cc) by mail.j.mat.cc (Postfix) with ESMTPSA id 64EAF942D80; Wed, 7 Jul 2021 07:00:46 +0000 (UTC) Date: Wed, 7 Jul 2021 09:00:45 +0200 From: Mathieu Arnold To: =?utf-8?B?TG/Dr2M=?= Bartoletti Cc: Christoph Moench-Tegeder , ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: be86c4fe23e0 - main - misc/qtchooser: remove Message-ID: <20210707070045.zrtfkdkblrbxesri@aching.in.mat.cc> References: <202107041732.164HW8HQ076467@gitrepo.freebsd.org> <2569fab0-d643-e22a-0d3a-2e4df497c75f@FreeBSD.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="4md2ixvavszwspc4" Content-Disposition: inline In-Reply-To: <2569fab0-d643-e22a-0d3a-2e4df497c75f@FreeBSD.org> X-BeenThere: dev-commits-ports-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commits to the main branch of the FreeBSD ports repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 07 Jul 2021 07:00:49 -0000 --4md2ixvavszwspc4 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jul 05, 2021 at 09:08:15AM +0200, Lo=EFc Bartoletti wrote: > Hello, >=20 > Le 04/07/2021 =E0 23:28, Christoph Moench-Tegeder a =E9crit=A0: > > ## Lo=EFc Bartoletti (lbartoletti@FreeBSD.org): > > This: > >=20 > > > misc/qtchooser: remove > > Does more than it says on the label: > Indeed, removing a tool like this one cannot be done without patching (ma= ny) > other ports. > >=20 > > > --- a/cad/freecad/Makefile > > > +++ b/cad/freecad/Makefile > > > -USES=3D dos2unix compiler:c++11-lib cmake gl eigen:3 fortran jpeg \ > > > +USES=3D dos2unix compiler:c++17-lang cmake gl eigen:3 fortran jpeg \ > > > + -DBUILD_ENABLE_CXX_STD=3DC++17 > Sorry, this line is not needed to build cad/freecad without qtchooser. >=20 > I added it during my tests, because I saw that it was necessary to reduce > some warnings, but it's not useful for this change. >=20 > > I'd appricate knowing why these additional changes happen - it reduces > > annoyances on my side when these changes interfere with other patches > > I've lined up (yeah, I should just commit faster, but sometimes live > > happens). >=20 > We have communicated in the past, publicly or directly with those affecte= d, > like you, for this kind of improvement. >=20 > Although this initiative has been public for several months, it may not h= ave > been visible. >=20 > I will ensure better communication for the next times. I think the problem Christoph is pointing out is that the commit says "remove misc/qtchooser" and you are changing which compiler is needed in cad/freecad, which has absolutely nothing to do with removing misc/qtchooser. It is either a mistake and should not have been committed, or it should have been committed in a separate commit, that would say why you are changing the compiler needed. Also, it feels like you're trying to commit stuff under the hood hoping nobody will notice. --=20 Mathieu Arnold --4md2ixvavszwspc4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQKTBAABCgB9FiEEVhwchfRfuV0unqO5KesJApEdfgIFAmDlURhfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDU2 MUMxQzg1RjQ1RkI5NUQyRTlFQTNCOTI5RUIwOTAyOTExRDdFMDIACgkQKesJApEd fgJrag/+KliG5IcqiEnbqcHTHJrQ5/2qboI4mGMlVk0YV5PMu4oE7FdKekO6CTlI G3QzCbXPbldA2ZcRgtet4tG23C3TautB6ocRrwJ/TlB24HFOe7mx+HBMLZtNlgiA w1pz30Na3kXyAKUAQw/tC5wiaZ9v+mV86Lsou/L581ZqxrKaoIfF76lkT63AF2tp XI5tdOj2wdF2YS7RCoWxH3mrBtjC0USVjpaw2FWPDnWZ7pj59UEAfhbz11cN/XQM L4Wq5P1Lbx1/z4BbL6Z7Oz4BW8vvaUJ0pR9onKKtZ1K0TItadsJhc/dR1wLI539E zSG9BBxl8RGSWaIq87MOJYI3x1DVqBLb3a2OtbiNk0R5dTbyQ0EAywthJwxAP8Rg hAq/NE5Nfw9prJXtK3N5yg/BFtRrzAuWb0CJiKYZK/oBTrOG6QnDWIZdC0yc2QOS 4n6xUZteeGWAKFUItfrqgpj+61/w7/cDeEgRbBmxe9qZpErCkl6XO76EpJlmZQbC 0flvpdfE/fcM2m6HA+g3CiqiSmcHiHQcS34FbZGtDxZwDSUQi+ITgYOFy3gK3zbK 2Jh4BCrSfIzHgYlVDv3fzSRGpJKu7GO9CfNeabliJ3HizDQTzO22EV/vTQb7xF84 exax7cCFdaQkfU8JFJvDxvhrsUltLiJoCY7WmBDRzsujZnjEtlk= =Vt+k -----END PGP SIGNATURE----- --4md2ixvavszwspc4--