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

2013-11-21 Thread Amit Kapila
On Thu, Nov 21, 2013 at 4:22 PM, 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]

Strange, I could not get this error on latest code.
I think you need to regenerate preproc.y by using command perl
mkvcbuild.pl before build.


With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


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

2013-11-21 Thread Boszormenyi Zoltan

2013-11-21 11:52 keltezéssel, David Rowley írta:
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 

+++ b/src/interfaces/ecpg/preproc/type.h 

@@ -186,7 
 
+186,7 
 
@@ struct assignment

 enum errortype
 {
-   ET_WARNING, ET_ERROR, ET_FATAL
+   ET_WARNING, ET_ERROR
 };


Regards

David Rowley


You should regenerate preproc.y.
"./configure --enable-depend" does the trick under Linux
but I don't know how to do it under Windows.

Best regards,
Zoltán Böszörményi

--
--
Zoltán Böszörményi
Cybertec Schönig & Schönig GmbH
Gröhrmühlgasse 26
A-2700 Wiener Neustadt, Austria
Web: http://www.postgresql-support.de
 http://www.postgresql.at/



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

2013-11-21 Thread Andrew Dunstan


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 

+++ b/src/interfaces/ecpg/preproc/type.h 

@@ -186,7 
 
+186,7 
 
@@ 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


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers