Re: contribcheck and modulescheck of MSVC's vcregress.pl cannot work independently

From: Noah Misch <noah(at)leadboat(dot)com>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL mailing lists <pgsql-bugs(at)postgresql(dot)org>, peter_e(at)gmx(dot)net
Subject: Re: contribcheck and modulescheck of MSVC's vcregress.pl cannot work independently
Date: 2015-07-12 06:57:58
Message-ID: 20150712065758.GA1085189@tornado.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Mon, Jul 06, 2015 at 11:53:58PM -0700, Michael Paquier wrote:
> On Mon, Jul 6, 2015 at 10:04 PM, Noah Misch wrote:
> >> Also, if we would
> >> want to have a real installcheck mode, what we should use is not the
> >> path to what has been built but the path to the installation that the
> >> Postgres instance needed is using. Now if you want to fix it if you
> >> fix that's incorrect I won't complain about that :)
> >
> > I don't wish to. I was content with released-branch vcregress.pl semantics.
>
> Attached is a patch to revert to the pre-9.4 way of doing...

This is a start. Study "git diff REL9_4_STABLE src/tools/msvc/vcregress.pl"
after applying it. To give just one example of material left to revert, it
leaves two absolute-path "chdir" calls in a row.

> If MSVC
> should imitate what make does, perhaps you have an opinion about the
> ideas I sent upthead to add a check-mode to contribcheck and
> modulescheck? That's 9.6 work.

As you note, we lack a predefined notion of the installation path like we have
in the GNU make build. That challenges our ability to make the two build
systems more similar than they are in 9.4. While I have occasionally wished
for a vcregress.pl target corresponding to "make -C contrib check", it's also
one more thing to bitrot in an area that already experiences too much bitrot.
Overall, I recommend against this project. The benefits of a correct patch
are slight, yet writing one is difficult.

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Jaimin Pan 2015-07-12 12:21:10 [Postgresql Master Branch Patch] object class patch
Previous Message Tom Lane 2015-07-11 22:06:19 Re: BUG #13490: Segmentation fault on pg_stat_activity