Re: svn commit: r1801940 - in /subversion/trunk: ./ notes/ subversion/include/ subversion/include/private/ subversion/libsvn_delta/ subversion/libsvn_fs_fs/ subversion/libsvn_subr/ subversion/tests/libsvn_delta/ subversion/tests/libsvn_subr/

kotkov_at_apache.org writes:

> Author: kotkov
> Date: Fri Jul 14 11:13:47 2017
> New Revision: 1801940
>
> URL: http://svn.apache.org/viewvc?rev=1801940&view=rev
> Log:
> fsfs: Add initial support for LZ4 compression.

> * build.conf
> (libsvn_subr): Build LZ4 library sources.

Why do we have an embedded copy of the source code? That's not our
usual policy with 3rd party libraries and the LZ4 library is available
on Linux systems:

  $ pkg-config lz4 --libs
  -llz4

If the reason for embedding is simply to avoid an external dependency
then I think that is wrong -- we should not accumuate 3rd party code in
our code base. I suppose there could be an argument to embed the LZ4
code if the compressed form is not stable from one release to another.
Is that the case? If so then how stable is the current format?

-- 
Philip