X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/f8555c9ae1868e0f8ec4861826965ebd003740b8..76f925bc8e4f8f4059be13e469e6f63d7bbc3a2f:/doc/todo/mirrorlist_with_per-mirror_usedirs_settings.mdwn diff --git a/doc/todo/mirrorlist_with_per-mirror_usedirs_settings.mdwn b/doc/todo/mirrorlist_with_per-mirror_usedirs_settings.mdwn index d40be0b08..c164cf54a 100644 --- a/doc/todo/mirrorlist_with_per-mirror_usedirs_settings.mdwn +++ b/doc/todo/mirrorlist_with_per-mirror_usedirs_settings.mdwn @@ -27,6 +27,8 @@ and decided this time it was really needed to implement this feature. >> (I'm not an ikiwiki committer, opinions may vary.) >> +>>> In my opinion, you're an ikiwiki committer! --[[Joey]] +>> >> This would be easier to review if there weren't a million merges from >> master; perhaps either leave a branch as-is, or rebase it, or merge >> only at "significant" times like after a release? @@ -59,3 +61,25 @@ and decided this time it was really needed to implement this feature. >>> the future, though. >>> >>> --[[intrigeri]] + +>>>> FWIW, I don't care about merge commits etc because I review +>>>> `git diff ...intrigeri/mirrorlist` -- and if I want to dig deeper +>>>> into the why of some code, I'll probably checkout the branch and +>>>> use git blame. +>>>> +>>>> I agree with what smcv said, my other concern though is that +>>>> this is such an edge case, that supporting it just adds clutter. +>>>> Have to wonder if it wouldn't perhaps be better to do something +>>>> using the goto plugin and cgiurl, so that the mirror doesn't have +>>>> to know about the configuration of the other mirror. --[[Joey]] + +>>>>> I have implemented something using the cgi + goto in my (history +>>>>> rewrite warning) mirrorlist branch. Please review, please pull. +>>>>> --[[intrigeri]] + +>>>>> concerning goto/cgiurl, what about having that as the default in +>>>>> mirrorlist, but keeping ``nousedirs|file:///home/intrigeri/wiki`` and +>>>>> ``usedirs|http://example.com/wiki`` valid for cgi-less cases? +>>>>> that would keep typical installation with a clutter-less configuration, +>>>>> and support more individual setups too. +>>>>> --[[chrysn]]