Date: Thu, 14 Apr 2022 08:44:52 +0100 From: Mark Murray <markm@FreeBSD.org> To: David Chisnall <theraven@FreeBSD.org> Cc: freebsd-hackers@freebsd.org Subject: Re: [RFC] patch's default backup behavior Message-ID: <973282F3-3AAD-469C-BF44-169C7C9F751A@FreeBSD.org> In-Reply-To: <6c8a812f-438c-dfac-7ba8-b9ae6abf0840@FreeBSD.org> References: <202204111658.23BGwmcC073621@gndrsh.dnsmgr.net> <6c8a812f-438c-dfac-7ba8-b9ae6abf0840@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_6359A126-D024-4165-9FA6-475F44C3999E Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=us-ascii Thank you David. Well said! M > On 12 Apr 2022, at 17:36, David Chisnall <theraven@FreeBSD.org> wrote: >=20 > On 11/04/2022 17:58, Rodney W. Grimes wrote: >> Personally, if YOU like the behavior of gnu patch, by all means, >> please USE gnu patch. Please do NOT make bsd patch behave in >> a different manner simply because you personally like that >> other behavior. >> If you want the stuff to look like Linux/GNU by all means, >> go RUN linux/gnu!!!! >=20 > There are two opinions that as almost invariably wrong, in any = context: >=20 > - Popular thing X does Y, therefore Y is good. > - Popular thing X does Y, therefore Y is bad. >=20 > This thread started with 'popular thing X does Y, we do Z, let's = evaluate which is better'. Reducing that to 'popular thing X does Y, = therefore we should do Z, if you want Y then you should run X' is = unhelpful. It is also how we end up in a situation where everyone runs = X and we sit around wondering where all of our users and contributors = went. >=20 > We should neither adopt or discard a particular behaviour in patch on = the basis that GNU patch does it. We should use the fact that GNU patch = does it to gather data on whether it's a desirable behaviour. We should = adopt their good ideas and avoid their bad ideas. >=20 > That is precisely the process that Kyle is trying to drive and the = FreeBSD system will be better as a result of his work. Personally, I = hate having .rej and .orig files scattered over my filesystem as a = result of patch failing and I end up having to write a `find` command to = delete them all. Does that mean that I want to give up kqueue, = capsicum, out-of-the-box ZFS, a sane /dev/dsp, jails, clang as the = system compiler, a `tar` that knows that `x` means 'extract the thing, = you don't need me to duplicate the information in the file header to = know what it is', and so on and run GNU/Linux? No. >=20 > I take Joerg's point that GNU patch *sometimes* creating them makes = tooling difficult. I would be quite happy with a solution that they are = created unconditionally with a flag to disable creating them (I would = then `alias patch=3D"patch --do-not-leave-stuff-on-my-filesystem"` in my = `.profile` and forget about it for interactive use) or that they are = never created with a flag to enable creating them, which I would never = pass except when working with bits of infrastructure that explicitly = want the .orig files. >=20 > David >=20 -- Mark R V Murray --Apple-Mail=_6359A126-D024-4165-9FA6-475F44C3999E Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.2 Comment: GPGTools - http://gpgtools.org iQEzBAEBCgAdFiEEyzPHvybPbOpU9MCxQlsJDh9CUqAFAmJX0PQACgkQQlsJDh9C UqBorgf+Kd7eR3zNIxyTmtmI1tYjUp6z9uTtVWBLfzjhHIXm+VvEqD8/oevkPHyS s49QX5z68z09i2H04KLb10we4EAKXWOfY5o4e8wkWTxc/FUiASEbqoU+RPMeWTaL tsSaKIDD6zGIoH2/VYTGC+aMK5eHgNNf1Uw2JJPyYIr6aKIOiIJkLEWHTO6Gy4mr HbHmdFxHnMLo3++co/PpagBpohkKbznL9XOz/70NaB9Qf7swkw9Zhg70pvgGibfI JmowuczJ7BcDUXTRxgTgC5YGEKNPK6nFrjcl5I9kCmUy+8HJUVckRQxlk2teaOpO xteAQ0LytxyH5XbkXLxZNu+auUysGg== =jzlY -----END PGP SIGNATURE----- --Apple-Mail=_6359A126-D024-4165-9FA6-475F44C3999E--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?973282F3-3AAD-469C-BF44-169C7C9F751A>