From nobody Sat Jan 13 17:41:06 2024 X-Original-To: dev-commits-ports-all@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 4TC5Ky2WP1z56GCj for ; Sat, 13 Jan 2024 17:41:10 +0000 (UTC) (envelope-from daniel.engberg.lists@pyret.net) Received: from smtp-42aa.mail.infomaniak.ch (smtp-42aa.mail.infomaniak.ch [84.16.66.170]) (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 (2048 bits) client-digest SHA256) (Client CN "relay.mail.infomaniak.ch", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4TC5Kx4qwhz4q0d; Sat, 13 Jan 2024 17:41:09 +0000 (UTC) (envelope-from daniel.engberg.lists@pyret.net) Authentication-Results: mx1.freebsd.org; none Received: from smtp-3-0001.mail.infomaniak.ch (unknown [10.4.36.108]) by smtp-3-3000.mail.infomaniak.ch (Postfix) with ESMTPS id 4TC5Kt4SJpzMpyjJ; Sat, 13 Jan 2024 17:41:06 +0000 (UTC) Received: from unknown by smtp-3-0001.mail.infomaniak.ch (Postfix) with ESMTPA id 4TC5Kt0gXbzMppFZ; Sat, 13 Jan 2024 18:41:06 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=pyret.net; s=20231006; t=1705167666; bh=a0I/9M1+QopHdCBESG+VtwMukA+vGMAFScEvZx1XsGQ=; h=Date:Subject:From:Reply-To:To:Cc:References:In-Reply-To:From; b=i2vIUzOQ1fqpx18t5IEKjd3OjUXIhFzyTsgALD56ZZ7u/yOjxgeFHX2WpoKN/2sOo wSyZLG1jEzYC5TkC9mI24jssIWhni37w8jM6BbP2oES6Nn0rIOrAK4mm61NwVJKy/U 2pDc1NQQ7Drqf9KG91d+WjZxq7Bry9h3r+1jfx385a0PLlmh+/soH9Wq9TTMwhRluy q+wXaC+XrWr457rFUQYFdJ8EnsPEJARNyuGXYFIiXk7AFprzNVw6JbHaEdi7n7C6Z4 bgnUrhYzopfj97+nhIIZxhmqRE2qqpauubOXis7nhVIdMWoSN42Z6CSe3rKxXAIhOM /+34WIBQbo4kQ== Message-ID: <6e4dca8719bcb044186a179e708a32a1@mail.infomaniak.com> Date: Sat, 13 Jan 2024 18:41:06 +0100 Subject: Re: git: 25e6f68a6661 - main - textproc/libxml2: Update to 2.11.6 From: Daniel Engberg Reply-To: Daniel Engberg To: Charlie Li Cc: Gleb Popov , Alexey Dokuchaev , Daniel Engberg , ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org List-Id: Commit messages for all branches of the ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-all@freebsd.org X-BeenThere: dev-commits-ports-all@freebsd.org MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="_=_swift_1705167666_1c7aecfa727f1d5b08d0577ff1d95a80_=_" X-WS-User-Origin: eyJpdiI6IlFWWEY2ODRuRlFDT28zMUF1eUdOMUE9PSIsInZhbHVlIjoiVWowMnVWOG40dStBcVAwMnpvQ1VnQT09IiwibWFjIjoiYTgwZmU4MGRhZTA1YWEzZGQ4YTJiMzZiMDhkZTc0Yzk1N2E1MDE2MGM5ZGFjZWFiOWY4NDFiMGZjMzYyM2I5ZSIsInRhZyI6IiJ9 X-WS-User-Mbox: eyJpdiI6IktpdUk1K3hyc2gybmkzdy8wQlpIdWc9PSIsInZhbHVlIjoicmdIajlWaGIwd2l1TnhQOG9iS0Rkdz09IiwibWFjIjoiOTIxZjFlMDQ2OTNkYzQzODZkYjNjZWM1ZTE0Mzg5YTMwZmQzMDc3MzkzY2RjMTZiMzY3MmMxNjVhYjBkZjE4MiIsInRhZyI6IiJ9 X-WS-Location: eJxzKUpMKykGAAfpAmU- X-Mailer: Infomaniak Workspace (1.3.620) References: <202401121705.40CH5JhG014492@gitrepo.freebsd.org> <02ebab1b-a763-45e4-9380-0d82c83c22a4@freebsd.org> In-Reply-To: <02ebab1b-a763-45e4-9380-0d82c83c22a4@freebsd.org> X-Infomaniak-Routing: alpha X-Rspamd-Queue-Id: 4TC5Kx4qwhz4q0d X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:29222, ipnet:84.16.64.0/19, country:CH] --_=_swift_1705167666_1c7aecfa727f1d5b08d0577ff1d95a80_=_ Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 2024-01-13T14:36:58.000+01:00, Charlie Li wrote:= >=C2=A0Daniel=C2=A0Engberg=C2=A0wrote: >>=C2=A0=C2=A0On=C2=A02024-0= 1-13T09:02:35.000+01:00,=C2=A0Gleb=C2=A0Popov=C2=A0 >= >=C2=A0=C2=A0wrote: >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0On=C2=A0Sat,= =C2=A0Jan=C2=A013,=C2=A02024=C2=A0at=C2=A08:14=C2=A0AM=C2=A0Alexey=C2=A0Dok= uchaev >>=C2=A0=C2=A0>=C2=A0=C2=A0 >>=C2=A0=C2= =A0=C2=A0>=C2=A0wrote: >>=C2=A0=C2=A0 >>=C2=A0=C2= =A0=C2=A0Wait,=C2=A0what?=C2=A0You've=C2=A0just=C2=A0removed=C2=A0the=C2= =A0comment=C2=A0which=C2=A0tells=C2=A0to=C2=A0NOT >>=C2=A0=C2=A0 >>= =C2=A0=C2=A0=C2=A0use=C2=A0the >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0dread= ed=C2=A0CMake=C2=A0and=C2=A0made=C2=A0the=C2=A0switch=C2=A0without=C2=A0log= ging=C2=A0any=C2=A0rationale? >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0CMake= >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0in=C2=A0such=C2=A0low-sitting=C2= =A0in=C2=A0dependency=C2=A0tree=C2=A0port=C2=A0is=C2=A0a=C2=A0PITA=C2=A0for= =C2=A0many >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0people, >>=C2=A0=C2= =A0 >>=C2=A0=C2=A0=C2=A0please=C2=A0don't=C2=A0push=C2=A0it=C2=A0down= =C2=A0our=C2=A0throats=C2=A0as=C2=A0you've=C2=A0been=C2=A0repeatedly >>= =C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0asked=C2=A0by >>=C2=A0=C2=A0 >>= =C2=A0=C2=A0=C2=A0other=C2=A0fellow=C2=A0developers. >>=C2=A0=C2=A0 >>= =C2=A0=C2=A0=C2=A0./danfe >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0I=C2=A0adm= it=C2=A0that=C2=A0there=C2=A0is=C2=A0no=C2=A0rationale=C2=A0for=C2=A0the= =C2=A0switch=C2=A0in=C2=A0the=C2=A0commit >>=C2=A0=C2=A0 >>=C2=A0=C2= =A0=C2=A0message.=C2=A0But=C2=A0at=C2=A0the=C2=A0same=C2=A0time,=C2=A0reduc= ing=C2=A0build=C2=A0times=C2=A0for=C2=A0Ports >>=C2=A0=C2=A0 >>=C2= =A0=C2=A0=C2=A0consumers=C2=A0has=C2=A0much=C2=A0lower=C2=A0priority=C2= =A0for=C2=A0us=C2=A0than=C2=A0actually=C2=A0updating >>=C2=A0=C2=A0 >>= =C2=A0=C2=A0=C2=A0software. >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0Daniel= =C2=A0made=C2=A0sure=C2=A0that=C2=A0the=C2=A0switch=C2=A0doesn't=C2=A0cause= =C2=A0circular >>=C2=A0=C2=A0dependencies >>=C2=A0=C2=A0 >>=C2=A0= =C2=A0=C2=A0which=C2=A0was=C2=A0a=C2=A0problem=C2=A0before.=C2=A0Because= =C2=A0of=C2=A0that=C2=A0I=C2=A0approved=C2=A0the >>=C2=A0=C2=A0change, = >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0just=C2=A0for=C2=A0the=C2=A0sake=C2= =A0of=C2=A0the=C2=A0"updating"=C2=A0part. >>=C2=A0=C2=A0 >>=C2=A0=C2= =A0=C2=A0Daniel=C2=A0is=C2=A0not=C2=A0being=C2=A0paid=C2=A0for=C2=A0this= =C2=A0work,=C2=A0so=C2=A0if=C2=A0he's=C2=A0more >>=C2=A0=C2=A0comfortable= >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0with=C2=A0CMake=C2=A0(which=C2= =A0I=C2=A0wholeheartedly=C2=A0understand!)=C2=A0it=C2=A0is=C2=A0a >>= =C2=A0=C2=A0sufficient >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0rationale= =C2=A0for=C2=A0the=C2=A0switch.=C2=A0I=C2=A0believe=C2=A0no=C2=A0one=C2= =A0will=C2=A0be=C2=A0against=C2=A0if=C2=A0you >>=C2=A0=C2=A0 >>=C2= =A0=C2=A0=C2=A0prepare=C2=A0a=C2=A0patch=C2=A0that=C2=A0will=C2=A0switch= =C2=A0back=C2=A0to=C2=A0autotools=C2=A0and=C2=A0do=C2=A0the >>=C2=A0= =C2=A0same >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0amount=C2=A0of=C2=A0testi= ng=C2=A0(500+=C2=A0ports)=C2=A0that=C2=A0Daniel=C2=A0did. >=C2=A0 >= =C2=A0=C2=A0I=C2=A0will=C2=A0be=C2=A0doing=C2=A0exactly=C2=A0this=C2=A0for= =C2=A0the=C2=A02.12=C2=A0update.=C2=A0Already=C2=A0have=C2=A0a >=C2=A0WIP= . >=C2=A0 >>=C2=A0=C2=A0Hi, >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0I= =C2=A0want=C2=A0to=C2=A0add=C2=A0a=C2=A0few=C2=A0things=C2=A0that=C2=A0were= n't=C2=A0in=C2=A0the=C2=A0commit=C2=A0message=C2=A0but >>=C2=A0=C2=A0that= 's=C2=A0 >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0been=C2=A0discussed=C2= =A0and=C2=A0mentioned=C2=A0in=C2=A0the=C2=A0bugs=C2=A0report(s)=C2=A0and= =C2=A0elsewhere. >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0Local=C2=A0patches= =C2=A0are=C2=A0a=C2=A0pain,=C2=A0no=C2=A0one=C2=A0has=C2=A0upstreamed=C2= =A0patches=C2=A0for=C2=A0years >>=C2=A0=C2=A0and=C2=A0 >>=C2=A0=C2= =A0 >>=C2=A0=C2=A0=C2=A0there's=C2=A0seemingly=C2=A0little=C2=A0to=C2= =A0no=C2=A0interest=C2=A0in=C2=A0doing=C2=A0so.=C2=A0For=C2=A0a=C2=A0single= >>=C2=A0=C2=A0 >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0committer=C2=A0tha= t=C2=A0might=C2=A0not=C2=A0be=C2=A0much=C2=A0of=C2=A0an=C2=A0issue=C2=A0but= =C2=A0it=C2=A0adds=C2=A0a >>=C2=A0=C2=A0overall=C2=A0 >>=C2=A0=C2=A0 = >>=C2=A0=C2=A0=C2=A0maintenance=C2=A0burden=C2=A0and=C2=A0that=C2=A0specifi= c=C2=A0maintainer/committer=C2=A0will >>=C2=A0=C2=A0not=C2=A0be=C2=A0 >= >=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0around=C2=A0forever.=C2=A0As=C2=A0a= =C2=A0community=C2=A0we've=C2=A0reported=C2=A0issues=C2=A0upstream, >>= =C2=A0=C2=A0issues=C2=A0 >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0have=C2= =A0been=C2=A0fixed=C2=A0and=C2=A0upstreamed=C2=A0patches.=C2=A0This=C2= =A0by=C2=A0far=C2=A0a=C2=A0better >>=C2=A0=C2=A0success=C2=A0 >>=C2= =A0=C2=A0 >>=C2=A0=C2=A0=C2=A0story=C2=A0than=C2=A0previously.=C2=A0It= =C2=A0isn't=C2=A0perfect=C2=A0but=C2=A0it=C2=A0is=C2=A0in=C2=A0better=C2= =A0shape >>=C2=A0=C2=A0than=C2=A0 >>=C2=A0=C2=A0 >>=C2=A0=C2=A0=C2= =A0before=C2=A0and=C2=A0people=C2=A0are=C2=A0willing=C2=A0to=C2=A0contribut= e.=C2=A0There=C2=A0are=C2=A0more >>=C2=A0=C2=A0improvments=C2=A0 >>= =C2=A0=C2=A0 >>=C2=A0=C2=A0=C2=A0upstream=C2=A0which=C2=A0be=C2=A0in= =C2=A0later=C2=A0releases=C2=A0too. >=C2=A0 >=C2=A0=C2=A0Until=C2=A0ups= tream=C2=A0specifically=C2=A0declares=C2=A0and=C2=A0recommends=C2=A0CMake= =C2=A0as=C2=A0ready >=C2=A0for=C2=A0 >=C2=A0 >=C2=A0Unix-like=C2= =A0systems=C2=A0in=C2=A0at=C2=A0least=C2=A0their=C2=A0documentation,=C2= =A0nothing=C2=A0else=C2=A0is >=C2=A0relevant. >=C2=A0 >=C2=A0--=C2= =A0 >=C2=A0 >=C2=A0Charlie=C2=A0Li >=C2=A0 >=C2=A0...nope,=C2=A0sti= ll=C2=A0don't=C2=A0have=C2=A0an=C2=A0exit=C2=A0line. Hi, Unless y= ou're planning to upstream patches you can already stop working on it. Th= is has been mention multiple times which you've so far ignored. There's l= ittle point in bumping before 2.13 is out which should be relatively soon= . Best regards, Daniel --_=_swift_1705167666_1c7aecfa727f1d5b08d0577ff1d95a80_=_ Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable
On 2024-01-13T14:36:58.000+01:00, Charlie Li <vishwin@freebsd.org&g= t; wrote:
Daniel=
 Engberg wrote:
  On 2024-0=
