Date: Mon, 3 Feb 2025 10:43:19 -0800 From: Kevin Oberman <rkoberman@gmail.com> To: Yasuhiro Kimura <yasu@freebsd.org> Cc: kiwi@freebsd.org, tatsuki_makino@hotmail.com, freebsd-ports@freebsd.org Subject: Re: Strange version inconsistency in Samba t* utils (e.g. talloc) Message-ID: <CAN6yY1ua23dZwJYuqjRFrEqdwCUz1FSqBfOXAS14P=boPN17Qg@mail.gmail.com> In-Reply-To: <20250203.030645.406589481977955710.yasu@FreeBSD.org> References: <CAN6yY1su=zjkZa9w-H7vJe9XHXoU0bkSrBG0EqcAYSWv4Z2t_w@mail.gmail.com> <SI2PR01MB5036DD9784998FB0728B1C56FAEA2@SI2PR01MB5036.apcprd01.prod.exchangelabs.com> <1453319087.51758.1738487855881.JavaMail.zimbra@oav.net> <20250203.030645.406589481977955710.yasu@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000006b2289062d4145d3 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Sun, Feb 2, 2025 at 10:08=E2=80=AFAM Yasuhiro Kimura <yasu@freebsd.org> = wrote: > From: Xavier Beaudouin <kiwi@freebsd.org> > Subject: Re: Strange version inconsistency in Samba t* utils (e.g. talloc= ) > Date: Sun, 2 Feb 2025 10:17:35 +0100 (CET) > > > I understand, but samba is really picky with their "satellite" librarie= s. > > When you don't use them as embedded tools then you can have strange > results > > if the version it not the one needed. > > For example, net/samba420 needs tdb >=3D 1.4.10, but other net/samba ma= y > not > > work with this version. > > > > This is very unfortunate but also some other tools like sssd2 and sudo > may > > need this libraries as external when compiled with samba support. > > > > Kind regards, > > Xavier > > > > When net/samba420 was added, following ports were added as > dependencies. > > * databases/tdb1410 > * devel/talloc242 > * devel/tevent016 > > But I think we should have taken following steps. > > 1. Repocopy databases/tdb, devel/talloc and devel/tevent to > databases/tdb149, devel/talloc241 and devel/tevent015. > 2. Update dependencies of net/samba416 and net/samba419 from the > former to the latter. > 3. Update databases/tdb, devel/talloc and devel/tevent to the versions > that Smaba 4.20 requires. > > The reason is as following. > > Right now there are 3 Samba ports in ports tree. That is, > > * net/samba416 > * net/samba419 > * net/samba420 > > net/samba416 has already reached its EoL and net/samba419 security > fixes only phase. So it is unlikely they requires update of > dependecies. You may say it is also unlikely net/samba420 requires > update. But when newer version of Samba, for example 4.21, has come to > ports tree, it is very likely it requires newer version of > dependencies and it is still possible net/samba420 works fine with > versions required by 4.21 as is the case with net/samba416 and > net/samba419. And when it really happens all we do is simply updating > generic databases/tdb, devel/talloc and devel/tevent rather than > creating new specific databases/tdbNNNN, devel/tallocNNN and > devel/teventNNN ports and removing old ones. > > This is common practice in ports tree. And by following it we can > minimize the frequency to create and remove port with specific > version. > > --- > Yasuhiro Kimura > This explains what I am seeing, but one issue is that bsd.default-versions.mk still shows EOL samba416 as the default version. Is there a good reason that this has not been updated to either 419 or 420? I obviously don't like running EOL versions, but I also fear that running something other than the default will break a port dependent on it. I assume that maintiners more aware of these issues will deal with them and, in the case of samba, I really don't use it, but it gets pulled in by filesystems/gvfs. --=20 Kevin Oberman, Part time kid herder and retired Network Engineer E-mail: rkoberman@gmail.com PGP Fingerprint: D03FB98AFA78E3B78C1694B318AB39EF1B055683 --0000000000006b2289062d4145d3 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><div class=3D"gmail_default" style=3D"fon= t-family:tahoma,sans-serif;font-size:small">On Sun, Feb 2, 2025 at 10:08=E2= =80=AFAM Yasuhiro Kimura <<a href=3D"mailto:yasu@freebsd.org">yasu@freeb= sd.org</a>> wrote:</div></div><div class=3D"gmail_quote gmail_quote_cont= ainer"><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;= border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Xavier Beaud= ouin <<a href=3D"mailto:kiwi@freebsd.org" target=3D"_blank">kiwi@freebsd= .org</a>><br> Subject: Re: Strange version inconsistency in Samba t* utils (e.g. talloc)<= br> Date: Sun, 2 Feb 2025 10:17:35 +0100 (CET)<br> <br> > I understand, but samba is really picky with their "satellite&quo= t; libraries.<br> > When you don't use them as embedded tools then you can have strang= e results <br> > if the version it not the one needed.<br> > For example, net/samba420 needs tdb >=3D 1.4.10, but other net/samb= a may not<br> > work with this version.<br> > <br> > This is very unfortunate but also some other tools like sssd2 and sudo= may<br> > need this libraries as external when compiled with samba support.<br> > <br> > Kind regards,<br> > Xavier<br> > <br> <br> When net/samba420 was added, following ports were added as<br> dependencies.<br> <br> * databases/tdb1410<br> * devel/talloc242<br> * devel/tevent016<br> <br> But I think we should have taken following steps.<br> <br> 1. Repocopy databases/tdb, devel/talloc and devel/tevent to<br> =C2=A0 =C2=A0databases/tdb149, devel/talloc241 and devel/tevent015.<br> 2. Update dependencies of net/samba416 and net/samba419 from the<br> =C2=A0 =C2=A0former to the latter.<br> 3. Update databases/tdb, devel/talloc and devel/tevent to the versions<br> =C2=A0 =C2=A0that Smaba 4.20 requires.<br> <br> The reason is as following.<br> <br> Right now there are 3 Samba ports in ports tree. That is,<br> <br> * net/samba416<br> * net/samba419<br> * net/samba420<br> <br> net/samba416 has already reached its EoL and net/samba419 security<br> fixes only phase. So it is unlikely they requires update of<br> dependecies. You may say it is also unlikely net/samba420 requires<br> update. But when newer version of Samba, for example 4.21, has come to<br> ports tree, it is very likely it requires newer version of<br> dependencies and it is still possible net/samba420 works fine with<br> versions required by 4.21 as is the case with net/samba416 and<br> net/samba419. And when it really happens all we do is simply updating<br> generic databases/tdb, devel/talloc and devel/tevent rather than<br> creating new specific databases/tdbNNNN, devel/tallocNNN and<br> devel/teventNNN ports and removing old ones.<br> <br> This is common practice in ports tree. And by following it we can<br> minimize the frequency to create and remove port with specific<br> version.<br> <br> ---<br> Yasuhiro Kimura<br> </blockquote></div><div><br clear=3D"all"></div><div style=3D"font-family:t= ahoma,sans-serif;font-size:small" class=3D"gmail_default">This explains wha= t I am seeing, but one issue is that <a href=3D"http://bsd.default-versions= .mk">bsd.default-versions.mk</a> still shows EOL samba416 as the default ve= rsion. Is there a good reason that this has not been updated to either 419 = or 420? I obviously don't like running EOL versions, but I also fear th= at running something other than the default will break a port dependent on = it. I assume that maintiners more aware of these issues will deal with them= and, in the case of samba, I really don't use it, but it gets pulled i= n by filesystems/gvfs.</div><span class=3D"gmail_signature_prefix">-- </spa= n><br><div dir=3D"ltr" class=3D"gmail_signature"><div dir=3D"ltr"><div><div= dir=3D"ltr"><div><div dir=3D"ltr"><div><div dir=3D"ltr">Kevin Oberman, Par= t time kid herder and retired Network Engineer<br>E-mail: <a href=3D"mailto= :rkoberman@gmail.com" target=3D"_blank">rkoberman@gmail.com</a><br></div><d= iv>PGP Fingerprint: D03FB98AFA78E3B78C1694B318AB39EF1B055683</div></div></d= iv></div></div></div></div></div></div> --0000000000006b2289062d4145d3--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAN6yY1ua23dZwJYuqjRFrEqdwCUz1FSqBfOXAS14P=boPN17Qg>