Checkout Tools
  • last updated 4 hours ago
Constraints: committers
Constraints: files
Constraints: dates

Changeset 1461535 is being indexed.

Add an XFAIL test for issue 4340, "repos layer should reject filenames with

trailing \n".

* subversion/tests/libsvn_repos/repos-test.c

(filename_trailing_newline, test_funcs): New test.

The new (1.8) zero_copy_limit parameter in svn_repos_begin_report3

can only be used under certain circumstances. Document those.

Since we use that interface at our RA layers only, the preconditions

are fulfilled. However, our tests do create some things with this

API and this patch disables the zero copy optimization for those.

* subversion/include/svn_repos.h

(svn_repos_begin_report3): mention limitation on zero_copy_limit

* subversion/tests/libsvn_repos/repos-test.c

(reporter_depth_exclude): make our tests comply to that limitation

  1. … 1 more file in changeset.
Allow svn_ra_get_file_revs2() and svn_repos_get_file_revs2() to report file

revisions backwards. This to allow optimizing 'svn blame' in a future

Subversion release without having to wait for yet another server upgrade

to support this new feature.

* subversion/include/svn_ra.h

(svn_ra_get_file_revs2): Update documentation.

* subversion/include/svn_repos.h

(svn_repos_get_file_revs2): Update documentation.

* subversion/libsvn_repos/rev_hunt.c

(svn_repos_get_file_revs2): Update comment. Detect reversed arguments,

which would originally just return an error or no revisions..

and in that case reverse the revision array before walking it.

* subversion/tests/libsvn_repos/repos-test.c

(test_get_file_revs): Extend test to also walk the revisions backwards.

  1. … 3 more files in changeset.
Fix test failures after r1440023.

* subversion/tests/libsvn_repos/repos-test.c

(in_repo_authz, in_repo_groups_authz): Expect SVN_ERR_ILLEGAL_TARGET

for non-existant paths in the repo.

Introduce AuthzSVNGroupsFile option to allow Subversion configurations with

groups stored in a separate file.

* subversion/include/svn_config.h


* subversion/include/svn_repos.h


(svn_repos_authz_read2): New optional 'groups_path' parameter.

(svn_repos_authz_parse): New optional 'groups_stream' parameter.

* subversion/libsvn_repos/authz.c

(authz_copy_group): Introduce a callback for groups copying.

(authz_copy_groups): Introduce a helper routine to copy groups from a

specified config to the authz structure. Report error if the

destination authz already contains group definitions.

(svn_repos__authz_read): Support the 'groups_path' parameter. If it is set,

parse the corresponding groups config and copy the groups to the

resulting authz structure using authz_copy_groups.

(svn_repos_authz_read2): New optional 'groups_path' parameter.

(svn_repos_authz_read): Support the 'groups_stream' parameter. If it is

set, parse the corresponding groups config stream and copy the groups

to the resulting authz structure.

* subversion/libsvn_repos/deprecated.c

(svn_repos_authz_read): Pass NULL as the 'groups_path' parameter when

calling svn_repos__authz_read.

* subversion/libsvn_repos/repos.h

(svn_repos__authz_read): New optional 'groups_path' parameter.

* subversion/libsvn_repos/repos.с

(create_conf): Explain the purpose of the new groups-db option.

* subversion/mod_authz_svn/mod_authz_svn.c

(authz_svn_config_rec): Add the 'groups_file' config member.

(AuthzSVNGroupsFile_cmd): Introduce a function to canonicalize the groups

file config value.

(authz_svn_cmds): Add the AuthzSVNGroupsFile option.

(get_access_conf): Log the path to the groups file if it is set. Pass the

groups file path to svn_repos_authz_read2.

* subversion/svnserve/serve.c

(canonicalize_access_file): New function. Factored out from

load_authz_config to be reused for the groups file.

(load_authz_config): Retrieve the SVN_CONFIG_OPTION_GROUPS_DB from the

