Re: Patch for pgAdmin Installer web folder upgrade mode.

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Paresh More <paresh(dot)more(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Hamid Quddus <hamid(dot)quddus(at)enterprisedb(dot)com>, Sandeep Thakkar <sandeep(dot)thakkar(at)enterprisedb(dot)com>, Karen Blatchley <karen(dot)blatchley(at)enterprisedb(dot)com>
Subject: Re: Patch for pgAdmin Installer web folder upgrade mode.
Date: 2017-04-10 11:05:36
Message-ID: CA+OCxozBhk1OTFTUyj8BVmNAppSWpqPa8UKBmCT9k5x9zj_efg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks - looks good, applied.

On Mon, Apr 10, 2017 at 10:53 AM, Paresh More <paresh(dot)more(at)enterprisedb(dot)com>
wrote:

> Hello Dave
>
> I have figured out the issue.
>
>
> For Innosetup we have two downloads available.
>
> 1) Normal (innosetup-5.5.9.exe)
> 2) Unicode (innosetup-5.5.9-unicode.exe)
>
>
> The error which mentioned comes only for unicode , for normal its working
> fine. I have made changes to installer.iss.in which works now fine for
> both the versions. (Normal and Unicode)
>
>
> Attached is the patch.
>
>
> end; <== Added ; and its working fine
>
> until not FindNext(FindRec);
>
>
>
>
>
>
> On Mon, Apr 10, 2017 at 3:01 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Attached.
>>
>> On Mon, Apr 10, 2017 at 10:26 AM, Paresh More <
>> paresh(dot)more(at)enterprisedb(dot)com> wrote:
>>
>>> Hello Dave,
>>>
>>> Can you please send me C:\Users\dpage\Documents\pg
>>> admin4\pkg\win32\installer.iss file
>>>
>>> On Mon, Apr 10, 2017 at 2:30 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>> Hi
>>>>
>>>> On Sun, Apr 9, 2017 at 12:56 PM, Paresh More <
>>>> paresh(dot)more(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hello Dave,
>>>>>
>>>>> This is regarding issue # 2187 ( Cleaning up web/* *except*
>>>>> config_local.py) through installer. https://redmine.pos
>>>>> tgresql.org/issues/2187
>>>>>
>>>>>
>>>>> Now Inno installer would clean up web folder except config_local.py in
>>>>> upgrade mode and copy latest web folder in the installed location.
>>>>>
>>>>> Its working fine now.
>>>>>
>>>>> Attached is the patch for same.
>>>>>
>>>>
>>>> I get the following error when building:
>>>>
>>>> Preparing for creation of windows installer...
>>>> 1 file(s) copied.
>>>> Creating windows installer... using INNO tool
>>>> Error on line 111 in C:\Users\dpage\Documents\pgadmin4\pkg\win32\installer.iss:
>>>> Column 9:
>>>> Semicolon (';') expected.
>>>> Compile aborted.
>>>>
>>>> --
>>>> Dave Page
>>>> Blog: http://pgsnake.blogspot.com
>>>> Twitter: @pgsnake
>>>>
>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>
>>>
>>>
>>> --
>>>
>>> Thanks & Regards
>>>
>>> *Paresh More*
>>>
>>> [image: NEW-EDB-logo-4c]
>>>
>>> Pune, India.
>>> Cell : +919922000564 <+91%2099220%2000564> | www.enterprisedb.com
>>>
>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
>
> --
>
> Thanks & Regards
>
> *Paresh More*
>
> [image: NEW-EDB-logo-4c]
>
> Pune, India.
> Cell : +919922000564 <+91%2099220%2000564> | www.enterprisedb.com
>

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

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

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2017-04-10 13:11:53 [pgAdmin4][PATCH] To fix the issue in browser tree
Previous Message Dave Page 2017-04-10 11:05:14 pgAdmin 4 commit: Ensure the web/ directory is cleared before upgrading