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: | Whole Thread | Raw Message | 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
>
>
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. |