Re: svn commit: r316980 - head/contrib/zstd/programs

2017-12-06 Thread Baptiste Daroussin
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. > > > > Thanks, > > Conrad > > > > On Wed, Nov 15, 2017 at 7:38 PM, Conrad Meyer

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-12-06 Thread Baptiste Daroussin
On Tue, Dec 05, 2017 at 05:17:22PM -0800, Conrad Meyer wrote: > Ping. Please revert this change. If you want zstd to behave this > way, pursue it upstream first. This is planned, for now I first need to modify newsyslog to accept more arguments for the compression program, then I'll switch back

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-12-05 Thread Allan Jude
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. > > Thanks, > Conrad > > On Wed, Nov 15, 2017 at 7:38 PM, Conrad Meyer wrote: >> Please revert this change. >> >> First, it introduces

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-12-05 Thread Conrad Meyer
Ping. Please revert this change. If you want zstd to behave this way, pursue it upstream first. Thanks, Conrad 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

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-11-17 Thread Conrad Meyer
On Fri, Nov 17, 2017 at 1:34 AM, 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

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-11-17 Thread Baptiste Daroussin
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. I have modified zstdcat to behave like zcat/bzcat. The commit you stated

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-11-16 Thread Allan Jude
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. > > This is not a

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-11-16 Thread Baptiste Daroussin
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. This is not a POLA-violating behavior, this is a bug! that I introduced. >

Re: svn commit: r316980 - head/contrib/zstd/programs

2017-11-15 Thread Conrad Meyer
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

svn commit: r316980 - head/contrib/zstd/programs

2017-04-15 Thread Baptiste Daroussin
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