Ticket #7974 (closed defect: duplicate)
Opened 3 years ago
Last modified 3 years ago
UnicodeDecodeError showing transplanted revision
| Reported by: | wbruna@… | Owned by: | cboos |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | plugin/mercurial | Version: | 0.11 |
| Severity: | normal | Keywords: | |
| Cc: | |||
| Release Notes: | |||
| API Changes: | |||
Description
Transplanted changesets include an extra field 'transplanted_source', containing the original changeset hash (as a 20-byte buffer). The content is returned as-is by the plugin, so it triggers an UnicodeDecodeError when rendering.
I'd suggest simply discarding unknown extra fields. IMHO, there isn't a good way to display these fields generically, since they don't carry type information. And they're usually interesting only to extensions, since they can't be created with a regular commit.
For transplant_source in particular, it would be nice showing a link to the transplanted revision - or simply its ID, if the original revision isn't available (could happen on transplants across branches).
Attachments
Change History
comment:1 in reply to: ↑ description ; follow-up: ↓ 2 Changed 3 years ago by cboos
- Resolution set to duplicate
- Status changed from new to closed
comment:2 in reply to: ↑ 1 ; follow-ups: ↓ 3 ↓ 4 Changed 3 years ago by wbruna@…
comment:3 in reply to: ↑ 2 Changed 3 years ago by wbruna@…
comment:4 in reply to: ↑ 2 Changed 3 years ago by cboos
Replying to wbruna@…:
... it's good to keep this ticket around, since Debian Lenny will ship version 0.11.
Thanks, this prompted me to update the TracMercurial page with a reference to the recent stable revisions (r7744 / r7757).
Replying to wbruna@…:
It already should do so, please upgrade :-) - see #7471.