On 05/23/2011 02:28 PM, Prabhu Gnana Sundar wrote:
> On Monday 23 May 2011 02:02 PM, Stefan Sperling wrote:
>> On Mon, May 23, 2011 at 01:08:28PM +0530, Prabhu Gnana Sundar wrote:
>>> Hi all,
>>>
>>> I have attached my recent patch along with this mail. Please
>>> share your views.
>>>
>>>
>> Can you provide a log message, too?
>>
> Sure Stefan :)
>
> Attaching the log message with this mail. Also attached is the updated
> patch with the change that you suggested below.
>
>
>
> Thanks and regards
> Prabhu
>>> Index: subversion/svnadmin/main.c
>>> ===================================================================
>>> --- subversion/svnadmin/main.c (revision 1126350)
>>> +++ subversion/svnadmin/main.c (working copy)
>>> @@ -585,7 +585,6 @@
>>> {
>>> struct svnadmin_opt_state *opt_state = baton;
>>> svn_repos_t *repos;
>>> - apr_hash_t *config;
>>> apr_hash_t *fs_config = apr_hash_make(pool);
>>>
>>> /* Expect no more arguments. */
>>> @@ -624,12 +623,9 @@
>>> APR_HASH_KEY_STRING,
>>> "1");
>>>
>>> - SVN_ERR(svn_config_get_config(&config, opt_state->config_dir,
>>> pool));
>>> - SVN_ERR(svn_cmdline__apply_config_options(config,
>>> opt_state->config_options,
>>> - "svnadmin: ",
>>> "--config-option"));
This PATCH ignores --config-option.
Bert via r1080198 you seemed to have introduced --config-option to svnadmin.
I am not aware of a case where --config-option would be needed for svnadmin.
Can you clarify?
With regards
Kamesh Jayachandran
>>> SVN_ERR(svn_repos_create(&repos, opt_state->repository_path,
>>> NULL, NULL,
>>> - config, fs_config, pool));
>>> + NULL, fs_config, pool));
>> We could wrap the above two lines into a single line.
>> So that we have just one line reading:
>>
>> NULL, NULL, NULL, fs_config_pool));
>>
>> Looks good to me, thanks!
>>
>>> svn_fs_set_warning_func(svn_repos_fs(repos), warning_func, NULL);
>>> return SVN_NO_ERROR;
>>> }
>
Received on 2011-05-23 11:41:31 CEST