Re: svn commit: r903896 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

danielsh_at_apache.org wrote on Wed, 27 Jan 2010 at 22:58 -0000:
> Author: danielsh
> Date: Wed Jan 27 22:58:11 2010
> New Revision: 903896
>
> URL: http://svn.apache.org/viewvc?rev=903896&view=rev
> Log:
> Fix packing of format-4 FSFS repositories.
>
> * subversion/libsvn_fs_fs/fs_fs.c
> (pack_body):
> Don't try to pack revprops unless the format allows that.

I found this trying to sync my mirrors of s.c.n. But does the FS code
get enough testing? Do people here (except me) run trunk-code
repositories?

Different issue: I haven't run packing-enabled 'make check' on this
change. I'll try and do that over the weekend --- but I won't mind
being beaten[1] to it :-).

Daniel

[1] I plan to build with -DPACK_AFTER_EVERY_COMMIT and
-DSVN_FS_FS_DEFAULT_MAX_FILES_PER_DIR=4. (There is also 'make check
FSFS_SHARDING=3 FSFS_PACKING=yes', which is somewhat different.)
Received on 2010-01-28 00:39:41 CET