Re: [PATCH] Find additional connection service files inpg_service.conf.d directory - Mailing list pgsql-hackers

From Curt Tilmes
Subject Re: [PATCH] Find additional connection service files inpg_service.conf.d directory
Date
Msg-id CAMv20O9XHvSJO_ed84QfM6D4j5j4H0VMbw=ZB9_XuT33gYfsDA@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Find additional connection service files inpg_service.conf.d directory  ("Andreas 'ads' Scherbaum" <adsmail@wars-nicht.de>)
List pgsql-hackers
On Mon, Mar 5, 2018 at 7:45 AM, Andreas 'ads' Scherbaum
<adsmail@wars-nicht.de> wrote:
> On Thu, Mar 1, 2018 at 7:40 PM, Curt Tilmes <curt@tilmes.org> wrote:
>> On Thu, Mar 1, 2018 at 1:13 PM, Andres Freund <andres@anarazel.de> wrote:
> Looked over this patch and found that you limit the directory entries to
> files only.
> Would it make sense to allow links as well? That would allow other
> programs/distributions to place a link in the config directory and point to
> a service file in their own directory.

I call stat(2), which from my understanding for a link will follow the
link and return
the information from the file linked to, not the link itself (which
lstat(2) would do).

I tried it out and it seems to work fine for links.

Curt


pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: Transform for pl/perl
Next
From: Alvaro Herrera
Date:
Subject: Re: Incorrect use of "an" and "a" in code comments and docs