Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path - Mailing list pgsql-hackers

From Rajeev rastogi
Subject Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path
Date
Msg-id BF2827DCCE55594C8D7A8F7FFD3AB7713DDBECD4@SZXEML508-MBX.china.huawei.com
Whole thread Raw
In response to Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path  ("MauMau" <maumau307@gmail.com>)
Responses Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path  ("MauMau" <maumau307@gmail.com>)
List pgsql-hackers
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

pgsql-hackers by date:

Previous
From: "MauMau"
Date:
Subject: Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path
Next
From: Christian Kruse
Date:
Subject: Re: Patch: show xid and xmin in pg_stat_activity and pg_stat_replication