[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] add version queries and version checks (issue 1861)

From: Edmund Horner <chrysophylax_at_chrysophylax.cjb.net>
Date: 2004-06-14 07:45:41 CEST

Branko Čibej wrote:
> Edmund Horner wrote:
>
> [...]
>
> If those headers don't declare public APIs, then we should make those
> functions "private exported"; that is, instead of "svn _fs_fs_version",
> you'd have "svn _fs_fs__version", and similariy for RA implementations,
> unless they have a public API (AFAIK only ra_svn has one).
>
> Then, of course, the RA and FS vtables need to be extended with a
> version-check entry. I think that's another issue, though.
>
> (Maybe the RA and FS loader should check the version number instead of
> each having its own ABI version?)

OK, I'm happy to try that out tonight.

> [...]
>
> You also need to add the missing checks to the svn client.

Oh. Missed them. OK.

>>* subversion/libsvn_delta/version.c
>>* subversion/libsvn_diff/version.c
>>* subversion/libsvn_fs/version.c
>>* subversion/libsvn_fs_base/version.c
>>* subversion/libsvn_fs_fs/version.c
>>* subversion/libsvn_ra/version.c
>>* subversion/libsvn_ra_dav/version.c
>>* subversion/libsvn_ra_local/version.c
>>* subversion/libsvn_ra_svn/version.c
>>* subversion/libsvn_repos/version.c
>>* subversion/libsvn_wc/version.c:
>> New files.
>
>
> If the library already has an appropriate file (e.g., util.c) then the
> version check can be defined there. There's no need for new files all
> over the place.

Righyho.

>>* subversion/include/svn_delta.h (svn_delta_version)
>>* subversion/include/svn_diff.h (svn_diff_version)
>>* subversion/include/svn_fs.h (svn_fs_version)
>>* subversion/libsvn_fs_base/fs.h (svn_fs_base_version)
>
>
> Should be svn_fs_base__version
>
>
>>* subversion/libsvn_fs_fs/fs.h (svn_fs_fs_version)
>
>
> Should be svn_fs_fs__version
>
>
>>* subversion/include/svn_ra.h (svn_ra_version)
>>* subversion/libsvn_ra_dav/ra_dav.h (svn_ra_dav_version)
>
>
> Should be svn_ra_dav__version
>
>
>>* subversion/libsvn_ra_local/ra_local.h (svn_ra_local_version)
>
>
> Should be svn_ra_local__version

Yup.

> [...]
>
> (by the way, the canonical GNU-ish way to list several files in a single
> log entry is:
>
> * file1 (symbol2),
> file2 (symbol2): log
>
> That way, the asterisk in column 1 signals the start of a log message).

Thanks.

> [...]
>
> We don't add keyword expansion to source files.

Sorry about that, it came from my local config.

> The rest looks fine. Thanks for working on this!

OK.

In summary: thanks for the hints, I'll go ahead and redo the patch then.
  You should hear from me again tomorrow or so.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jun 14 07:47:21 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.