Re: Incorporating gitweb-xmms2 features in trunk gitweb

Sham Chukoury <eleusis[@]xmms[.]org> wrote:

Jakub Narebski wrote:

quoted

Now (from git 1.4.0) that gitweb is incorporated in main git.git 
repository, and patches to it are accepted by Junio on git mailing 
list, I have taken to adding new features to gitweb.

I've planned on adding gitweb-xmms2 features (using separate topic 
branches for each feature, like snapshots, hightlighting, committags).
Perhaps we could coordinate efforts?

Coordination sounds great. What do you have in mind? :)

Actually it seems that I wouldn't have much time (at least in near future)
for work on gitweb. The main thing stopping me is that gitweb needs
refactoring before adding (or re-adding) new features, as it is written
partially at least in copy'n'paste style, AND waiting for Git.pm Perly git
interface (I'd like to have perl-only Git.pm and use it in gitweb, but 
pasky who develops it uses XS for speed).
 
An example of such refactoring is your separation of navbar generation
code. Similar work was done by Sven Verdoolaege when introducing in-gitweb
snapshot code in
  "gitweb and tar snapshots"
  http://marc.theaimsgroup.com/?l=git&m=111909432415478&w=2
  http://www.liacs.nl/~sverdool/gitweb.cgi?p=gitweb.git;a=summary
  http://www.liacs.nl/~sverdool/gitweb.git#master
In my opinion your navbar code is too tied up with current navbar, while
sverdool is a little too perl-hackerish (e.g. there is only one instance,
i.e. root commit, where "top" part of navbar is not default, i.e. it
lacks "commitdiff" (to parent)).

quoted

For now from gitweb-xmms2.git I have only cherry-picked "Make CSS 
readable" commit 561262030d58a6325f500b36d836dbe02a5abc68.

Bear in mind that, since 'forking' (for lack of a better word) from v264, I 
haven't merged any changes from the 'official' gitweb tree.. Last time I 
checked, there's at least one changeset I reimplemented differently (perhaps 
wrongly). I need to go back and see what needs to be fixed properly..

gitweb-xmms2 merged quite cleanly with 1.4.0. It was later changes in next
(among  others adding optional "blame" view for blobs, using old navbar 
generation) that makes automatic merge fail. 

Going back to the point on coordination - I currently track bugs and feature 
requests for gitweb-xmms2 using the xmms2 bug tracker at 
http://bugs.xmms2.xmms.se (You'll need to select the 'Misc - gitweb' project 
from the dropdown at the top right of the page to see the relevant reports)

Check my two threads on gitweb wishlist features, and plans on refactoring
on git mailing list:
  "[RFC] gitweb wishlist and TODO list"
  Message-ID: [ref]
  http://permalink.gmane.org/gmane.comp.version-control.git/22213
  http://marc.theaimsgroup.com/?l=git&m=115082279425118&w=2
and
  "gitweb refactoring"
  Message-ID: [ref]
  http://permalink.gmane.org/gmane.comp.version-control.git/22345
  http://marc.theaimsgroup.com/?l=git&m=115099031220954&w=2


P.S. Now that gitweb is included in git.git (from git version 1.4.0), you should
have easier to have your patches accepted into gitweb trunk.

-- 
Jakub Narebski
ShadeHawk on #git