Re: pg_hba.conf: samehost and samenet

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

In response to

Browse pgsql-hackers by date

  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