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

Re: svn commit: r1809570 - /subversion/branches/1.9.x/STATUS

From: Branko Čibej <brane_at_apache.org>
Date: Thu, 28 Sep 2017 16:15:17 +0200

On 28.09.2017 14:41, Daniel Shahaf wrote:
> Branko Čibej wrote on Wed, 27 Sep 2017 01:28 +0200:
>> On 24.09.2017 23:23, danielsh_at_apache.org wrote:
>>> +++ subversion/branches/1.9.x/STATUS Sun Sep 24 21:23:40 2017
>>> @@ -76,6 +76,8 @@ Candidate changes:
>>> ^/subversion/branches/1.9.x-r1808955
>>> Votes:
>>> +1: brane
>>> + +0: danielsh (this works on OS X and Linux but nevertheless I wonder
>>> + about its portability)
>>
>> It's as portable as 'httpd -V'. If someone builds with a patched httpd
>> that does not print a the sever version with this option, then the
>> script will fail.
>>
> Why would it fail? It neither does 'set -e' nor validates the value of $HTTPD_VERSION:
>
>>> +HTTPD_VERSION=$("$HTTPD" -V -f $HTTPD_CFG | grep '^Server version:' | sed 's|^.*/\([0-9]*\.[0-9]*\.[0-9]*\).*$|\1|')

It will fail to DTRT.

> Besides, * in sed is greedy, so an output like
>
> Server version: Apache/2.4.50 (Counterexample Linux/1.2.3)
>
> would DTWT.

Yes. It would. It's not perfect. Neither is the current situation. But
one can always avoid using 'davautocheck' on strange platforms with
strange httpd builds; 'davcheck' works just fine, it only takes a bit
more effort; probably less effort than patching and building httpd in
the first place.

>> On the other hand, if we don't backport this change, the OSX tests on
>> the 1.9.x branch will keep failing indefinitely because (a) 'configure'
>> reports the wrong version of httpd, and (b) the 1.9.x tests do not have
>> the concept of an httpd version whitelist.
>>
>> IMO it's more important to have reliable test results on supported
>> branches than to support unlikely patches that people might come up with.
> The patchset may be an improvement for OS X, but I am concerned that it
> might introduce a regression for other platforms; that's why I didn't vote +1
> on it.

So first of all, the problem is not specific to OSX. It will happen in
any environment where the available httpd headers are for a different
version than the server binaries. In the case of the particular version
of OSX running on the slave, that's 2.2.26 (headers) vs. 2.2.29
(binary), the former being in our blacklist of broken httpd versions WRT
URL quoting in mod_dav.

Here are some other possible solutions:

  * Backport the httpd version whitelist logic to 1.9.x (more complex?)
  * Use a homegrown version of httpd on the bot (not gonna happen)
  * Fix system headers on the bot (nice try)
  * Leave the OSX DAV tests broken
  * ? I'm out of ideas

The first option is the only viable one apart from what I've already
done, IMO.

-- Brane
Received on 2017-09-28 16:15:26 CEST

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