On 08.12.2013 16:34, Stefan Fuhrmann wrote: > On Sun, Dec 8, 2013 at 1:31 PM, Bert Huijben <b...@qqmail.nl > <mailto:b...@qqmail.nl>> wrote: > > We revved almost the entire wc api for Subversion 1.7, with almost > no breakage for the older wc apis. (8 errata if I remember > correctly). I don't see why we can't do this for the fs or ra > layer APIs before 2.0. The old public APIs that miss arguments > won't be as fast as the newer APIs that have them but... I don't > see this as an excuse to use a bad design for the fs layer until 2.0. > > > Well, the alternative would be to eliminate svn_fs_id_t entirely from > the API, > which implies revving svn_fs_dirent_t and svn_fs_path_change2_t plus > related functions. Simply introducing svn_fs_id2_t in parallel to > svn_fs_id_t > would needlessly duplicate a lot of code exactly because the underlying > representations are incompatible. > > I would actually like to remve svn_fs_id_t from the API since it violates > our general path@rev / path@root pattern (deprecating the old API for > backward compat). Any objections?
If we go down that route, we may as well begin designing the FSv2 API instead. This piecemeal twiddling for the sake of a back-end that we won't release for several years yet doesn't make much sense to me. -- Brane -- Branko Čibej | Director of Subversion WANdisco // Non-Stop Data e. br...@wandisco.com
