Re: TODO-Item: include for guc

Lists: pgsql-patches
From: Joachim Wieland <joe(at)mcknight(dot)de>
To: pgsql-patches(at)postgresql(dot)org
Subject: TODO-Item: include for guc
Date: 2005-12-01 23:25:07
Message-ID: 20051201232507.GA4213@mcknight.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

Hi,

the attached patch implements the TODO-item

o %Add "include file" functionality in postgresql.conf

However I was not able to test it on Windows :-(

I talked to Peter Eisentraut about how to handle relative file names and his
opinion was that a relative file name should be interpreted to be relative
to the path of the including file, i.e. to the path of the file that
contained the actual "include = other.conf" line. The patch follows this
idea.

Joachim

Attachment Content-Type Size
includeforguc.diff text/plain 11.3 KB

From: Joachim Wieland <joe(at)mcknight(dot)de>
To: pgsql-patches(at)postgresql(dot)org
Subject: Re: TODO-Item: include for guc
Date: 2006-01-20 18:47:40
Message-ID: 20060120184740.GA16874@mcknight.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

The attached patch is a revised version of the patch I sent in before:

http://archives.postgresql.org/pgsql-patches/2005-12/msg00018.php

The old version did not apply to current CVS anymore and the new one
now also works with flex 2.5.4.

The patch works as proposed by Tom Lane in:

http://archives.postgresql.org/pgsql-patches/2004-06/msg00030.php
(see lower part)

Joachim

Attachment Content-Type Size
includeforguc.diff text/plain 12.3 KB

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Joachim Wieland <joe(at)mcknight(dot)de>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: TODO-Item: include for guc
Date: 2006-01-20 21:32:42
Message-ID: 200601202132.k0KLWgt16293@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches


Previous patch version removed from queue.

---------------------------------------------------------------------------

Joachim Wieland wrote:
> Hi,
>
> the attached patch implements the TODO-item
>
> o %Add "include file" functionality in postgresql.conf
>
> However I was not able to test it on Windows :-(
>
> I talked to Peter Eisentraut about how to handle relative file names and his
> opinion was that a relative file name should be interpreted to be relative
> to the path of the including file, i.e. to the path of the file that
> contained the actual "include = other.conf" line. The patch follows this
> idea.
>
>
> Joachim

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/docs/faq

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073


From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Joachim Wieland <joe(at)mcknight(dot)de>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: TODO-Item: include for guc
Date: 2006-03-05 05:07:15
Message-ID: 200603050507.k2557Fd26806@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches


Patch applied. Thanks.

---------------------------------------------------------------------------

Joachim Wieland wrote:
> The attached patch is a revised version of the patch I sent in before:
>
> http://archives.postgresql.org/pgsql-patches/2005-12/msg00018.php
>
> The old version did not apply to current CVS anymore and the new one
> now also works with flex 2.5.4.
>
> The patch works as proposed by Tom Lane in:
>
> http://archives.postgresql.org/pgsql-patches/2004-06/msg00030.php
> (see lower part)
>
>
> Joachim
>

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Have you searched our list archives?
>
> http://archives.postgresql.org

--
Bruce Momjian http://candle.pha.pa.us
SRA OSS, Inc. http://www.sraoss.com

+ If your life is a hard drive, Christ can be your backup. +