svnserve config. Canonicalize this value if it is present and pass

it to svn_repos_authz_read2 when loading the authz configuration.

* tools/server-side/svnauthz.c

(svnauthz_opt_state): Add the 'groups_file' member.

(svnauthz__cmdline_options_t): Add the svnauthz__groups_file enum member.

(options_table): Add the --groups-file option and short description for it.

(cmd_table): Update the documentation to reflect the added

svnauthz__groups_file option.

(read_file_contents): New function. Factored out from get_authz_from_txn to

be reused for the groups file contents.

(get_authz_from_txn): Use read_file_contents for both authz and groups

files. Pass the resulting contents to svn_repos_authz_parse.

(get_authz): Pass the groups file from options to get_authz_from_txn

or svn_repos_authz_read2 depending on whether transaction option is set.

(canonicalize_access_file): New function. Factored out from

sub_main to be reused for the groups file.

(sub_main): Grab the groups file option from the command line, canonicalize

it if it present and pass it further as a part of the opt_state.

* subversion/tests/libsvn_repos/repos-test.c

(authz-get-handle, in_repo_authz): Pass NULL as the 'groups_path' parameter

when calling svn_repos_authz_read2. Pass NULL as the 'groups_stream'

parameter when calling svn_repos_authz_parse.

(authz_groups_get_handle): Introduce the helper routine for tests similiar

to authz_get_handle but supporting a separate groups file.

(groups_authz, in_repo_groups_authz): Add.

* subversion/tests/cmdline/

(authz_svnserve_groups): Add the access test for svnserve configured with

a separate groups file.

(test_list): Add a reference to the new test.

* subversion/tests/cmdline/svntest/

(write_restrictive_svnserve_conf_with_groups): New method. Creates a

default restrictive svnserve configuration with a separate groups file.

(write_groups_file): Introduce a helper method to write the groups file

in tests.

* subversion/tests/cmdline/svntest/

(_set_name): Store the default path to the groups file in the

'Sandbox.groups_file' variable.

* subversion/mod_authz_svn/INSTALL

(II.1): Describe usage of AuthzSVNGroupsFile directive.

Patch by: Evgeny Kotkov <evgeny.kotkov{_AT_}>

  1. … 13 more files in changeset.
Fix compilation warning.

* subversion/tests/libsvn_repos/repos-test.c

(prop_validation): Use '\xff' instead of '(char) 0xff'.

Follow-up to r1437354: Add test for svn_repos_delete().

* subversion/tests/libsvn_repos/repos-test.c

(test_delete_repos): New test for svn_repos_delete().

(test_funcs): Add new test to list.

Suggested by: philip

Add a svn_repos_authz_parse() function which parses an authz config from

a stream.

This also switches most testing of the authz code to using streams, which

should result in a tiny bit faster testing since we're not needing to

do disk I/O.

* subversion/libsvn_repos/authz.c

(svn_repos__authz_read): Factor out walking the config for authz errors

into ...

(authz_validate): New function.

(svn_repos_authz_parse): New function.

* subversion/include/svn_repos.h

(svn_repos_authz_parse): New function.

* subversion/tests/libsvn_repos/repos-test.c

(authz_get_handle): Adjust to allow authz conf to be optionally written

to a in memory stream rather than always going to a temp file.

(authz): Change most of the tests to use in memory streams and duplicate

one test so that the code that is unique to the file path is excercised.

(commit_editor_authz,issue_4060): Use in memory streams.

  1. … 2 more files in changeset.
* subversion/tests/libsvn_repos/repos-test.c (authz): Remove unused variable.

Merge the in-repo-authz branch back to trunk.


  1. … 23 more files in changeset.
Refactor some test code to make it easier to extend.

* subversion/tests/libsvn_repos/repos-test.c

(nls_receiver_baton, format_segment, nls_receiver, check_location_segments):

Add doc strings and constify the expected result pointers.

