Opened 14 years ago
Closed 14 years ago
#6238 closed defect (fixed)
[patch] upgrade doesn't handle absence of 'watchlist_version' from db with trac-0.12
| Reported by: | anonymous | Owned by: | Martin Scharrer |
|---|---|---|---|
| Priority: | normal | Component: | WatchlistPlugin |
| Severity: | normal | Keywords: | upgrade |
| Cc: | Trac Release: | 0.12 |
Description
Seemingly the 0.11 version of WatchlistPlugin is incompatible with trac 0.12 and a new branch has to be created.
The symptoms are: after installing (works only with 0.11 from source tar ball), the message "database needs to upgraded" appears and doesn't go away anymore. Doing "trac-admin /path/to/trac upgrade" does not help.
Attachments (3)
Change History (10)
comment:1 Changed 14 years ago by
| Status: | new → assigned |
|---|
comment:2 Changed 14 years ago by
I'm not anonymouse, but I have the same problem. I've attached patches to fix the obvious bugs that cause the problem. I'm using revision 7110 of WatchList and revision 9163 of trac trunk.
Changed 14 years ago by
| Attachment: | 0002-fixes-needs_upgrade-bug.patch added |
|---|
Changed 14 years ago by
| Attachment: | 0003-fixes-bug-with-exception-message-formatting.patch added |
|---|
Changed 14 years ago by
| Attachment: | 0004-Fixes-bug-with-version-creation.patch added |
|---|
comment:3 Changed 14 years ago by
A final patch added. After restarting trac I realized that the former two patches didn't completely resolve the issue.
comment:4 Changed 14 years ago by
| Keywords: | upgrade added |
|---|---|
| Summary: | Does not work with 0.12 → [patch] database upgrade fails with trac-0.12 |
comment:5 Changed 14 years ago by
| Summary: | [patch] database upgrade fails with trac-0.12 → [patch] upgrade doesn't handle absence of 'watchlist_version' from db with trac-0.12 |
|---|
#6602 was closed as a duplicate of this one, already carried a reference
however, it has better description of the underlying problem:
"doesn't handle installations where the database doesn't contain a 'watchlist_version' gracefully"
comment:6 Changed 14 years ago by
Thanks a lot for the patches. I appreciate the contribution greatly. I created a Trac 0.12 branch and added the patches in [7976], [7977] and [7978], respectively.
I don't have Trac 0.12 installed at the moment and I'm too busy right now for any testing. Please verify if it's working on you installations so that I can close this ticket. Thanks!
comment:7 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
I tested it now a little with a Trac 0.12 installation and can't find any errors. I'm closing this ticket now, please reopen it if there are any issues remaining.
Dear anonymous, please have a look at BugReporting to learn how to report bugs. I need at least the revision number of WatchlistPlugin you installed. Your database backend would also be a important information. The error you describe looks more like #6166 (which is fixed) and not a 0.12 incompatibility. I'm also pretty sure that if this is a different bug then it could be fixed in a 0.11 compatible way which would avoid the need of a new branch.