Re: pgsql: Move security_label test

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Move security_label test
Date: 2014-12-02 07:59:19
Message-ID: CAB7nPqSCK7ArNC0OfjjONZX0ieRnPRFJ2ih7qxO1A8LjL4z7bw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Tue, Dec 2, 2014 at 4:13 AM, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
> Move security_label test
>
> Rather than have the core security_label regression test depend on the
> dummy_seclabel module, have that part of the test be executed by
> dummy_seclabel itself directly. This simplifies the testing rig a bit;
> in particular it should silence the problems from the MSVC buildfarm
> phylum, which haven't yet gotten taught how to install src/test/modules.
After this module has been moved to src/test/modules, I am seeing
failures with make check when manually kicking the test on at least
OSX where I tried because expected/ and sql/ are missing. pg_regress
does not like that much. That's annoying because check-world would
simply fail.
Also, I think that src/test/modules/dummy_seclabel/Makefile should be
taught to clean sql/dummy_seclabel.sql and expected/dummy_seclabel.out
with EXTRA_CLEAN and that we should have a .gitignore to ignore
tmp_check/ results/ and log/ in this path. Per se the patch attached
with those things added.
Thanks,
--
Michael

Attachment Content-Type Size
20141202_dummy_seclabel_fix.patch text/x-diff 1.3 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2014-12-02 14:15:21 pgsql: dummy_seclabel: add sql/, expected/, and .gitignores
Previous Message Tom Lane 2014-12-01 20:25:28 pgsql: Guard against bad "dscale" values in numeric_recv().