Clarify in TracIni documentation how to set downloadable_paths for a multirepository environment

[Ryan J Ollos <ryano@…>]

I have a multirepository environment and was attempting to set the downloadlable_path option in the browser section. By trial and error, I arrived at the finding that I must prefix the repository name to these paths. It seems like it would be useful to add this to the documentation. Suggested addition in italics:

List of repository paths that can be downloaded. Leave the option empty if you want to disable all downloads, otherwise set it to a comma-separated list of authorized paths (those paths are glob patterns, i.e. "*" can be used as a wild card). In a multi-repository environment, the repository name must be prefixed to the paths (e.g. /repos1/trunk, /repos2/trunk) (since 0.10)

… or is this idea of prefixing the repository name to trac.ini options more obvious and I've overlooked it elsewhere in the documentation? In retrospect, I shouldn't be so surprised that I have to prefix the repository name, but the exact syntax for the prefixing wasn't obvious to me. At first I tried repos1:/trunk, before noticing that the path in the repository browser was /repos/trunk, which made the solution more obvious. It also wasn't obvious to me though that simply specifying /trunk, wouldn't make /trunk a downloadable path for the default repository.

[Ryan J Ollos <ryano@…>]

Minor refinement to clarify that the repository name only needs to be included in the path if it is not the default repository.

List of repository paths that can be downloaded. Leave the option empty if you want to disable all downloads, otherwise set it to a comma-separated list of authorized paths (those paths are glob patterns, i.e. "*" can be used as a wild card). In a multi-repository environment, the path must be qualified with the repository name if the path does not point to the default repository (e.g. /reposname/trunk) (since 0.10).

In testing this out, I found what might be considered a minor bug. If I have two repositories, repos and repos2, with repos being the default repository, then [source:/repos/trunk] and [source:/trunk] point to the same location in the default repository. However, dowloadable_paths = /repos/trunk does not result in a download link being added to the trunk directory in the default repository. Only downloadable_paths = /trunk works correctly.

[Christian Boos]

Replying to Ryan J Ollos <ryano@…>:

Minor refinement to clarify that the repository name only needs to be included in the path if it is not the default repository.

List of repository paths that can be downloaded. Leave the option empty if you want to disable all downloads, otherwise set it to a comma-separated list of authorized paths (those paths are glob patterns, i.e. "*" can be used as a wild card). In a multi-repository environment, the path must be qualified with the repository name if the path does not point to the default repository (e.g. /reposname/trunk) (since 0.10).

Patch please!

In testing this out, I found what might be considered a minor bug. If I have two repositories, repos and repos2, with repos being the default repository, then [source:/repos/trunk] and [source:/trunk] point to the same location in the default repository. However, dowloadable_paths = /repos/trunk does not result in a download link being added to the trunk directory in the default repository. Only downloadable_paths = /trunk works correctly.

Or you can add a warning about this in the docstring as well (much easier than adding the necessary code!):

Note that a simple prefix matching is performed on the paths, so aliases won't get automatically resolved.

[Ryan J Ollos <ryano@…>]

Patch against r11111 of the trunk.

[Ryan J Ollos <ryano@…>]

Replying to cboos:

Or you can add a warning about this in the docstring as well (much easier than adding the necessary code!):

I had spent a bit of time looking at the code for this, and it did look challenging indeed.

t10673-r11111-1.patch​ implements the proposed documentation changes, including your suggestions re prefix matching.

[Remy Blank]

Patch applied in [11506].