On Fri, 25 Jan 2008, Daniel Barkalow wrote: > On Fri, 25 Jan 2008, Jakub Narebski wrote: > >> >> So this config variable is actually about _rewriting_ URLs, rather than >> having multiple _alternate_ URLs to fetch from. IMHO either the >> documentation above should be (re)written better (there is time, as we >> are in feature freeze), or the example mentioned above should be added >> (but where?). > > Maybe we should have a "host.<name>.*" section that explains the > collection of configuration options as a group? Like: > > ---------- > host.<name>.*:: > These options provide a way to rewrite URLs when there is a > pattern of URLs with a common prefix which should be replaced > with a different prefix. For every 'alias' prefix given, any > URL git receives that starts with that prefix will be > rewritten to have the 'base' prefix instead. In cases where > some site serves a large number of repositories, and serves > them with multiple access methods, and some users need to use > different access methods, this feature allows people to > specify any of the equivalent URLs and have git automatically > rewrite the URL to the best alternative for the particular > user, even for a never-before-seen repository on the site. > > host.<name>.base:: > The base URL which should be used for this particular > host. > > host.<name>.alias:: > Additional base URLs which refer to this host. If a URL > matches this, any access to it will use the URL formed with > the corresponding base URL instead of the given URL. Good idea. IMVHO it reads better than previous version. Together with explanation and example usage in Documentation/urls.txt (or somewhere else) it would be enough and easy to understand. BTW. shouldn't it be rather host.<name>.rewritebase (a la mod_rewrite)? I think it is better name for this configuration variable. -- Jakub Narebski Poland - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