(location_segment_test_t): New struct.

(node_location_segments): Move the test data into an array and iterate

over it.

Make the unit test for svn_repos_node_location_segments() stronger.

* subversion/tests/libsvn_repos/repos-test.c

(strcmp_null): New function.

(nls_receiver): Check 'range_end' and 'path' as well as 'range_start'.

(node_location_segments): Fix two typos in the expected paths.

Merge the remainder of the 10Gb branch into /trunk.

This patch revs APIs and touches the UI. For a detailed

description of each merged revision see the respective

log entry.

* Implement zero-copy code path

r1388102: extend the FS vtable. Add a function that allows for

processing file content in-situ.

r1388195: Rev reporter API and attempt zero-copy reporting for

files contents below a given threshold

r1388202: Add --zero-copy-limit parameter to svnserve and pass it

down to the reporter

r1388205: update remaining callers to use svn_repos_begin_report3.

r1388211: rename *_try_process_file_content *_try_process_file_contents.

r1388362: Bugfix: try_process_file_contents code plus a handful of bugs

masked by the first.

r1388640: Bugfix: off-by-one in contents size.

* Implement delayed checking for client-side abortion

r1388375: Extend aready rev'ed APIs and add --error-check-internal

option to svnserve.

* Tune ra_svn TX buffer handling

r1388643: tune ra_svn TX buffer usage by reducing the number of copy

operations etc.

r1388807: page-aligned I/O buffers.

* Finalize APIs and UI

r1390409: replace --zero-copy-limit and --error-check-internal with a

single option (--client-speed) at UI level but keep internal

APIs as they are

r1390414: replace help string for the --client-speed parameter.

r1390419: UI typo

r1390955: Schedule API changes for 1.8 instead of 1.9.

  1. … 25 more files in changeset.
Silence GCC warnings when assigning to chars (-Wconversion).

In all these cases, the value may never exceed the [-128,127]

range for char.

* subversion/libsvn_delta/svndiff.c

(window_handler): explicitly cast to char

* subversion/libsvn_fs_fs/key-gen.c

(svn_fs_fs__next_key): prevent conversion to int

* subversion/libsvn_fs_fs/temp_serializer.c

(encode_number): explicitly cast to char

* subversion/libsvn_ra_svn/marshal.c

(svn_ra_svn_write_string, svn_ra_svn_write_cstring): ditto

* subversion/libsvn_subr/base64.c

(decode_group): ditto

* subversion/libsvn_subr/checksum.c

(svn_checksum_parse_hex): same here

* subversion/libsvn_subr/dirent_uri.c

(canonicalize_to_lower, canonicalize_to_upper): here

* subversion/mod_dav_svn/repos.c

(get_entry): and here

* subversion/tests/libsvn_repos/repos-test.c

(prop_validation): char

* subversion/tests/libsvn_subr/stream-test.c

(generate_test_bytes): explicitly cast to char

  1. … 9 more files in changeset.
Merge all changes (-r1298521-1326293) from branches/revprop-cache to trunk

and resolve minor conflicts.

  1. … 25 more files in changeset.
Add a regression test for issue 4060, spurious authz denied when

svnserve root is a repository.

* subversion/test/libsvn_repos/repos-test.c

(issue_4060): New test.

(test_funcs): Add new test.

One more attempt to fix the Windows build in followup to r1197217.

* subversion/tests/libsvn_repos/repos-test.c

(action_t): Change enum values, to avoid a possible previously defined symbol


(test_path_authz, commit_editor_authz): Update references.

Make the windows compiler happy.

* subversion/tests/libsvn_repos/repos-test.c

(action_t): New, broken out of...

(authz_path_action_t): ...this.

Rewrite repos test 8 to adhere to the proper editor semantics, namely not

reusing a delta editor once it has emitted an error, and accounting for the

possibilities of delayed errors.

* subversion/tests/libsvn_repos/repos-test.c

(authz_path_action_t): New.

