From owner-svn-src-head@freebsd.org Wed Dec 6 08:38:48 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 AFEB8E9972E; Wed, 6 Dec 2017 08:38:48 +0000 (UTC) (envelope-from bapt@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 6DBFD689FF; Wed, 6 Dec 2017 08:38:48 +0000 (UTC) (envelope-from bapt@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1235) id B1F106A89; Wed, 6 Dec 2017 08:38:47 +0000 (UTC) Date: Wed, 6 Dec 2017 09:38:47 +0100 From: Baptiste Daroussin To: Allan Jude Cc: cem@freebsd.org, src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r316980 - head/contrib/zstd/programs Message-ID: <20171206083847.f4t5wdtf6toaua63@ivaldir.net> References: <201704152015.v3FKFiwZ006836@repo.freebsd.org> <89b00017-aa92-66ca-5a28-47f0eb8f399b@freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="26zso3mqgppcsewa" Content-Disposition: inline In-Reply-To: <89b00017-aa92-66ca-5a28-47f0eb8f399b@freebsd.org> User-Agent: NeoMutt/20171027 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: Wed, 06 Dec 2017 08:38:48 -0000 --26zso3mqgppcsewa Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Dec 05, 2017 at 08:42:11PM -0500, Allan Jude wrote: > On 2017-12-05 20:17, Conrad Meyer wrote: > > Ping. Please revert this change. If you want zstd to behave this > > way, pursue it upstream first. > >=20 > > Thanks, > > Conrad > >=20 > > On Wed, Nov 15, 2017 at 7:38 PM, 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. > >> > >> 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. > >> > >> Thanks, > >> Conrad > >> > >> On Sat, Apr 15, 2017 at 1:15 PM, Baptiste Daroussin = wrote: > >>> Author: bapt > >>> Date: Sat Apr 15 20:15:44 2017 > >>> New Revision: 316980 > >>> URL: https://svnweb.freebsd.org/changeset/base/316980 > >>> > >>> Log: > >>> Change some default to make zstd a dropin replacement for gzip,bzip= etc > >>> in most cases > >>> > >>> Changes ares: > >>> - quiet by default > >>> - remove the source files one compression completion by default > >>> > >>> Modified: > >>> head/contrib/zstd/programs/fileio.c > >>> head/contrib/zstd/programs/zstdcli.c > >>> > >>> Modified: head/contrib/zstd/programs/fileio.c > >>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > >>> --- head/contrib/zstd/programs/fileio.c Sat Apr 15 20:06:24 2017 = (r316979) > >>> +++ head/contrib/zstd/programs/fileio.c Sat Apr 15 20:15:44 2017 = (r316980) > >>> @@ -138,7 +138,7 @@ static U32 g_dictIDFlag =3D 1; > >>> void FIO_setDictIDFlag(unsigned dictIDFlag) { g_dictIDFlag =3D dictI= DFlag; } > >>> static U32 g_checksumFlag =3D 1; > >>> void FIO_setChecksumFlag(unsigned checksumFlag) { g_checksumFlag =3D= checksumFlag; } > >>> -static U32 g_removeSrcFile =3D 0; > >>> +static U32 g_removeSrcFile =3D 1; > >>> void FIO_setRemoveSrcFile(unsigned flag) { g_removeSrcFile =3D (flag= >0); } > >>> static U32 g_memLimit =3D 0; > >>> void FIO_setMemLimit(unsigned memLimit) { g_memLimit =3D memLimit; } > >>> > >>> Modified: head/contrib/zstd/programs/zstdcli.c > >>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > >>> --- head/contrib/zstd/programs/zstdcli.c Sat Apr 15 20:06:24 2= 017 (r316979) > >>> +++ head/contrib/zstd/programs/zstdcli.c Sat Apr 15 20:15:44 2= 017 (r316980) > >>> @@ -61,7 +61,7 @@ > >>> #define MB *(1 <<20) > >>> #define GB *(1U<<30) > >>> > >>> -#define DEFAULT_DISPLAY_LEVEL 2 > >>> +#define DEFAULT_DISPLAY_LEVEL 1 > >>> > >>> static const char* g_defaultDictName =3D "dictionary"; > >>> static const unsigned g_defaultMaxDictSize =3D 110 KB; > >>> > >=20 >=20 > Upstream recommends we create an additional hardlink, called 'zz' (like > xz but zstd), to do such compatibility, and keep 'zstd' the same as zstd > on every other distro. >=20 Actually I would prefer not to do it and simply adapt newsyslog to be more flexible. it is the only consumer of the zstd command. Bapt --26zso3mqgppcsewa Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEgOTj3suS2urGXVU3Y4mL3PG3PloFAlonrJcACgkQY4mL3PG3 PlralQ/9H7S2z1dWwe7Z7EmcL4IjbpmOIYs2gx70GsdVzQWwtayDQomFNUNRCQM0 r+VVuyDAiV+Co38C/+ItkzxG5H++MCDzBQDDGdk4k/M5si9PyZLP0mFs6QVnlCwu UGtpV1UpHN+djpZc7HfIkiReJ6Y/2paXg2oVeD+RmHu7bcZgqI2vz7WxVP+J03cS 84xfNkhvfJb02NYqARE8fBHBITXkk5Y+gQWNtQkndQHtNnpWVJL0ZTdW6kyG2enw LuLhdiQLHPCN+q1CxtidsgEQgphth8khUPyfQLyxctWefuhL8JclaR4GD1iOdbIl BpZ09o9hH6gsxTWQ1urqChGff1XdcHxa2I8cI+32XILW/XiW4u9A2GR467DEjyBF P2PdSOLOG29Ylq45/E1z/moKFs4XX/2QbmpvtK12+wruvOsgf6i+5aQhtMzlBaZS ofJySbcM2O8bQdJ1/H0KikS0fBmKy6I6905LCvGYZKyy9mrUbfCMQ+zKtAHh8HPk pkYi4chLEbjcyjzkmH35JesY0cgFfxsdjyxvHZHo6ioV81nTO2VkMUe1ItE12qNx f8+2m3bIhGntZd2ejTWa/hIVlzvVzgPQvn9tWHSxu3+FFXpGDxVvPmGoz3PaYamk I1+M4ZBO2aqlMRbJF369cB5FlIcUJ0FbnGimIEH+VkNs6M1LzBI= =H5xT -----END PGP SIGNATURE----- --26zso3mqgppcsewa--