Re: Build broken with nmake since ea74e47

From: "Inoue, Hiroshi" <h-inoue(at)dream(dot)email(dot)ne(dot)jp>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: PostgreSQL mailing lists <pgsql-odbc(at)postgresql(dot)org>
Subject: Re: Build broken with nmake since ea74e47
Date: 2016-07-02 11:18:07
Message-ID: 5777A2EF.2070901@dream.email.ne.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-odbc

Hi Michael,

On 2016/07/01 13:27, Michael Paquier wrote:
> On Thu, Jun 30, 2016 at 12:45 AM, Michael Paquier
> <michael(dot)paquier(at)gmail(dot)com> wrote:
>> Hi,
>>
>> Since commit ea74e47, the build with nmake is broken:
>> NMAKE : fatal error U1073: don't know how to make 'exe\connect-test.exe'
>> I have yet to diagnose what is wrong here, but a guess is that win.mak
>> is missing a trick.
> After spending some time on that, I can reproduce the problem, but I
> have little idea how to properly link each exe and src file with
> common.obj at the same time using this nmake Makefile. ea74e47 has
> really introduced a broken concept in win.mak, and I am sure that this
> has not been tested. Could it be possible to get back to the prior
> behavior for just nmake? For example the patch attached does the job
> for me again.

Oops my mistake.
The commit was irrelevant to Windows..
I would put back the change.
Thanks.

>
> At the same time, I have noticed a compilation error in
> large-object-data-at-exec-test.c regarding a type declaration in the
> middle of a code block, basically that:
> - int error = SQL_SUCCESS != rc && SQL_SUCCESS_WITH_INFO != rc
> && SQL_NEED_DATA != rc;
> + error = SQL_SUCCESS != rc &&
> + SQL_SUCCESS_WITH_INFO != rc &&
> + SQL_NEED_DATA != rc;

OK I would fix the compilation error.
Thanks.

Hiroshi Inoue

In response to

Responses

Browse pgsql-odbc by date

  From Date Subject
Next Message Huong Dangminh 2016-07-03 23:06:19 Re: Default for "Level of rollback on errors"
Previous Message Inoue, Hiroshi 2016-07-02 10:58:36 Re: Default for "Level of rollback on errors"