Trac links to branch-specific changesets don't work

[Andrew Schulman]

Our trac-bzr install points to a plain directory which contains several shared repositories. In each of these shared repositories are a number of related branches. This is the same as in #264734, although I'm not sure this bug is related to that one.

In wiki pages and bug reports, trac links to changesets don't work. To refer to a changeset in trunk, I should be able to write e.g. [200/trunk] or changeset:200/trunk . But when I try either of these, the resulting text isn't a link. It's grey, and when I hover my mouse over it, the tooltip says "No changeset 200 in repository".

I've also tried using just r200 or changeset:200, but those don't work either. That's as expected, since the branch name would need to be specified.

Thanks,
Andrew.

[Andrew Schulman (andrex)]

Forgot to mention that my trac-bzr is trunk r85.

[Martin von Gagern (gagern)]

The notation to use would be "changeset:trunk,200" as described in the README file. This denotes "everything that belongs to revno 200 in the trunk branch". "changeset:200/trunk" would have been "everything in the trunk subdirectory of revision 200 of a branch located at the root of the repository", and as you don't have such a branch, it won't work and isn't expected to.

[Andrew Schulman (andrex)]

OK, you're right. changeset:trunk,200 works. However, [trunk,200] doesn't. Should it?

[Andrew Schulman (andrex)]

I'm sorry, I see that this is in the README.

The README seems to say that the bracket changeset syntax doesn't work because of a problem in Trac, rather than trac-bzr. So I'll close this again.

Thanks,
Andrew.

[Martin von Gagern (gagern)]

Yes, there is a regular expression hardcoded into trac as to what can be a changeset. In trac 0.11-stable it's this:
CHANGESET_ID = r"(?:\d+|[a-fA-F\d]{8,})" # only "long enough" hexa ids
There is no way for trac-bzr revids to follow such a scheme and still stay readable to humans.

Maybe we could provide our own IWikiSyntaxProvider in the future, but that would need some drafting.