Re: [pgAdmin][RM4590] Schema Backup fails with names that requires quoting

Akshay Joshi <[email protected]>
Newsgroups gmane.comp.db.postgresql.pgadmin.devel
Message-ID <CANxoLDdqNE_5VGmHah55LchdQXjL88OqjJCasCQ57SdpmeBmQA@mail.gmail.com>
Hi Aditya

I have tried your patch on MAC. Backup job is completed successfully but
got the following message:
pg_dump: last built-in OID is 16383
pg_dump: no matching schemas were found

Please verify it once again and send the modified patch.

On Tue, Oct 8, 2019 at 5:46 PM Aditya Toshniwal <
[email protected]> wrote:

> Hi Hackers,
>
> Turned out to be an issue only on Mac. Backup works fine for other OS.
> Attached is the patch to fix the issue on Mac. Code is changed to add
> extra pairs of quotes wherever required on MacOS.
>
> Kindly review.
>
> On Thu, Oct 3, 2019 at 3:53 PM Aditya Toshniwal <
> [email protected]> wrote:
>
>> Hi Hackers,
>>
>> Attached is the patch to fix the schema backup with names that requires
>> double quotes.
>> Command line has special meaning for double quotes and so the command is
>> run without the double quotes if applied on objects. Code is changed to add
>> extra pairs of quotes to save our object quotes.
>>
>> Also fixed a test case which was failing if certain paths had spaces in
>> between.
>>
>> Kindly review.
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> Sr. Software Engineer | EnterpriseDB India | Pune
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> Thanks and Regards,
> Aditya Toshniwal
> 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*
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.