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


From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Fri, 08 Jul 2011 12:13:18 -0400

On 07/08/2011 06:01 AM, Philip Martin wrote:
> Stefan Sperling <stsp_at_elego.de> writes:
>> On Fri, Jul 08, 2011 at 10:44:27AM +0100, Julian Foad wrote:
>>> There's a comment in libsvn_ra_[neon|serf]/options.c saying "we should
>>> probably remove it before 1.7 goes final". Should we remove or keep it?
>>> #ifdef SVN_DEBUG
>>> /* ### This section is throw in here for development use. It
>>> ### allows devs the chance to force the client to speak v1,
>>> ### even if the server is capable of speaking v2. We should
>>> ### probably remove it before 1.7 goes final. */
>>> char *ignore_v2_env_var = getenv(SVN_IGNORE_V2_ENV_VAR);
>>> if (! (ignore_v2_env_var
>>> && apr_strnatcasecmp(ignore_v2_env_var, "yes") == 0))
>>> ras->me_resource = apr_pstrdup(ras->pool, val);
>>> #else
>>> ras->me_resource = apr_pstrdup(ras->pool, val);
>>> #endif
>> It is easier to set an environment variable than installing an older
>> client that only speaks the old protocol.
>> So it might be useful to keep this to facilitate problem diagnosis
>> once httpv2 clients are in the field. In which case the comment
>> should be adjusted, of course, and we should move this out of SVN_DEBUG.
> If we are going to keep it then we should make it a config option like
> http-compression (and avoid a name with 'httpv2' to make it clear that
> this is a Subversion protocol).

Why? It's intended for Subversion devs only, as evidenced by the fact that
its only use is wrapped in an #ifdef SVN_DEBUG macro. What's the driving
motivation for building this out into a user-accessible configuration
option? What, exactly, is the driving motivation for doing anything at all
in this space right now (besides the misguided comment that implies that
something life-changing will happen if we choose to remove the env-var after
1.7 ships)?

C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2011-07-08 18:13:52 CEST

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