Re: sepgsql contrib module - Mailing list pgsql-hackers

From Robert Haas
Subject Re: sepgsql contrib module
Date
Msg-id AANLkTinr5wuXVAFGT7R6kfbVzGLToxJkpsg05EZixCLb@mail.gmail.com
Whole thread Raw
In response to Re: sepgsql contrib module  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: sepgsql contrib module  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Tue, Feb 15, 2011 at 10:50 AM, Andrew Dunstan <andrew@dunslane.net> wrote:
> On 02/15/2011 10:34 AM, Robert Haas wrote:
>> On Mon, Feb 14, 2011 at 9:55 PM, Tom Lane<tgl@sss.pgh.pa.us>  wrote:
>>> On the whole, I don't think that sepgsql-regtest.pp should be built or
>>> installed at all during the build phase.  It ought to be generated
>>> during regression test startup, instead.
>>
>> You have to manually install and enable it before you can run the
>> regression tests.  This is documented.
>
> That's not the point. Right now you can't even run just a postgresql build
> on a machine that doesn't have selinux enabled, let alone run the regression
> tests. And if we construct the regression gadget at postgresql build time,
> who is to say it has the right settings for the run time environment, given
> that the build is apparently dependent on a runtime kernel setting?

Those are good points.  My point was just that you can't actually
build that file at the time you RUN the regression tests, because you
have to build it first, then install it, then run the regression
tests.  It could be a separate target, like 'make policy', but I don't
think it works to make it part of 'make installcheck'.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pageinspect's infomask and infomask2 as smallint
Next
From: Robert Haas
Date:
Subject: Re: pageinspect's infomask and infomask2 as smallint