From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
Cc: | Erik Rijkers <er(at)xs4all(dot)nl>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>, pgsql-committers-owner(at)postgresql(dot)org |
Subject: | Re: pgsql: Default monitoring roles |
Date: | 2017-03-30 19:36:39 |
Message-ID: | CA+OCxoxTuTVn4LRG-jkEcgs9nLq-YEg96gU2sVGTaJJrckS1bw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Thu, Mar 30, 2017 at 2:57 PM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> On 30 March 2017 at 19:31, Erik Rijkers <er(at)xs4all(dot)nl> wrote:
>> On 2017-03-30 20:20, Simon Riggs wrote:
>>>
>>> Default monitoring roles
>>
>>
>>
>> The buildfarm is showing red (the same errors that I get...):
>>
>>
>> pgrowlocks.c: In function ‘pgrowlocks’:
>> pgrowlocks.c:105:65: error: expected ‘)’ before ‘;’ token
>> is_member_of_role(GetUserId(), DEFAULT_ROLE_STAT_SCAN_TABLES);
>> ^
>> [...]
>
> Weird. make check-world just skipped that directory. I guess for Dave also.
>
> Bug fixed, but will look at makefile
Yes, Stephen and I were just trying to figure out how that happened. I
ran make check from within /contrib before I sent the patch, and that
passes just fine. I guess maybe it doesn't build modules that don't
have any tests.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-03-30 20:20:21 | Re: [COMMITTERS] pgsql: Default monitoring roles |
Previous Message | Tom Lane | 2017-03-30 19:27:23 | Re: [COMMITTERS] pgsql: Default monitoring roles |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-03-30 19:43:25 | Re: New CORRESPONDING clause design |
Previous Message | Dave Page | 2017-03-30 19:31:55 | Re: Monitoring roles patch |