Daniel Shahaf wrote:
> Gerald Macinenti wrote on Fri, 30 May 2008 at 10:33 +0200:
>> this new version of the patch addresses the following issues some of you
>> pointed me at:
>> * superfluous use of "count" and "iteration_pool"
>> * try find_option before find_section so that exactly matching sections
>> have precedence over wildcard sections
>> * added documentation and log message
>> * GNU coding style
>>
>>
>
> Gerald, did you write and test the patch against trunk or against 1.4.6?
> This time it applies cleanly, but in general patches should be written
> (and tested) against trunk. Did you test the new functionality with trunk?
against trunk, and yes I tested the new functionality in a real
installation from trunk sources (apache + mod_authz_svn only, not svnserve).
But the "make check" that was all good when I was working on the CentOS5
1.4.2 version skips all the authz tests in trunk (with or without the
patch):
SKIP: authz_tests.py 1: authz issue #2486 - open root
SKIP: authz_tests.py 2: authz issue #2486 - open directory
SKIP: authz_tests.py 3: broken authz files cause errors
SKIP: authz_tests.py 4: test authz for read operations
SKIP: authz_tests.py 5: test authz for write operations
SKIP: authz_tests.py 6: test authz for checkout
SKIP: authz_tests.py 7: test authz for log and tracing path changes
SKIP: authz_tests.py 8: test authz for checkout and update
SKIP: authz_tests.py 9: test authz for export with unreadable subfolder
SKIP: authz_tests.py 10: test authz for aliases
SKIP: authz_tests.py 11: test the authz validation rules
SKIP: authz_tests.py 12: test authz for locking
SKIP: authz_tests.py 13: authz issue #2712
SKIP: authz_tests.py 14: switched to directory, no read access on parents
why? I give the tests log in attachment.
--
Gerald MACINENTI, System/Network Administrator
LET IT WAVE, http://www.letitwave.fr
8-16, rue Paul-Vaillant-Couturier, 92240 Malakoff, FRANCE
Ph. +33-1-4092-5454 - Fax. +33-1-4092-5442
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-30 14:58:23 CEST