Re: b21de4e7b32f868a23bdc5507898d36cbe146164 seems to be two bricks shy of a load

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: David Rowley <dgrowleyml(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: b21de4e7b32f868a23bdc5507898d36cbe146164 seems to be two bricks shy of a load
Date: 2013-11-21 14:06:42
Message-ID: 528E1372.6030208@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 11/21/2013 05:52 AM, David Rowley wrote:
> I'm not quite sure why nobody else seems to be complaining, but the
> changes to type.h in this commit seems to have broken things little.
>
> In the visual studios build I'm getting:
>
> src\interfaces\ecpg\preproc\preproc.y(84): error C2065: 'ET_FATAL' :
> undeclared identifier [D:\Postgres\b\ecpg.vcxproj]
> src\interfaces\ecpg\preproc\preproc.y(84): error C2051: case
> expression not constant [D:\Postgres\b\ecpg.vcxproj]
> src\interfaces\ecpg\preproc\preproc.y(102): error C2065: 'ET_FATAL'
> : undeclared identifier [D:\Postgres\b\ecpg.vcxproj]
> src\interfaces\ecpg\preproc\preproc.y(102): error C2051: case
> expression not constant [D:\Postgres\b\ecpg.vcxproj]
> src\interfaces\ecpg\preproc\preproc.y(14664): error C2065:
> 'ET_FATAL' : undeclared identifier [D:\Postgres\b\ecpg.vcxproj]
>
> Which I'm guessing is something to do with:
>
> --- a/src/interfaces/ecpg/preproc/type.h
> <http://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/preproc/type.h;h=b7768fd66146e16547f385d30272e2a42e4d6832;hb=b7768fd66146e16547f385d30272e2a42e4d6832>
> +++ b/src/interfaces/ecpg/preproc/type.h
> <http://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/preproc/type.h;h=cd0d1da8c4a1a641acd9f6b6b7dfa88e7241394d;hb=cd0d1da8c4a1a641acd9f6b6b7dfa88e7241394d>
> @@ -186,7
> <http://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/preproc/type.h;h=b7768fd66146e16547f385d30272e2a42e4d6832;hb=b7768fd66146e16547f385d30272e2a42e4d6832#l186>
> +186,7
> <http://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/interfaces/ecpg/preproc/type.h;h=cd0d1da8c4a1a641acd9f6b6b7dfa88e7241394d;hb=cd0d1da8c4a1a641acd9f6b6b7dfa88e7241394d#l186>
> @@ struct assignment
> enum errortype
> {
> - ET_WARNING, ET_ERROR, ET_FATAL
> + ET_WARNING, ET_ERROR
> };
>
>
>

(Note, please use shortened commit hashes in your subjects - the leading
7 hex digits of the hash are usually sufficient to identify it.)

The buildfarm animals using MSVC are not exhibiting this problem. When
you see something like this, the buildfarm is probably the first place
you should look.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Kevin Grittner 2013-11-21 14:22:50 Re: pg_upgrade ?deficiency
Previous Message Amit Kapila 2013-11-21 13:55:00 Re: Proof of concept: standalone backend with full FE/BE protocol