On Fri, Nov 14, 2008 at 7:15 PM, Jakub Narebski <jnareb@xxxxxxxxx> wrote: > On Thu, 13 Nov 2008, Giuseppe Bilotta wrote: > >> With this feature enabled, remotes are retrieved (and displayed) >> when getting (and displaying) the heads list. > > I think it would be good idea to add in commit message idea _why_ > such feature would be useful, for example > > This is useful if you want to use git-instaweb to examine the state > of repository, influding remote-tracking branches, or a repository > is fork of other repository, and remote-tracking branches are used > to see what commits this fork has in addition to those from forked > (main) repository. > > Or something like that. Ah yes, many commit messages in this patchset are way too terse. A side effect of this being something like the first patchset I ever prepared. I'll rework them to something more sensible. > It would be also in my opinion a good idea to modify git-instaweb.sh > (I guess better in separate commit) to make it make use of this new > feature... unless it does it already, doesn't it? It doesn't, but it's something I have considered. I'll work on it (on a separate patch) >> + # Make gitweb show remotes too in the heads list > > I'm not native engish speaker, but shouldn't instead of "remotes too" > be "also remotes" or "remotes also"? No idea, I guess we'll wait for a native english speaker opinion 8-D -- Giuseppe "Oblomov" Bilotta -- 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