1-13T09:02:35.000+01:00, Gleb Popov <arrowd@freebsd.org> wrote:
On Sat, Jan 13, 2024 at 8:14 AM Alexey Dokuchaev <= ;danfe@freeb= sd.org
<= br>


Wait, what? You've j= ust removed the comment which tells to NOT
use the
dreaded CMake and made the switch without logging any= rationale?
CMake
in such low= -sitting in dependency tree port is a PITA for many
= people,
please don't push it down our throats as you= 've been repeatedly
asked by
= other fellow developers.

./danfe
=


I admit that there is no ra= tionale for the switch in the commit
message. But at the= same time, reducing build times for Ports
consumers has= much lower priority for us than actually updating
softw= are.
Daniel made sure that the switch doesn't cause circ= ular dependencies
which was a problem before. Because of= that I approved the change,
just for the sake of the "u= pdating" part.

Daniel is not being paid = for this work, so if he's more comfortable
with CMake (w= hich I wholeheartedly understand!) it is a sufficient
ra= tionale for the switch. I believe no one will be against if you
prepare a patch that will switch back to autotools and do the same<= br>
amount of testing (500+ ports) that Daniel did.

I will be doing exactly this for the 2.= 12 update. Already have a WIP.
<= div> Hi,

I want to add a few things that we= ren't in the commit message but that's
been discussed and m= entioned in the bugs report(s) and elsewhere.

Local patches are a pain, no one has upstreamed patches for years and
there's seemingly little to no interest in doing so. For a sin= gle
committer that might not be much of an issue but it add= s a overall
maintenance burden and that specific maintainer= /committer will not be
around forever. As a community we've= reported issues upstream, issues
have been fixed and upstr= eamed patches. This by far a better success
story than prev= iously. It isn't perfect but it is in better shape than
bef= ore and people are willing to contribute. There are more improvments
upstream which be in later releases too.

Until upstream specifically declares and recommends CMa= ke as ready for
Unix-like systems in at least their document= ation, nothing else is relevant.

--
=
Charlie Li
...nope, still don't have an exit line.
Hi,

Unless y= ou're planning to upstream patches you can already stop working on it. This= has been mention multiple times which you've so far ignored. There's littl= e point in bumping before 2.13 is out which should be relatively soon.
<= /div>

Best regards,
Daniel
<= br>

--_=_swift_1705167666_1c7aecfa727f1d5b08d0577ff1d95a80_=_--