FIxes to run dbbackup management command in Postgres for non-latin Windows.

[Dmitri-Sintsov]

Fix 'pg_dump too many command-line arguments' error in Windows / Postgres 9.4 (command-line arguments must precede database name).

Fix non-Latin bytes error result of pg_dump being displayed / logged as hex bytes instead of easily readable utf-8 string.

[ZuluPro]

Thank for your contribution,
Could you tell me what was the error before this patch ?

[Dmitri-Sintsov]

It was pg_dump 'too many command-line arguments' error in Windows / Postgres 9.4

[Merichbier]

It fixed the same problem (pg_dump 'too many command-line arguments' ), I'm using Windows 10, PostgreSQL 10.3 and Django 2.0.4

[WarToaster]

Also fixed my issue.
Win 10, Postgres 9.6.2, Django 1.11.1

[ghost]

Hi can you explain please why you rejected this pull request?

[ZuluPro]

@momar07005 The PR hasn't been rejected.

It's just the project doesn't have active mainteners
And to accept this we must ensure it works with functional/unit tests and doesn't break non-windows users.

[Dmitri-Sintsov]

Patched version works here locally in Ubuntu 18.04:

(ispdevenv) user@ispdev:~/work/ispdevenv/ispdev$ ./manage.py dbbackup -z
Backing Up Database: ispdevdb
Writing file to default-ispdev-2018-08-13-132053.psql.gz
(ispdevenv) user@ispdev:~/work/ispdevenv/ispdev$ ./manage.py mediabackup -z
Writing file to ispdev-2018-08-13-132111.tar.gz
(ispdevenv) user@ispdev:~/work/ispdevenv/ispdev$ 

Why does it break tests, I do not know.

[ZuluPro]

Merged with trust of community