(get_dir_baton, get_file_baton): New.

(test_path_authz): New.

(commit_editor_authz): Largely gut, and replace with a table of actions

and outcomes, and then iterate over that table.

Path-based authz: error out on non-canonical fspaths in the input. (We

already canonicalize fspaths passed to the API for access testing.)

This is an incompatible change: some previously-accepted authz files

will now cause visible errors. However, before this change the semantics

of such authz files might have been different from what a casual gloss

over them would have suggested.

Found by: Malte Schirmacher

(thana on IRC)

* subversion/libsvn_repos/authz.c

(authz_validate_section): Validate the fspath part of the section name.

* subversion/tests/libsvn_repos/repos-test.c

(authz): Add a basic regression test for this.

  1. … 1 more file in changeset.
Adjust references to pre-ASF-migration revisions by the +840074 offset.

Did *.h, *.c, and *.py files this go-round. Was thinking of doing everything

in five minutes, until I saw CHANGES (which needs a conversion script unless

you enjoy grunt work).

* subversion/libsvn_client/externals.c

* subversion/libsvn_client/log.c

* subversion/libsvn_client/repos_diff.c

* subversion/libsvn_delta/compose_delta.c

* subversion/libsvn_subr/error.c

* subversion/libsvn_subr/io.c

* subversion/libsvn_wc/diff.c

* subversion/svnserve/serve.c

* subversion/tests/cmdline/

* subversion/tests/cmdline/

* subversion/tests/cmdline/

* subversion/tests/cmdline/

* subversion/tests/cmdline/

* subversion/tests/cmdline/

* subversion/tests/libsvn_repos/repos-test.c

* tools/buildbot/master/

* tools/dev/

  1. … 16 more files in changeset.
Remove deprecation warnings in various tests. We explicitly allow tests

to use deprecated APIs as a way of spot-testing the backward compat wrappers.

* subversion/tests/libsvn_subr/target-test.c,





Define SVN_DEPRECATED before including any Subversion headers.

  1. … 4 more files in changeset.
Once and for all, name all our anonymous struct/enum typedefs.

Follows up on r1040058, and with thanks to Danny Trebbien.

* everywhere:

Change 'typedef struct {} foo_t;' to 'typedef struct foo_t {} foo_t;'.

* subversion/libsvn_ra_serf/fake_sspi.h

(PSecurityFunctionTable, SecurityFunctionTable):

Manually effect the same change in the single pointer-to-struct typedef.

  1. … 95 more files in changeset.
Follow up to r1053174, replace all SVN_ERR_ASSERT() with SVN_TEST_ASSERT()

so assertions do not cause core dumps.

Suggested by: danielsh

* subversion/tests/libsvn_fs/fs-test.c,

* subversion/tests/libsvn_fs/locks-test.c,

* subversion/tests/libsvn_fs_base/changes-test.c,

* subversion/tests/libsvn_fs_base/fs-base-test.c,

* subversion/tests/libsvn_fs_fs/fs-pack-test.c,

* subversion/tests/libsvn_repos/repos-test.c:


  1. … 5 more files in changeset.
Add assertions in all tests that all the commits done with

svn_fs_commit_txn() and svn_repos_fs_commit_txn() return a valid

revision number, per their contract that this is the true indicator of

a commits success.

The only code not touched is test_commit_txn() in fs-test.c, which is

handled in the next commit.

* subversion/tests/libsvn_client/client-test.c,

* subversion/tests/libsvn_fs/fs-test.c,

* subversion/tests/libsvn_fs/locks-test.c,

* subversion/tests/libsvn_fs_base/changes-test.c,

* subversion/tests/libsvn_fs_base/fs-base-test.c,

* subversion/tests/libsvn_fs_fs/fs-pack-test.c,

* subversion/tests/libsvn_repos/repos-test.c,

* subversion/tests/svn_test_fs.c:

Follow any call to svn_fs_commit_txn() and svn_repos_fs_commit_txn()

