Re: RM5576: Improve error messaging if the storage and logdirectories cannot be created

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: RM5576: Improve error messaging if the storage and logdirectories cannot be created
Дата
Msg-id CA+OCxoz22QTdgdmk9uGXonsM_rH5uWdZdpWHK-iiGgueWGJcxg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: RM5576: Improve error messaging if the storage and logdirectories cannot be created  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: RM5576: Improve error messaging if the storage and logdirectories cannot be created
Список pgadmin-hackers


On Tue, Jun 2, 2020 at 9:20 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave

On Tue, Jun 2, 2020 at 1:41 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

Urgh, didn't realise pwd wasn't available on Windows :-(. What about just using getpass.getuser() everywhere? According to the docs it falls back to using pwd where that's supported anyway: https://docs.python.org/3.8/library/getpass.html

   Sure we can use the getpass.getuser() everywhere. 

Are you working on that, or do you want me to update it and commit?

 

On Tue, Jun 2, 2020 at 8:00 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Dave

I have tested the logic on Mac OSX but not on Windows. The "pwd" module is not available on windows hence Jenkins build is failing. 
Attached is the patch to fix for Windows OS. I have used the "getpass" module for Windows.

Please review it. If looks good I'll commit the code. 

On Tue, Jun 2, 2020 at 11:12 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Thanks, patch applied.

On Mon, Jun 1, 2020 at 9:20 PM Dave Page <dpage@pgadmin.org> wrote:
Patch for $SUBJECT attached. Sample output:

python web/pgAdmin4.py
ERROR  : Failed to create the directory /var/lib/pgadmin:
           [Errno 13] Permission denied: '/var/lib/pgadmin'
HINT :   Create the directory /var/lib/pgadmin, ensure it is writeable by
         'dpage', and try again, or, create a config_local.py file
         and override the SQLITE_PATH setting per
         https://www.pgadmin.org/docs/pgadmin4/4.22/config_py.html


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

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


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246


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

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


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246


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

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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: RM5576: Improve error messaging if the storage and logdirectories cannot be created
Следующее
От: Akshay Joshi
Дата:
Сообщение: Re: RM5576: Improve error messaging if the storage and logdirectories cannot be created