Edgewall Software
Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#8759 closed defect (fixed)

Upgrade Failed - PostgreSQL backend

Reported by: brolewis <lewis.franklin@…> Owned by: lewis.franklin@…
Priority: normal Milestone: 0.11.6
Component: database backend Version: 0.11.5
Severity: normal Keywords: postgresql
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When attempting to perform an upgrade on a trac instance, I got the follow error message:

Command failed: Backup attempt failed (pg_dump: illegal option -- d
Try "pg_dump --help" for more information.)

I reviewed the code and found the issue in postgres_backend.py, line 117. Currently, the code reads:

args = [self.pg_dump_path, '-C', '-d', '-x', '-Z', '8']

However, in PostgreSQL 8.4, the "-d" flag was removed from pg_dump:

Remove -d and -D options from pg_dump and pg_dumpall (Tom)

These options were too frequently confused with the option to 
select a database name in other PostgreSQL client applications. 
The functionality is still available, but you must now spell out 
the long option name --inserts or --column-inserts.

Since the "—inserts" option remains, the args list should be instead:

args = [self.pg_dump_path, '-C', '--inserts', '-x', '-Z', '8']

Attachments (0)

Change History (3)

comment:1 by Remy Blank, 15 years ago

Owner: set to Remy Blank

Thanks for the heads-up!

comment:2 by Remy Blank, 15 years ago

Resolution: fixed
Status: newclosed

Fixed in [8682], thanks!

comment:3 by Remy Blank, 15 years ago

Owner: changed from Remy Blank to lewis.franklin@…

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain lewis.franklin@….
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from lewis.franklin@… to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.