that is expected to succeed with an assertion that the returned

revision number is valid, likewise, assert the reverse, that a

failed commit returns an invalid revision number.

  1. … 7 more files in changeset.
Ensure the correct type of NULL pointer is passed as a sentinel to the

variadic function apr_pstrcat().

Starting with version 4, gcc has been printing irritating warnings about

"missing sentinels in function call" for virtually every call to apr_pstrcat().

There is an issue about this in the APR bug tracker:

The fix proposed there is to include stddef.h in apr.h, in order to make

sure that NULL is defined as a pointer type, such as (void *)0.

Unfortunately, this doesn't work for all systems, since some (e.g. OpenBSD)

define NULL as 0L, rather than (void *)0.

So in spite of regrets about churn, I whipped up a sed script that changed

all places triggering the warning to use (char *)NULL instead of NULL as

sentinel, and adjusted the result to fix overlong lines and other minor

formatting issues. This fixes all current instances of the warning for me,

making it much easier to spot important warnings. No functional change.

* subversion/mod_authz_svn/mod_authz_svn.c

(get_access_conf, req_check_access, req_check_access): Use (char *)NULL

as sentinel for apr_pstrcat(), instead of NULL.

* subversion/libsvn_fs_base/lock.c

(svn_fs_base__generate_lock_token): Same.

* subversion/libsvn_fs_base/bdb/locks-table.c

(svn_fs_bdb__locks_get): Same.

* subversion/libsvn_ra_local/ra_plugin.c

(svn_ra_local__reparent): Same.

* subversion/libsvn_ra_svn/cyrus_auth.c

(try_auth, svn_ra_svn__get_addresses, svn_ra_svn__do_cyrus_auth): Same.

* subversion/libsvn_ra_svn/client.c

(ra_svn_get_locks): Same.

* subversion/mod_dav_svn/mirror.c

(static void proxy_request_fixup, dav_svn__proxy_request_fixup,

dav_svn__location_header_filter): Same.

* subversion/mod_dav_svn/reports/update.c

(malformed_element_error): Same.

* subversion/mod_dav_svn/reports/replay.c

(malformed_element_error): Same.

* subversion/mod_dav_svn/mod_dav_svn.c

(dav_svn__get_special_uri): Same.

* subversion/mod_dav_svn/lock.c

(svn_lock_to_dav_lock, unescape_xml): Same.

* subversion/mod_dav_svn/repos.c

(create_private_resource, get_parentpath_resource, get_resource,

deliver, do_walk, dav_svn__create_working_resource): Same.

* subversion/mod_dav_svn/version.c

(get_option): Same.

* subversion/libsvn_ra_neon/get_locks.c

(svn_ra_neon__get_locks): Same.

* subversion/libsvn_ra_neon/session.c

(svn_ra_neon__open): Same.

* subversion/libsvn_ra_neon/lock.c

(do_lock): Same.

* subversion/libsvn_ra_neon/props.c

(end_element, svn_ra_neon__get_props, svn_ra_neon__get_one_prop,

append_setprop): Same.

* subversion/libsvn_ra_neon/fetch.c

(add_props, filter_props, end_element): Same.

* subversion/libsvn_ra_serf/getlocks.c

(svn_ra_serf__get_locks): Same.

* subversion/libsvn_ra_serf/locks.c

(svn_ra_serf__unlock): Same.

* subversion/libsvn_ra_serf/update.c

(link_path): Same.

* subversion/libsvn_ra_serf/serf.c

(svn_ra_serf__open): Same.

* subversion/libsvn_ra_serf/property.c

(set_bare_props, svn_ra_serf__set_baton_props,

svn_ra_serf__set_baton_props): Same.

* subversion/libsvn_ra_serf/commit.c

(proppatch_walker, setup_proppatch_headers,

setup_put_headers, setup_delete_headers): Same.

* subversion/libsvn_fs_fs/caching.c

