From: | Stef Walter <stef-list(at)memberwebs(dot)com> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: pg_hba.conf: samehost and samenet |
Date: | 2009-08-14 20:26:32 |
Message-ID: | 20090814202631.31AC9303982F@mx.npubs.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Magnus Hagander wrote:
>
> A couple of comments on the patch:
Thanks I'll keep these in mind, as things progress and for future patches.
> * In general, don't include configure in the patch. Just configure.in.
> Makes it easier to read, and configure is normally built by the
> committer anyway.
>
> * How portable is this? For starters is clearly doesn't do Windows,
> which would need to be investigated for similar functionality, but how
> many others support getifaddr()? From what I can tell it's not in
> POSIX, at least.
I'll look further into this, and about compat shims. getifaddrs() is on
the BSDs, Linux and Mac OS.
> Please add it to the open commitfest
> (https://commitfest.postgresql.org/action/commitfest_view/open) This
> will cause it to be reviewed during the next commitfest, and then you
> just need to be around to answer any questions that reviewers come up
> with :-)
Cool, I'll do that once we've worked out the kinks here. Is the right
way to go about it?
Cheers,
Stef
From | Date | Subject | |
---|---|---|---|
Next Message | Stef Walter | 2009-08-14 20:32:33 | Re: pg_hba.conf: samehost and samenet |
Previous Message | Boszormenyi Zoltan | 2009-08-14 20:12:24 | Re: Split-up ECPG patches |