Re: [Patch]: Installer: pgAdmin4 should not install in specialcharacter path - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [Patch]: Installer: pgAdmin4 should not install in specialcharacter path
Date
Msg-id CA+OCxoyQ_tzKwU0PzPY1Pi1a6CbgKFPWR6_Nj_MKr=Pgb=PntA@mail.gmail.com
Whole thread Raw
In response to [Patch]: Installer: pgAdmin4 should not install in special character path  (Paresh More <paresh.more@enterprisedb.com>)
List pgadmin-hackers
Thanks, applied with minor changes to the default paths (to make them match my machine, which uses all default paths).

On Fri, Nov 17, 2017 at 12:17 PM, Paresh More <paresh.more@enterprisedb.com> wrote:
Hello Team,

I have made three changes for installer generation scripts.

1) Modified Make-MinGW.bat to support yarn and nodejs validation in the script and also QT 5.9.1 support.

2) Bug fix for RM1365.
Currently string datatypes used in Inno tool does not support regular expression, so added a user function which will do path validation and it would be work fine for inno unicode setup.

3) Restructured  installer script so that it should support unicode inno and for RM1365 as well i.e Inno Setup Installer(unicode) also updated README file for the same.


Please note - Installer path validation is invoked two times

1) Inno tool internally validate's path which user program do not have access to it , so section [MESSAGE] error string message will be used to show error message string. 

2) From the code section, so section [CustomMessages] error message will be used.

For both (1) and (2) common error message will be used.


Attached is the patch 


--

Thanks & Regards 

Paresh More

NEW-EDB-logo-4c

Pune, India.




--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Prevent the Windows installer accepting pathscontain
Next
From: Murtuza Zabuawala
Date:
Subject: Re: Build failed in Jenkins: pgadmin4-master-python26 #510