Re: [patch][pgAdmin] Fix for pgadmin4-linux-qa #1651 failure

From: Rahul Shirsat <rahul(dot)shirsat(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [patch][pgAdmin] Fix for pgadmin4-linux-qa #1651 failure
Date: 2021-06-29 03:38:13
Message-ID: CAKtn9dOWXDQo9K_HuqGOmhq=AeVVzcE8d=J+8obRj014G=0jxQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

I feel gettext sometimes won't escape the characters as it should be.

I now tried to escape those using some utils.

Please find the updated patch.

On Mon, Jun 28, 2021 at 9:33 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Mon, Jun 28, 2021 at 4:57 PM Rahul Shirsat <
> rahul(dot)shirsat(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Please find the attached patch for fixation of jenkins failure.
>>
>
> That won't work - you can't include variables (or string building
> operations) in the first argument to gettext calls, as there won't be any
> way to extract a complete message into the catalogs. The way it's being
> done at the moment is correct (I don't know why it's failing, but it's the
> correct way to structure the gettext calls).
>
> --
> Dave Page
> Blog: https://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EDB: https://www.enterprisedb.com
>
>

--
*Rahul Shirsat*
Senior Software Engineer | EnterpriseDB Corporation.

Attachment Content-Type Size
jenkins_failure_fix_v2.patch application/octet-stream 3.8 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2021-06-29 08:26:07 Re: [patch][pgAdmin] Fix for pgadmin4-linux-qa #1651 failure
Previous Message Dave Page 2021-06-28 16:03:28 Re: [patch][pgAdmin] Fix for pgadmin4-linux-qa #1651 failure