33 lines
1.1 KiB
Markdown
33 lines
1.1 KiB
Markdown
As described on [Tails issue #5879](https://labs.riseup.net/code/issues/5879),
|
||
cgit wants URLs like:
|
||
|
||
https://git-tails.immerda.ch/tails/commit/wiki/src/todo/fix_localized_iceweasel_search_engine.mdwn?id=c682a361418bef7bce040ea2905691d54be192ab
|
||
|
||
... while ikiwiki generates links like:
|
||
|
||
https://git-tails.immerda.ch/tails/commit/wiki/src/todo%2Ffix_localized_iceweasel_search_engine.mdwn?id=c682a361418bef7bce040ea2905691d54be192ab
|
||
|
||
on the recentchanges page.
|
||
|
||
Would it make sense to add an option to skip URL-encoding when doing
|
||
the substitution of `\[[file]]` in `diffurl` and `historyurl`?
|
||
|
||
--[[intrigeri]]
|
||
|
||
> In `91b477c` I made a similar change to how the cvs backend encodes
|
||
> `diffurl`. Perhaps it applies to all VCS backends? --[[schmonz]]
|
||
|
||
>> Thanks a lot! I've not looked at it yet, but will try to do so
|
||
>> soonish, as this bug is a real problem for Tails when welcoming potential
|
||
>> new contributors. --[[intrigeri]]
|
||
|
||
>>> My `fix_diffurl_vs._cgit` branch now implements schmonz' change
|
||
>>> for the Git plugin. Please review and merge if you like
|
||
>>> it. --[[intrigeri]]
|
||
|
||
>>>> Ping? --[[intrigeri]]
|
||
|
||
[[wishlist]]
|
||
|
||
[[!tag patch]]
|