Re: [PATCH] Change slash to backslash in make.bat

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: "zhangjie2(at)fujitsu(dot)com" <zhangjie2(at)fujitsu(dot)com>
Cc: "pgadmin-hackers(at)lists(dot)postgresql(dot)org" <pgadmin-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] Change slash to backslash in make.bat
Date: 2022-02-18 11:33:21
Message-ID: CAM9w-_mDGd4+tZUY7o6+pf55XWQp_EgXHf-kpsMp5ftU-vG6dA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Zhang,

Thank you for the contribution.
Backslashes can have different meanings sometimes and changing the build
scripts which are working since long are not needed.
So we can skip this patch.

On Wed, Feb 16, 2022 at 9:01 AM zhangjie2(at)fujitsu(dot)com <zhangjie2(at)fujitsu(dot)com>
wrote:

> Hi, all
>
> In windows, backslash should be used in the path.
> In Linux, slash should be used in path.
> But in pgadmin4\make.bat, slash is used in the path.
>
> ------------------------------------------------------------------------------------------------------------------------------------------------
> FOR /f "tokens=1 DELims=." %%G IN ('%PGADMIN_PYTHON_DIR%/python.exe -c
> "print('%APP_NAME%'.lower().replace(' ', ''))"') DO SET APP_SHORTNAME=%%G
>
> FOR /f "tokens=1 DELims=." %%G IN ('%PGADMIN_PYTHON_DIR%/python.exe -c
> "import sys; print(sys.version.split(' ')[0])"') DO SET PYTHON_MAJOR=%%G
> FOR /f "tokens=2 DELims=." %%G IN ('%PGADMIN_PYTHON_DIR%/python.exe -c
> "import sys; print(sys.version.split(' ')[0])"') DO SET PYTHON_MINOR=%%G
> FOR /f "tokens=3 DELims=." %%G IN ('%PGADMIN_PYTHON_DIR%/python.exe -c
> "import sys; print(sys.version.split(' ')[0])"') DO SET PYTHON_REVISION=%%G
>
> --------------------------------------------------------------------------------------------------------------------------------------------------
> Although the slash does not affect the compilation, I think it is better
> to modify it.
>
> Here is a patch for make.bat.
> Please review.
>
> Best Regards!
> Zhang Jie
>

--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | *edbpostgres.com*
<http://edbpostgres.com>
"Don't Complain about Heat, Plant a TREE"

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2022-02-19 04:59:08 pgAdmin 4 commit: Update the release date in the release notes. Fixes #
Previous Message Akshay Joshi 2022-02-18 09:09:59 Re: [pgAdmin4][Patch] - Cloud Deployment Fixes