Re: [GENERAL] patches

Travis Bauer <trbauer(at)indiana(dot)edu> writes:
> What is the appropriate way to submit a patch to a source file. From what
> I read, I'm supposed to send it here, but do I just attach the changed
> file, or do I send in only the changes?

Complete files are not good since it's too hard to see what you changed,
and even harder to merge your changes if anyone else has edited the file
since the version you were working from. Instead, diff -c or diff -u
against the unedited files is the best way. For example, a u-format
submission would look something like

--- src/backend/utils/adt/arrayfuncs.c.orig 2000/05/30 04:24:50
+++ src/backend/utils/adt/arrayfuncs.c 2000/06/02 15:57:28
@@ -549,11 +549,7 @@
{
FILE *afd;

-#ifndef __CYGWIN32__
- if ((afd = AllocateFile(accessfile, "r")) == NULL)
-#else
- if ((afd = AllocateFile(accessfile, "r")) == NULL)
-#endif
+ if ((afd = AllocateFile(accessfile, PG_BINARY_R)) == NULL)
elog(ERROR, "unable to open access pattern file");
*chunkFlag = true;
retStr = _ChunkArray(*fd, afd, ndim, dim, baseSize, nbytes,

regards, tom lane