(svn_fs_fs__initialize_caches): Same.

* subversion/libsvn_fs_fs/lock.c

(svn_fs_fs__generate_lock_token): Same.

* subversion/libsvn_fs_fs/fs_fs.c

(path_txn_dir, path_txn_proto_rev, path_txn_proto_rev_lock,

path_txn_node_props, create_txn_dir, get_new_txn_node_id,

svn_fs_fs__reserve_copy_id, svn_fs_fs__set_uuid): Same.

* subversion/svn/notify.c

(notify): Same.

* subversion/svn/conflict-callbacks.c

(svn_cl__conflict_handler): Same.

* subversion/libsvn_subr/auth.c

(svn_auth_first_credentials): Same.

* subversion/libsvn_subr/checksum.c

(svn_checksum_serialize): Same.

* subversion/libsvn_subr/cache-memcache.c

(build_key): Same.

* subversion/libsvn_subr/log.c

(log_depth): Same.

* subversion/libsvn_subr/utf.c

(get_xlate_key, invalid_utf8): Same.

* subversion/libsvn_subr/opt.c

(svn_opt_format_option, svn_opt__args_to_target_array): Same.

* subversion/libsvn_subr/dirent_uri.c

(svn_uri_get_file_url_from_dirent): Same.

* subversion/libsvn_subr/sqlite.c

(internal_open): Same.

* subversion/tests/libsvn_ra_local/ra-local-test.c

(check_split_url): Same.

* subversion/tests/libsvn_repos/repos-test.c

(print_chrevs): Same.

* subversion/tests/libsvn_subr/translate-test.c

(substitute_and_verify): Same.

* subversion/tests/libsvn_subr/config-test.c

(test_text_retrieval, test_boolean_retrieval, test_has_section): Same.

* subversion/tests/libsvn_subr/dirent_uri-test.c

(test_dirent_get_absolute): Same.

* subversion/tests/libsvn_subr/target-test.c

(condense_targets_tests_helper): Same.

* subversion/libsvn_wc/adm_files.c

(simple_extend): Same.

* subversion/libsvn_wc/update_editor.c

(create_tree_conflict): Same.

* subversion/libsvn_wc/old-and-busted.c

(svn_wc__serialize_file_external): Same.

* subversion/libsvn_wc/upgrade.c

(migrate_node_props): Same.

* subversion/libsvn_wc/wc_db.c

(construct_like_arg, wclock_obtain_cb): Same.

* subversion/libsvn_client/util.c

(svn_client__path_relative_to_root): Same.

* subversion/libsvn_client/info.c

(push_dir_info): Same.

* subversion/libsvn_client/mergeinfo.c

(svn_client__mergeinfo_from_segments): Same.

* subversion/libsvn_client/cmdline.c

(resolve_repos_relative_url, svn_client_args_to_target_array,

(svn_client_args_to_target_array): Same.

* subversion/libsvn_client/externals.c

(resolve_relative_external_url): Same.

* subversion/libsvn_client/patch.c

(copy_lines_to_target): Same.

* subversion/libsvn_repos/hooks.c

(check_hook_cmd): Same.

* subversion/libsvn_repos/rev_hunt.c

(svn_repos_trace_node_locations, svn_repos_node_location_segments): Same.

* subversion/libsvn_repos/authz.c

(authz_get_path_access, authz_get_tree_access,

authz_get_global_access): Same.

* subversion/libsvn_repos/reporter.c

(add_file_smartly): Same.

  1. … 57 more files in changeset.
Revert the addition of "const" to some input pointer parameters in public

function typedefs, that was part of r922239. API compatibility was broken

by changing the typedefs in this way.

* subversion/include/svn_client.h

(svn_client_get_commit_log_t): Remove 'const' from 'apr_array_header_t *'.

* subversion/include/svn_delta.h


svn_file_rev_handler_old_t): Remove 'const' from 'apr_array_header_t *'.

