Re: [Patch] Windows installer: allow manually setting WIXDIR as appropriate. - Mailing list pgadmin-hackers

From Nikolai Zhubr
Subject Re: [Patch] Windows installer: allow manually setting WIXDIR as appropriate.
Date
Msg-id 56112A4B.6080202@yandex.ru
Whole thread Raw
In response to Re: [Patch] Windows installer: allow manually setting WIXDIR as appropriate.  ("J.F. Oster" <jinfroster@mail.ru>)
List pgadmin-hackers
Hi all,
04.10.2015 7:55, J.F. Oster wrote:
> Hello,
>
> +1
> I proposed the same once but had no feedback.

I'm actually surprised by the very low feedback on the list altogether.

> Nikolai, why doesn't the first version work for you? I have it done similar:

Because if the variable itself contains quotes (and typically it does,
to deal with spaces in paths) then AFAICS command interpreter gets
confused when it sees yet more surrounding quotes in a comparison
operator in a line like you listed below. I'm no expert in cmd.exe
behaviour, but at least on windows xp it caused interpreter errors, so I
just used a simple workaround and it then ran fine.

> if NOT "%WIXDIR%"=="" GOTO WIXDIR_SET
> SET WIXDIR="C:\Program Files (x86)\Windows Installer XML v3\bin"
> :WIXDIR_SET
>
>


Thank you,
Nikolai


pgadmin-hackers by date:

Previous
From: "J.F. Oster"
Date:
Subject: Re: [Patch] Windows installer: allow manually setting WIXDIR as appropriate.
Next
From: Nikolai Zhubr
Date:
Subject: [PATCH] Fix EVT_STC_PAINTED recursion issue (Was: Re: Repaint lockup in ctlSQLBox::OnPositionStc on windows)