From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode |
Date: | 2020-05-28 12:08:07 |
Message-ID: | CANxoLDez2TzUB3LUm6Spxm8gkK5W3PZ65u6hX21Zw39KFsJ4og@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Aditya
When I run setup.py and specify the *--sqlite-path* parameter like below:
python setup.py --sqlite-path
"/Users/akshayjoshi/Downloads/pgadmin4-dev-env.db" --dump-servers
output.json
I got the following error:
PermissionError: [Errno 13] Permission denied: '/var/lib/pgadmin'
Please fix it and resend the patch.
On Thu, May 28, 2020 at 2:39 PM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
> Hi Hackers,
>
> Attached is the patch to:
> 1) Fix an issue when dumping servers from a desktop pgAdmin app by
> providing an option --sqlite-path. This will allow the user to dump a
> specific config db file.
> 2) While there is an option for --sqlite-path now, the other paths for
> session, storage, etc. were failing with permission errors. Code is changed
> to not to bother those directories when dumping or loading servers.
> 3) Improved --help for setup.py.
> 4) Docs updated.
>
> Please review.
>
> --
> Thanks and Regards,
> Aditya Toshniwal
> pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
> "Don't Complain about Heat, Plant a TREE"
>
--
*Thanks & Regards*
*Akshay Joshi*
*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-05-28 12:42:21 | pgAdmin 4 commit: Ensure that the query tool panel gets closed when cli |
Previous Message | Yogesh Mahajan | 2020-05-28 11:46:07 | [pgAdmin4][Patch] - RM 5539 - /usr/share/pgadmin4/web/pgadmin/utils/preferences.py contains 7 references to current_app.logger.exeception(e) and 1 to current_app.logger.exception(e) |