Re: [Patch] Windows installer: allow manually setting WIXDIR as appropriate.

From: Nikolai Zhubr <n-a-zhubr(at)yandex(dot)ru>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: [Patch] Windows installer: allow manually setting WIXDIR as appropriate.
Date: 2015-10-02 21:10:30
Message-ID: 560EF2C6.9040402@yandex.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi again,

there is some problem in my previous patch, it did not actually work,
here goes a better one:

--- pkg/win32/Make.bat.orig Fri Sep 25 13:20:24 2015
+++ pkg/win32/Make.bat Fri Oct 02 21:55:27 2015
@@ -9,7 +9,9 @@
SET APPKEYWORDS=PostgreSQL, pgAdmin
SET APPCOMMENTS=PostgreSQL Tools
SET APPDESCRIPTION=Management and administration tools for the
PostgreSQL DBMS
+IF NOT (%WIXDIR%)==() GOTO DONE_WIXDIR
SET WIXDIR="C:\Program Files (x86)\Windows Installer XML v3\bin"
+:DONE_WIXDIR

SET BUILDTREE="../.."

Thank you,
Nikolai

02.10.2015 22:33, I wrote:
> Hi all,
>
> How about allowing wix3 to be located in a different path, pointed to by
> setting WIXDIR environment variable _manually_ before building.
>
> The following patch allows doing so, while fully retaining previous
> behaviour in case this variable was not manually set.
>
> (E.g. when building in a 32bit windows, the "Program Files (x86)" is not
> good enough, and I'd rather not modify the scripts for just setting a
> correct path)
>
> --- pkg/win32/Make.bat.orig Fri Sep 25 13:20:24 2015
> +++ pkg/win32/Make.bat Fri Oct 02 21:55:27 2015
> @@ -9,7 +9,9 @@
> SET APPKEYWORDS=PostgreSQL, pgAdmin
> SET APPCOMMENTS=PostgreSQL Tools
> SET APPDESCRIPTION=Management and administration tools for the
> PostgreSQL DBMS
> +IF NOT "%WIXDIR%"=="" GOTO DONE_WIXDIR
> SET WIXDIR="C:\Program Files (x86)\Windows Installer XML v3\bin"
> +:DONE_WIXDIR
>
> SET BUILDTREE="../.."
>
>
> Thank you,
> Nikolai
>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Nikolai Zhubr 2015-10-02 23:44:50 Re: Building pgadmin on windows - instructions?
Previous Message Nikolai Zhubr 2015-10-02 19:33:19 [Patch] Windows installer: allow manually setting WIXDIR as appropriate.