git.oblomov.eu Git - ikiwiki/blob - doc/todo/support_linking_to_cgit.mdwn

   1 As described on [Tails issue #5879](https://labs.riseup.net/code/issues/5879),
   2 cgit wants URLs like:
   3
   4         https://git-tails.immerda.ch/tails/commit/wiki/src/todo/fix_localized_iceweasel_search_engine.mdwn?id=c682a361418bef7bce040ea2905691d54be192ab
   5
   6 ... while ikiwiki generates links like:
   7
   8         https://git-tails.immerda.ch/tails/commit/wiki/src/todo%2Ffix_localized_iceweasel_search_engine.mdwn?id=c682a361418bef7bce040ea2905691d54be192ab
   9
  10 on the recentchanges page.
  11
  12 Would it make sense to add an option to skip URL-encoding when doing
  13 the substitution of `\[[file]]` in `diffurl` and `historyurl`?
  14
  15 --[[intrigeri]]
  16
  17 > In `91b477c` I made a similar change to how the cvs backend encodes
  18 > `diffurl`. Perhaps it applies to all VCS backends? --[[schmonz]]
  19
  20 >> Thanks a lot! I've not looked at it yet, but will try to do so
  21 >> soonish, as this bug is a real problem for Tails when welcoming potential
  22 >> new contributors. --[[intrigeri]]
  23
  24 >>> My `fix_diffurl_vs._cgit` branch now implements schmonz' change
  25 >>> for the Git plugin. Please review and merge if you like
  26 >>> it. --[[intrigeri]]
  27
  28 >>>> Ping? --[[intrigeri]]
  29
  30 >>>> Normally, my tendency towards pedantic preservation of existing APIs
  31 >>>> would make me implement a new token \[[file_less_escaped]] or
  32 >>>> something, analogous to `%s` vs. `%S` in [[shortcuts]]; but
  33 >>>> gitweb and cgit are both OK with `/` not being escaped, whereas
  34 >>>> only gitweb allows it to be escaped as `%2F`, so I think your
  35 >>>> solution is fine. Also, it would need a much better name than
  36 >>>> \[[file_less_escaped]], and I can't think of one.
  37 >>>>
  38 >>>> I don't have commit access to ikiwiki.info, but if I did,
  39 >>>> [[I'd merge this|/users/smcv/yesplease]]. --[[smcv]]
  40
  41 [[wishlist]]
  42
  43 [[!tag patch]]