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-06 12:51:31
Message-ID: 577CFED3.1090002@dream.email.ne.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-odbc

Hi Michael,

On 2016/07/02 20:18, Inoue, Hiroshi wrote:
> 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.

After looking at regression test on Windows, I'm inclined to go forward
rather
than putting back win.mak, i.e. build obj and exe in exe directory. I
don't want
to put them in src directory because I specified src directory as
EXTRA_DIST.
I would also use fc command instead of diff command on Windows.

regards,
Hiroshi Inoue

In response to

Responses

Browse pgsql-odbc by date

  From Date Subject
Next Message Huong Dangminh 2016-07-06 13:08:09 Re: Default for "Level of rollback on errors"
Previous Message Inoue, Hiroshi 2016-07-06 12:26:26 Re: Default for "Level of rollback on errors"