Frustrated gitweb and cgit I started experimenting with building a git repository viewer for Drupal. I got something fairly functional together and I demoed it at our sprint wrapup call for the end of Sprint 6 and it generated some interest so I'm creating this meta issue to track the discussion about what we would need to use this instead of cgit/gitweb.

The current code lives in my GitHub repo called git_repository_viewer.

Some items that probably need to be taken care of before this is committed as a submodule:

  1. browser meta-data needs to stay specific to the version
  2. links on items viewed from a previous commit bring you back to HEAD
  3. add an interface to switch branches
  4. improve "previous commit" so it tracks the previous commit in the db not the first parent
  5. create "next commit" (git data doesn't track this but our DB does)

Comments

webchick’s picture

Assigned: Unassigned » tizzo
marvil07’s picture

implementation of the respective ctools plugin to link to it(when #354509: Make repository URL backends pluggable is ready) would be a good addition too

tizzo’s picture

Assigned: tizzo » Unassigned

Also, the meta data displayed in the list is always from HEAD. We should add some kind of condition that narrows that data to stuff that happened at or before $commit_hash.

It would also be nice to display data about the directories (even though we don't track that data because dirs don't really get modified in git). We could run a second query where path is like [url segment]/%.

The other thing we really need to work on is providing more context about what you're viewing. Right now neither page titles nor breadcrumbs tell you what you're viewing or at what revision. Both should. We probably need to fix that. A nice to have would be a list of commits in a sidebar that gives you the commit summary of the commit you're on and the two or three before and after (again pulled from the db).

eliza411’s picture

Priority: Normal » Critical
Issue tags: +git sprint 8

Tagging for Git Sprint 8

eliza411’s picture

Status: Active » Postponed
Issue tags: -git sprint 8

Removing from Sprint 8 per the decision to roll this out shortly after initial deployment. See the decision background at #939024: Decide if we should switch to a homemade git viewer instead of cgit or gitweb

eliza411’s picture

Issue tags: +git phase 3

tagging git phase 3

sdboyer’s picture

retagging out of phase 2

marvil07’s picture

Project: Version Control API -- Git backend » Repoview

Moving to the right queue, maybe time to close this?