On 3rd February 2014, MauMau Wrote: 

> Thanks, your patch looks good.  I confirmed the following:
> 
> * Applied cleanly to the latest source tree, and built on Linux and
> Windows (with MSVC) without any warnings.
> 
> * Changed to D:\ and ran "pg_ctl register -N pg -D pgdata", and checked
> the registry value ImagePath with regedit.  It contained -D "D:\pgdata".
> 
> * Changed to D:\pgdata and ran "pg_ctl register -N pg -D ..\pgdata",
> and checked the registry value ImagePath with regedit.  It contained -D
> "D:\pgdata".

Thanks for reviewing and testing the patch.

> Finally, please change the function description comment of
> get_absolute_path() so that the style follows other function in path.c.
> That is:
> 
> /*
>  * <function_name>
>  *
>  * Function description
>  */

I have modified the function description comment as per your suggestion.

> Then update the CommitFest entry with the latest patch and let me know
> of that.  Then, I'll change the patch status to ready for committer.

I will update commitFest with the latest patch immediately after sending the 
mail.

Thanks and Regards,
Kumar Rajeev Rastogi



Attachment: pgctl_win32service_rel_dbpath_v4.patch
Description: pgctl_win32service_rel_dbpath_v4.patch

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to