From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, List pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Recent SIGSEGV failures in buildfarm HEAD |
Date: | 2007-01-02 21:16:42 |
Message-ID: | 459ACBBA.5020801@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>> Revised patch attached, doing just this. I will apply it soon unless
>> there are objections.
>>
>
> Probably a good idea to check defined(HAVE_GETRLIMIT) && defined(RLIMIT_CORE),
> rather than naively assuming every getrlimit implementation supports
> that particular setting. Also, should the -c option exist but just not
> do anything if the platform doesn't support it? As is, you're making it
> impossible to just specify -c without worrying if it does anything.
>
> The documentation fails to list the long form of the switch
> (--corefiles, which should probably really be --core-files for consistency).
> There's a typo in this message, too:
>
> + _("%s: cannot set core size,: disallowed by hard limit.\n"),
>
>
>
OK, I'll fix all this. Thanks.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | uwcssa | 2007-01-02 21:31:25 | SearchSysCache |
Previous Message | Tom Lane | 2007-01-02 21:11:37 | Re: [HACKERS] Recent SIGSEGV failures in buildfarm HEAD |
From | Date | Subject | |
---|---|---|---|
Next Message | Bill Moran | 2007-01-02 21:47:19 | Patch to log usage of temporary files |
Previous Message | Tom Lane | 2007-01-02 21:11:37 | Re: [HACKERS] Recent SIGSEGV failures in buildfarm HEAD |