Re: pgsql: Move security_label test

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

Michael Paquier wrote:
> 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.

I had the directories locally but hadn't git-pushed them.

> 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.

Pushed the whole thing, thanks.

--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2014-12-02 15:22:53 pgsql: pageinspect/BRIN: minor tweaks
Previous Message Alvaro Herrera 2014-12-02 14:15:21 pgsql: dummy_seclabel: add sql/, expected/, and .gitignores