Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path

From: "MauMau" <maumau307(at)gmail(dot)com>
To: "Rajeev rastogi" <rajeev(dot)rastogi(at)huawei(dot)com>
Cc: <pgsql-hackers(at)postgresql(dot)org>, "Magnus Hagander" <magnus(at)hagander(dot)net>
Subject: Re: [review] PostgreSQL Service on Windows does not start if data directory given is relative path
Date: 2014-02-22 00:46:15
Message-ID: 10DE394009FB4117B17C8807B92FDB62@maumau
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Rajeev,

From: "Rajeev rastogi" <rajeev(dot)rastogi(at)huawei(dot)com>
Changed the patch as per your suggestion.
Now only one version of make_absolute_path there defined in src/port/path.c

Found one small memory leak in the existing function
make_absolute_path(miscinit.c),
fixed the same.

Thanks for refactoring. I confirmed that the revised patch applies to HEAD
cleanly, the source files built without extra warnings, and the original
intended problem was solved.

Please make small cosmetic changes so that make_absolute_path() follows the
style of other parts. Then I'll make this ready for committer.

(1)
Add the function name in the comment as in:

/*
* make_absolute_path
*
* ...existing function descripton
*/

(2)
Add errno description as in:

fprintf(stderr, _("could not get current working directory: %s\n",
strerror(errno)));

Regards
MauMau

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2014-02-22 01:07:23 Re: pg_stat_tmp files for dropped databases
Previous Message Thom Brown 2014-02-22 00:13:18 pg_stat_tmp files for dropped databases