From owner-svn-src-head@freebsd.org Thu Nov 16 16:04:29 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id B86E6DE1EFD; Thu, 16 Nov 2017 16:04:29 +0000 (UTC) (envelope-from allanjude@freebsd.org) Received: from mx1.scaleengine.net (mx1.scaleengine.net [209.51.186.6]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 57A927FF2E; Thu, 16 Nov 2017 16:04:29 +0000 (UTC) (envelope-from allanjude@freebsd.org) Received: from [10.1.1.2] (Seawolf.HML3.ScaleEngine.net [209.51.186.28]) (Authenticated sender: allanjude.freebsd@scaleengine.com) by mx1.scaleengine.net (Postfix) with ESMTPSA id BE7F8EE1D; Thu, 16 Nov 2017 15:56:31 +0000 (UTC) Subject: Re: svn commit: r316980 - head/contrib/zstd/programs To: Baptiste Daroussin , Conrad Meyer Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201704152015.v3FKFiwZ006836@repo.freebsd.org> <20171116090447.sjwtzrwgyuarh7kv@ivaldir.net> From: Allan Jude Message-ID: <4fe304eb-eb98-2fcb-f8aa-4f0b433d2209@freebsd.org> Date: Thu, 16 Nov 2017 10:56:24 -0500 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171116090447.sjwtzrwgyuarh7kv@ivaldir.net> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="l5dcWr240hjmPr8r3X35MvwqPHDFxCehu" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Nov 2017 16:04:29 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --l5dcWr240hjmPr8r3X35MvwqPHDFxCehu Content-Type: multipart/mixed; boundary="ji1S7R5KVWLFqa4QRDfTSowAaNdnpRwO3"; protected-headers="v1" From: Allan Jude To: Baptiste Daroussin , Conrad Meyer Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <4fe304eb-eb98-2fcb-f8aa-4f0b433d2209@freebsd.org> Subject: Re: svn commit: r316980 - head/contrib/zstd/programs References: <201704152015.v3FKFiwZ006836@repo.freebsd.org> <20171116090447.sjwtzrwgyuarh7kv@ivaldir.net> In-Reply-To: <20171116090447.sjwtzrwgyuarh7kv@ivaldir.net> --ji1S7R5KVWLFqa4QRDfTSowAaNdnpRwO3 Content-Type: text/plain; charset=windows-1252 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 2017-11-16 04:04, Baptiste Daroussin wrote: > On Wed, Nov 15, 2017 at 07:38:13PM -0800, Conrad Meyer wrote: >> Please revert this change. >> >> First, it introduces the POLA-violating behavior that zstdcat deletes >> its source files. This is not how zcat/bzcat behaves. >=20 > This is not a POLA-violating behavior, this is a bug! that I introduced= =2E >> >> Second, it introduces a needless behavioral difference between FreeBSD= >> zstd and the rest of the world's zstd. The zstd documentation we ship= >> continues to claim that zstd preserves source files by default, yet >> this change makes that documentation exactly backwards. While we can >> change FreeBSD's documentation to accommodate the change, we can't >> change Google results. >> >=20 > The difference has been made so that zstd follow by default the same be= haviour > has gzip/bzip2/xz so it can be a dropped in replacement. >=20 > The argument about the documentation is however a good one. Let me do f= irst some > tests to ensure restoring the initial behaviour does not break existing= usage in > base >=20 > Best regards, > Bapt >=20 I think in this case, it is safer to surprise the user by NOT deleting a file, than to surprise a user by DELETING a file. If you really want to modify the behaviour (I suggest we don't, and stick closer to upstream zstd), then you'll want to set the 'delete' flag for the specific invocation cases inside zstdcli.c, rather than modifying the default as was done in this commit. I think we can deal with changing the default verbosity level. I think if we want to compromise, we make 2 additional hard links, zzip and zunzip that maintain the gzip like behaviour, since those will not conflict with the documentation that exists in the rest of the world for zstd(1) --=20 Allan Jude --ji1S7R5KVWLFqa4QRDfTSowAaNdnpRwO3-- --l5dcWr240hjmPr8r3X35MvwqPHDFxCehu Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (MingW32) iQIcBAEBAgAGBQJaDbUuAAoJEBmVNT4SmAt+pkQP/0GFptJp8xFsvcxp6I/yRdWd 5fbIp7O/dsldglnlB89FhO/TO0P1QWk8vwsRRkTojXw+cx0FTekouNg6/q9NvLMh GLZB1z8ulKV02NbnxF1SJR8ZHadUnkbymhCEhAW1WiiQeJ2O66DKPC+0mlhYbVWF GcrRXDqCZ19WovipuoJ/GmFEW0+L2r4r44UekVJ5iY65y0n7mwGu7WKrsgqyfQGp Lb1U4MBnMGJh0S0dUNNRjsklPeoGkHjSIAbL5easrPXGkjp7kXVj5BqRv4VDT26f 71k5CQRcJEsQhPFzvqNOfT314zZlutCQJ84O628I9qZefQhpmod6Y8ThnWmaJai1 HmQpfnaqgSpRwwzCr/wuYazW7CmQnrjRgre9wXSNT7TVglDOCG53ZcaxJVIU/7Lm FtfCDJK6MQvGmsBNviKfWm8qp6UFq3vhUQM+sUiUVE7Fg53AmVOuBmCWSF3afM4c 6ezEVoOFP4ZnwBSkDaH956ZUvNgiKnspAvn8vZN6U/w7dh96v8kR8L3CCZykc3an 6yXyJ1v30cV59ytimu9hJrS1SW4cq6uzSrp3tiVC/A61NJB78syEk2hV40ws1ChK Nl3SRWPCJ/WfNumKQpei3AheptsCJFUabjmK/2IqXGFxTp3C5bnRDIqpbVnu5Eb3 9h2xxZFMk4Ipr0sDDjmF =o/mq -----END PGP SIGNATURE----- --l5dcWr240hjmPr8r3X35MvwqPHDFxCehu--