* subversion/include/svn_ra.h


svn_ra_plugin_t.get_locations): Remove 'const' from 'apr_array_header_t *'.

* subversion/include/svn_ra_svn.h

(svn_ra_svn_command_handler): Remove 'const' from 'apr_array_header_t *'.

* subversion/include/svn_repos.h

(svn_repos_file_rev_handler_t): Remove 'const' from 'apr_array_header_t *'.

* subversion/libsvn_client/blame.c

(file_rev_handler): Adjust accordingly.

* subversion/libsvn_delta/compat.c

(file_rev_handler_wrapper): Adjust accordingly.

* subversion/libsvn_ra/wrapper_template.h

(compat_get_locations): Adjust accordingly.

* subversion/mod_dav_svn/reports/file-revs.c

(file_rev_handler): Adjust accordingly.

* subversion/svnserve/serve.c

(set_path, delete_path, link_path, finish_report, abort_report, reparent,

get_latest_rev, get_dated_rev, change_rev_prop, rev_proplist, rev_prop,

commit, get_file, get_dir, update, switch_cmd, status, diff,

get_mergeinfo, log_cmd, check_path, stat_cmd, get_locations,

file_rev_handler, get_file_revs, lock, lock_many, unlock, unlock_many,

get_lock, get_locks, replay, replay_range, get_deleted_rev):

Adjust accordingly.

* subversion/tests/libsvn_repos/repos-test.c

(file_rev_handler): Adjust accordingly.

  1. … 10 more files in changeset.
Cosmetic: indents and line wraps, else-ifs.

* subversion/libsvn_client/export.c (copy_one_versioned_file): Line width wrap.

* subversion/libsvn_client/merge.c (merge_file_changed): Else-if wrap/indent.

* subversion/libsvn_wc/adm_crawler.c

(svn_wc__internal_transmit_text_deltas): Indent.

* subversion/libsvn_wc/adm_ops.c (svn_wc__get_pristine_contents): Else-if wrap.

* subversion/libsvn_wc/old-and-busted.c (read_url): Else-if wrap/indent.

* subversion/libsvn_wc/questions.c (svn_wc__internal_text_modified_p): Indent.

* subversion/libsvn_wc/update_editor.c (check_tree_conflict): Else-if wrap.

* subversion/svnlook/main.c (do_plist): Else-if wrap/indent.

* subversion/svn/propset-cmd.c (svn_cl__propset): Else-if wrap/indent.

* subversion/tests/libsvn_repos/repos-test.c

(prop_validation_commit_with_revprop, prop_validation): Else-if wrap/indent.

  1. … 9 more files in changeset.
Add "const" to all "apr_array_header_t *" input parameters in public and

private APIs.

