#8502 closed defect (fixed)
[Patch] trac-admin upgrade fails on postgresql databases that have capital letters in the schema
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | normal | Milestone: | 0.11.6 |
| Component: | database backend | Version: | 0.11.5 |
| Severity: | major | Keywords: | patch, trac-admin, postgresql |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
This bug appears to be present in the trunk as well. Essentially if you have a schema like "TracName" in your trac connection string, then trac-admin <env> upgrade fails with an error about not finding the schema.
The solution is to wrap the schema name in double quotes.
I also found that the message from trac admin about bypassing backups (which also worked) was never being displayed because it was looking for the word "backup" in a string that contained the word "Backup".
Attached is a patch that should fix the two minor bugs.
Thanks for all the great work and dedication, trac is wonderful and makes my life better.
Attachments (2)
Change History (6)
by , 15 years ago
| Attachment: | postgres_backup_fix.diff added |
|---|
by , 15 years ago
| Attachment: | postgres_backup_fix.2.diff added |
|---|
Patch to fix two bugs about trac-admin upgrade failing on postgres (Without extra debugging info in teh error message)
comment:2 by , 15 years ago
| Owner: | set to |
|---|
Problem reproduced and fix confirmed to work - will be applied shortly.
comment:3 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Patch applied in [8382]. Thanks!
comment:4 by , 9 years ago
| Keywords: | postgresql added; postgres removed |
|---|
Patch to fix two bugs about trac-admin upgrade failing on postgres