| From: | Craig Ringer <craig(at)postnewspapers(dot)com(dot)au> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, Dave Page <dpage(at)pgadmin(dot)org>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Re: Proposed Windows-specific change: Enable crash dumps (like core files) |
| Date: | 2010-12-16 08:21:03 |
| Message-ID: | 4D09CBEF.3090007@postnewspapers.com.au |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi all
Updated patch attached. This one avoids the inline function stuff and
instead just decides whether to compile unix_crashdump.c or
win32_crashdump.c based on build system tests. It passes "make check" on
nix and tests on win32, both with and without the "crashdumps" directory
existing. The inadvertent duplication of the functon prototype is fixed.
I haven't added any SGML documentation yet, as I'm not sure (a) to what
level it should be documented, and (b) whether it should be in the main
docs or just the developer docs plus the crash debugging guide on the
wiki. Thoughts?
You can also find this patch at:
https://github.com/ringerc/postgres/tree/crashdump
ie git://github.com/ringerc/postgres.git, branch "crashdump" .
--
System & Network Administrator
POST Newspapers
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Add-support-for-generating-a-crash-dump-on-backend-p.patch | text/x-patch | 0 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Craig Ringer | 2010-12-16 08:28:23 | Re: Re: Proposed Windows-specific change: Enable crash dumps (like core files) |
| Previous Message | Pavel Stehule | 2010-12-16 06:20:13 | orafce on 64 bit windows |