Date: Sun, 7 Apr 2019 18:11:58 +0200 From: Mariusz Zaborski <oshogbo@freebsd.org> To: Cy Schubert <Cy.Schubert@cschubert.com> Cc: rgrimes@freebsd.org, Shawn Webb <shawn.webb@hardenedbsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r345982 - in head: include lib/libc/sys sys/cddl/compat/opensolaris/sys sys/compat/cloudabi sys/compat/freebsd32 sys/compat/linux sys/kern sys/sys sys/ufs/ffs Message-ID: <20190407161158.GA40247@x-wing> In-Reply-To: <201904071535.x37FZ7bk073860@slippy.cwsent.com> References: <freebsd@gndrsh.dnsmgr.net> <201904071510.x37FA7tm050626@gndrsh.dnsmgr.net> <201904071535.x37FZ7bk073860@slippy.cwsent.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--ibTvN161/egqYuK8 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable In the https://wiki.freebsd.org/AddingSyscalls we mentions that we need to = bump __FreeBSD_version. I confirmed that with Warner. So this was my mistake. Thanks Shawn. --=20 Mariusz Zaborski oshogbo//vx | http://oshogbo.vexillium.org FreeBSD committer | https://freebsd.org Software developer | http://wheelsystems.com If it's not broken, let's fix it till it is!!1 On Sun, Apr 07, 2019 at 08:35:07AM -0700, Cy Schubert wrote: > In message <201904071510.x37FA7tm050626@gndrsh.dnsmgr.net>, "Rodney W.=20 > Grimes" > writes: > > > On April 7, 2019 7:11:52 AM PDT, Shawn Webb <shawn.webb@hardenedbsd.o= rg> wr > > ote: > > > >On Sat, Apr 06, 2019 at 09:34:26AM +0000, Mariusz Zaborski wrote: > > > >> Author: oshogbo > > > >> Date: Sat Apr 6 09:34:26 2019 > > > >> New Revision: 345982 > > > >> URL: https://svnweb.freebsd.org/changeset/base/345982 > > > >>=20 > > > >> Log: > > > >> Introduce funlinkat syscall that always us to check if we are > > > >removing > > > >> the file associated with the given file descriptor. > > > >> =20 > > > >> Reviewed by: kib, asomers > > > >> Reviewed by: cem, jilles, brooks (they reviewed previous version) > > > >> Discussed with: pjd, and many others > > > >> Differential Revision: https://reviews.freebsd.org/D14567 > > > > > > > >Hey Mariusz, > > > > > > > >Is __FreeBSD_version supposed to be bumped after adding new syscalls? > > > >I can't remember off-hand. > > > > > > > >Thanks, > > >=20 > > > I don't think so. Why force the rebuild of all ports through poudrier= e over > > something that would never affect any of them? > > > > So that you can if version >=3D foo to know it is safe to use the new s= yscal? > > Or if version < foo you must use the old way. >=20 > Granted. However we do need something to avoid gratuitous rebuilds of=20 > ports. >=20 > Personally, my poudriere script adjusts the pkg version=20 > ($JAILPATH/data/packages/${JAIL}-${PORTS}/.building/.jailversion) with=20 > that of the jail version (reported by poudriere jail -i -j $JAIL),=20 > rebuilding all ports when I (the human) determines when the machine=20 > should rebuild all ports with -c. >=20 > In that regard FreeBSD version bumps occasionally seem a little=20 > gratuitous. Using the same indicator to tell whether software should be= =20 > able to use a new feature and when ports build infrastructure should=20 > summarily delete all packages forcing a rebuild of absolutely=20 > everything is probably not the best. >=20 > Just throwing out an idea, what if poudriere considers the first N=20 > bytes of __FreeBSD_version significant? Having said that, looking at=20 > __FreeBSD_version, I don't think we have enough digits to do what I was= =20 > planning on suggesting. But, you get the idea of what I'm driving at.=20 > Maybe a new macro such as __FreeBSD_ports that is incremented every=20 > time a change that affects ports? >=20 > Anyhow, I'm not too terribly concerned as what I have (selfishly=20 > speaking) works. But we may as a group might want to consider this at=20 > some point to build some efficiency into the ports part of the equation. >=20 >=20 > --=20 > Cheers, > Cy Schubert <Cy.Schubert@cschubert.com> > FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org >=20 > The need of the many outweighs the greed of the few. > =20 >=20 --ibTvN161/egqYuK8 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEkD1x0xkJXVVY1Gwf38KEGuLGxWQFAlyqIUoACgkQ38KEGuLG xWRSaQ/+K9Xb1fcdDZ5U+5bVEfQKBJJGnMjla+sGEDqTzTN24NWqs9KETAtyqXjY c7PLJxlWCaEtWZPsFMNikBm0k6OA4WaqgNS4EIdwgS2oGW9uKRv3dv1iuFzD6Ovl qt7DQd+Bb9AX/3tFamQHc6ZgDyFwWq2GzbH0g1F9XXLEdXAbK5W/itU54RxBP795 iIOwg66KcppvHZ/UytgV90+nuK8VbQqL4pFYqPCNk7vADwqWQF02GRrqWaKimmlt CA3D2q+gDWA81mb+hxc6CrhZTJxws8i/rA7qEodUjmBqlsZulLyw19/B+hVSjy/2 tx1rvvk8eIjuM/lDOQftvmjN3jLreld4Q3zZewOCYPR4CwqzaiwIwF+t5M/TLy99 jRoZXZmzeNCpvo6bDcHs5RUU1xL8Ma6diOGbMnqHLl3hptOcODEFyaxrd/frrIZ4 WmmG4RwjkVkcmsvOlB4k6PAEH3p1OeSPnMclQb8OYdZy965CaGB2M/wbUTeOtxIZ IV1zWgd2/rwSPPJRGryXAikaNNEtkHVjNxC0Y56MPr4vbwxuO1pUbkWVQAdKzvPJ L8ITyDZUIieVrsbQSwMlNq814MNoO0eXO9oaLpbZP1ArXE+eDfSobablyDVPcNxz VCFCwalaiel9p71pTPb5kxSIDdwbfYadzIbcTtxufSSlUgsatkA= =OXw0 -----END PGP SIGNATURE----- --ibTvN161/egqYuK8--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20190407161158.GA40247>