From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Mark Dilger <hornschnorter(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Hao Lee <mixtrue(at)gmail(dot)com>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
Subject: | Re: Windows build broken starting at da9b580d89903fee871cf54845ffa2b26bda2e11 |
Date: | 2018-05-15 16:29:23 |
Message-ID: | 20180515162923.GN27724@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Greetings,
* Mark Dilger (hornschnorter(at)gmail(dot)com) wrote:
> > On May 15, 2018, at 8:58 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > Mark Dilger <hornschnorter(at)gmail(dot)com> writes:
> >> My best guess at the moment is:
> >
> >> diff --git a/src/backend/utils/init/globals.c b/src/backend/utils/init/globals.c
> >> index c1f0441b08..0a3163398f 100644
> >> --- a/src/backend/utils/init/globals.c
> >> +++ b/src/backend/utils/init/globals.c
> >> @@ -16,8 +16,11 @@
> >> *
> >> *-------------------------------------------------------------------------
> >> */
> >> +#include <sys/stat.h>
> >> +
> >> #include "postgres.h"
> >
> >> +#include "common/file_perm.h"
> >
> > Yipes. Frost, you didn't really do that did you? That's a blatant
> > break of the "c.h must come first" rule. Whether or not it broke the
> > Windows build, there are other platforms it'll break.
Evidently I managed to.
> >> Indeed, the following change (shown here for illustrative purposes only; please
> >> don't commit it this way) fixes the problem, at least in my build environment:
> >
> > That's pretty ugly, but what happens if you just move the <sys/stat.h>
> > inclusion to immediately after postgres.h, as is our normal custom?
>
> That also works.
Good, will fix.
Thanks!
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Mark Dilger | 2018-05-15 16:39:23 | Re: Windows build broken starting at da9b580d89903fee871cf54845ffa2b26bda2e11 |
Previous Message | Mark Dilger | 2018-05-15 16:27:30 | Re: Windows build broken starting at da9b580d89903fee871cf54845ffa2b26bda2e11 |