(Note that we are allowed to add "const" to a pointer parameter of a public

API, and have done so before, because that is a backward-compatible API

change and doesn't change the ABI at all.)

The public functions whose prototypes are changed are listed individually

below; the private functions are not.

* subversion/include/svn_auth.h


* subversion/include/svn_client.h






* subversion/include/svn_delta.h




* subversion/include/svn_fs.h


* subversion/include/svn_mergeinfo.h








* subversion/include/svn_opt.h





* subversion/include/svn_ra.h




* subversion/include/svn_ra_svn.h





* subversion/include/svn_repos.h






* subversion/include/svn_sorts.h


* subversion/include/svn_string.h


* subversion/include/svn_wc.h







* subversion/include/private/svn_cmdline_private.h

* subversion/include/private/svn_opt_private.h

* subversion/libsvn_client/blame.c

* subversion/libsvn_client/client.h

* subversion/libsvn_client/commit.c

* subversion/libsvn_client/commit_util.c

* subversion/libsvn_client/copy.c

* subversion/libsvn_client/deprecated.c

* subversion/libsvn_client/merge.c

* subversion/libsvn_client/mergeinfo.c

* subversion/libsvn_client/mergeinfo.h

* subversion/libsvn_client/patch.c

* subversion/libsvn_client/ra.c

* subversion/libsvn_delta/compat.c

* subversion/libsvn_delta/path_driver.c

* subversion/libsvn_fs_base/reps-strings.c

* subversion/libsvn_fs_base/revs-txns.c

* subversion/libsvn_fs_base/revs-txns.h

* subversion/libsvn_fs_fs/fs_fs.c

* subversion/libsvn_fs_fs/fs_fs.h

* subversion/libsvn_fs/fs-loader.c

* subversion/libsvn_fs/fs-loader.h

* subversion/libsvn_ra/compat.c

* subversion/libsvn_ra/ra_loader.c

* subversion/libsvn_ra/ra_loader.h

* subversion/libsvn_ra/wrapper_template.h

* subversion/libsvn_ra_local/ra_plugin.c

* subversion/libsvn_ra_neon/get_locations.c

* subversion/libsvn_ra_neon/props.c

* subversion/libsvn_ra_neon/ra_neon.h

* subversion/libsvn_ra_serf/getlocations.c

* subversion/libsvn_ra_serf/ra_serf.h

* subversion/libsvn_ra_svn/client.c

* subversion/libsvn_ra_svn/cyrus_auth.c

* subversion/libsvn_ra_svn/editorp.c

* subversion/libsvn_ra_svn/internal_auth.c

* subversion/libsvn_ra_svn/marshal.c

* subversion/libsvn_ra_svn/ra_svn.h

* subversion/libsvn_repos/fs-wrap.c

* subversion/libsvn_repos/log.c

* subversion/libsvn_repos/repos.c

* subversion/libsvn_repos/repos.h

* subversion/libsvn_repos/rev_hunt.c

* subversion/libsvn_subr/auth.c

* subversion/libsvn_subr/cmdline.c

* subversion/libsvn_subr/deprecated.c

* subversion/libsvn_subr/mergeinfo.c

* subversion/libsvn_subr/opt.c

* subversion/libsvn_subr/sorts.c

* subversion/libsvn_subr/svn_string.c

* subversion/libsvn_wc/adm_ops.c

* subversion/libsvn_wc/deprecated.c

* subversion/libsvn_wc/diff.c

* subversion/libsvn_wc/status.c

* subversion/libsvn_wc/update_editor.c

* subversion/libsvn_wc/util.c

* subversion/mod_dav_svn/dav_svn.h

* subversion/mod_dav_svn/reports/file-revs.c

* subversion/mod_dav_svn/util.c

* subversion/svn/cl.h

* subversion/svn/props.c

* subversion/svn/switch-cmd.c

* subversion/svn/util.c

* subversion/svndumpfilter/main.c

* subversion/svnserve/serve.c

* subversion/tests/libsvn_repos/repos-test.c

* subversion/tests/libsvn_subr/mergeinfo-test.c

  1. … 78 more files in changeset.
Add a (partial) test for issue #3243, and move some common code to the

test framework for later reuse.

* subversion/tests/libsvn_repos/repos-test.c

(test_get_file_revs): New test svn_repos_get_file_revs2.

(file_rev_handler): Helper function.

(test_funcs): Run the new test.

* subversion/tests/libsvn_ra_local/ra-local-test.c

(global): Remove now unneeded includes.

(make_and_open_local_repos): Update function call.

(current_directory_url): Move function from here ...

* subversion/tests/svn_test_fs.c

(svn_test__current_directory_url): ... to here.

(svn_test__create_greek_tree_at): New test function, create greek

tree at a path (doesn't have to be '/') in the repository.

(svn_test__create_greek_tree): Wrap svn_test__create_greek_tree_at.

(svn_test__create_blame_repository): Set up a new repo, similar to

the one used to reproduce issue #3243.

* subversion/tests/svn_test_fs.h



svn_test__current_directory_url): New function declarations.

  1. … 3 more files in changeset.