[ANNOUNCE] cgit v0.7

[ANNOUNCE] cgit v0.7

[Lars Hjemli]

cgit v0.7 (a fast webinterface for git) is now available at

     git://hjemli.net/pub/git/cgit

This release includes better search capabilities, better diff,
filtered and sorted branch/tag lists on the summary page, a simple way
to switch between branches and finally a much needed restructuring of
the user interface. There is even a brand new logo, and a number of
bugfixes.

The latest version can be seen in action on http://hjemli.net/git/
(disclaimer: I'm not a web/user interface designer, and should not be
held responsible for any eyeball-related damages).

Big thanks to everyone who submitted patches and feedback!

---
Sortlog since v0.6


Chris Pickel (3):
      Makefile: add support for DESTDIR
      Make cgit honor CACHE_ROOT as defined in Makefile
      Improve the sample cgitrc file

Lars Hjemli (47):
      Makefile: add missing references to DESTDIR
      cgit v0.6.1
      Revert "Makefile: add missing references to DESTDIR"
      Revert part of "Makefile: add support for DESTDIR"
      cgit v0.6.2
      ui-tree: specify parameter position for all htmlf formats
      ui-tree: show last line of blob
      Add cgit.conf to .gitignore, remove *~
      Remove a few compiler warnings
      Use trim_end() to remove trailing slashes
      Upgrade to GIT 1.5.3.2
      cgit v0.6.3
      Add support for a renamelimit option in cgitrc
      Add prefix parameter to cgit_diff_tree()
      Add prefix parameter to cgit_print_diff()
      ui-commit.c: link to diff instead of tree from diffstat
      css: remove the annoying tr:hover rule for diffstat
      gen-version.sh: don't sed the output from git describe
      ui-diff: add links to pre- and postversion of blobs
      cgit.css: make diff headers more visible
      Use git-1.5.3.3
      Skip unknown header fields when parsing tags and commits
      Add functions and types for ref lists
      Use reflist to print branch info
      Use reflist to print tag info
      Sort tags by age
      Add support for config param summary-tags
      Move logic for age comparision from cmp_tag_age into cmp_age()
      Add support for config param summary-branches
      Add descriptions of summary-branches and summary-tags to cgitrc
      Make cgit_print_branches()/cgit_print_tags() external
      Add support for refs view
      Add links to the new refs page from summary page
      Cleanup code introduced by the filter-refs topic
      cgit_parse_commit(): Add missing call to xstrdup()
      Add html_option() function
      Teach log search about --grep, --author and --committer
      Make print_branch() handle refs not pointing at commits
      Teach cgit_object_link() about tag objects
      Add config param 'index-info'
      Change the cgit layout
      Add search parameters to cgit_log_link
      Fix search form action/hidden fields
      Don't include current path in menu links
      Don't include current SHA1 in 'log' menu-item
      Use GIT-1.5.3.5
      CGIT 0.7

Michael Krelin (2):
      fixed typo in cgitrc
      correct typo in CSS

Shunichi Fuji (1):
      Fix typo in css

Re: [ANNOUNCE] cgit v0.7

[Nguyen Thai Ngoc Duy]

I wonder why this is not in git's contrib part. It requires git source
code to build and does not seem to require additional dependencies.
It's smaller than gitweb and does not need perl... MinGW porting guys
may even have a look at it while porting git (and I'm thinking about
porting busybox httpd so that it could run out of the box on Windows
;) ).

Any reasons?

On 11/3/07, Lars Hjemli <hjemli@gmail.com> wrote:
> cgit v0.7 (a fast webinterface for git) is now available at
>
>      git://hjemli.net/pub/git/cgit
>
> This release includes better search capabilities, better diff,
> filtered and sorted branch/tag lists on the summary page, a simple way
> to switch between branches and finally a much needed restructuring of
> the user interface. There is even a brand new logo, and a number of
> bugfixes.
>
> The latest version can be seen in action on http://hjemli.net/git/
> (disclaimer: I'm not a web/user interface designer, and should not be
> held responsible for any eyeball-related damages).
>
> Big thanks to everyone who submitted patches and feedback!
>
> ---
> Sortlog since v0.6
>
>
> Chris Pickel (3):
>       Makefile: add support for DESTDIR
>       Make cgit honor CACHE_ROOT as defined in Makefile
>       Improve the sample cgitrc file
>
> Lars Hjemli (47):
>       Makefile: add missing references to DESTDIR
>       cgit v0.6.1
>       Revert "Makefile: add missing references to DESTDIR"
>       Revert part of "Makefile: add support for DESTDIR"
>       cgit v0.6.2
>       ui-tree: specify parameter position for all htmlf formats
>       ui-tree: show last line of blob
>       Add cgit.conf to .gitignore, remove *~
>       Remove a few compiler warnings
>       Use trim_end() to remove trailing slashes
>       Upgrade to GIT 1.5.3.2
>       cgit v0.6.3
>       Add support for a renamelimit option in cgitrc
>       Add prefix parameter to cgit_diff_tree()
>       Add prefix parameter to cgit_print_diff()
>       ui-commit.c: link to diff instead of tree from diffstat
>       css: remove the annoying tr:hover rule for diffstat
>       gen-version.sh: don't sed the output from git describe
>       ui-diff: add links to pre- and postversion of blobs
>       cgit.css: make diff headers more visible
>       Use git-1.5.3.3
>       Skip unknown header fields when parsing tags and commits
>       Add functions and types for ref lists
>       Use reflist to print branch info
>       Use reflist to print tag info
>       Sort tags by age
>       Add support for config param summary-tags
>       Move logic for age comparision from cmp_tag_age into cmp_age()
>       Add support for config param summary-branches
>       Add descriptions of summary-branches and summary-tags to cgitrc
>       Make cgit_print_branches()/cgit_print_tags() external
>       Add support for refs view
>       Add links to the new refs page from summary page
>       Cleanup code introduced by the filter-refs topic
>       cgit_parse_commit(): Add missing call to xstrdup()
>       Add html_option() function
>       Teach log search about --grep, --author and --committer
>       Make print_branch() handle refs not pointing at commits
>       Teach cgit_object_link() about tag objects
>       Add config param 'index-info'
>       Change the cgit layout
>       Add search parameters to cgit_log_link
>       Fix search form action/hidden fields
>       Don't include current path in menu links
>       Don't include current SHA1 in 'log' menu-item
>       Use GIT-1.5.3.5
>       CGIT 0.7
>
> Michael Krelin (2):
>       fixed typo in cgitrc
>       correct typo in CSS
>
> Shunichi Fuji (1):
>       Fix typo in css
> -
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>


-- 
Duy

Re: [ANNOUNCE] cgit v0.7

[Lars Hjemli]

On Nov 3, 2007 1:15 PM, Nguyen Thai Ngoc Duy <pclouds@gmail.com> wrote:
> I wonder why this is not in git's contrib part.

Well, there is at least one technical reason: cgit uses git as a
submodule, so if it got merged into contrib I believe it would require
all git cloners to use git 1.5.2 or newer. It could also be
potentially interesting if/when git-checkout becomes submodule-aware
;-)

--
larsh

Re: [ANNOUNCE] cgit v0.7

[Jakub Narebski]

Lars Hjemli wrote:

> cgit v0.7 (a fast webinterface for git) is now available at
> 
>      git://hjemli.net/pub/git/cgit
> 
> This release includes better search capabilities, better diff,
> filtered and sorted branch/tag lists on the summary page, a simple way
> to switch between branches and finally a much needed restructuring of
> the user interface. There is even a brand new logo, and a number of
> bugfixes.
> 
> The latest version can be seen in action on http://hjemli.net/git/
> (disclaimer: I'm not a web/user interface designer, and should not be
> held responsible for any eyeball-related damages).

A few comments about new cgit:

 * Very nice cgit logo, but no favicon. Perhaps pacman head and G,
   or pacman head (like in logo) and +/-...

 * Providing reference with full sha1 of referenced object for tags
   list is not IMVHO a good design: what is interesting is type of
   tag, if it is signed it's first line, and if it is lightweight
   pointing to tag then perhaps commit subject.

 * Nice diffstat in commit view; the diff view is better, although I
   wouldn't lump from-file / to-file diff header together with git
   diff header and extended git diff header.

 * I like the sidebar very much, although I'm not sure how it would
   work for larger projects (more branches, much more tags). Also the
   search textbox is not very visible; I'd rather it have "groove"
   view.

 * I like separate 'mirrors' section, although I think it rather
   clashes badly with notion of forks (alternates).

 * I'm not sure if it wouln't be beter to provide -n/+m lines changed
   instead of nn likes changed column.

 * Nice submodule support!

By the way, Freedesktop provides besides standard gitweb interface
also cgit interface at
  http://cgit.freedesktop.org/
Take a look at how such site looks like with large number of projects
(perhaps sidebar is noot such a good idea then?), and with large
projects.

-- 
Jakub Narebski
Poland

Re: [ANNOUNCE] cgit v0.7

[Lars Hjemli]

On Nov 5, 2007 1:29 AM, Jakub Narebski <jnareb@gmail.com> wrote:
> Lars Hjemli wrote:
> > cgit v0.7 (a fast webinterface for git) is now available
>
>  * Very nice cgit logo,

Thanks!

> but no favicon. Perhaps pacman head and G,
>    or pacman head (like in logo) and +/-...

I've never cared much about favicons, but I guess cgit could provide one.

>
>  * Providing reference with full sha1 of referenced object for tags
>    list is not IMVHO a good design: what is interesting is type of
>    tag, if it is signed it's first line, and if it is lightweight
>    pointing to tag then perhaps commit subject.

Yes, the full sha1 is not very interesting. But I'm not sure what to
replace it with: the first line of annotated tags is very often
identical to tag name. Maybe it should just abbreviate the sha1?

>
>  * Nice diffstat in commit view; the diff view is better, although I
>    wouldn't lump from-file / to-file diff header together with git
>    diff header and extended git diff header.

I've tried to make the diff look similar to 'git log -p' output, but I
agree the first line per file is probably overkill.

>
>  * I like the sidebar very much, although I'm not sure how it would
>    work for larger projects (more branches, much more tags).

How do you think it works out with http://hjemli.net/git/xorg/xserver/
? It's got an impressive number of branches and tags ;-)

> Also the
>    search textbox is not very visible; I'd rather it have "groove"
>    view.

Agreed, it's probably useless trying to style input-controls: the
result is heavily browser/platform dependent.


>  * I like separate 'mirrors' section, although I think it rather
>    clashes badly with notion of forks (alternates).

Well, it's only a section header, i.e. a parameter in cgitrc


>  * I'm not sure if it wouln't be beter to provide -n/+m lines changed
>    instead of nn likes changed column.

Maybe. I think it used to be -n/+m, but then I changed it; don't
remember why...

>
>  * Nice submodule support!

Heh, it's a simple hack, but thanks anyway. It probably needs to be
configurable per repo though.

>
> By the way, Freedesktop provides besides standard gitweb interface
> also cgit interface at
>   http://cgit.freedesktop.org/
> Take a look at how such site looks like with large number of projects
> (perhaps sidebar is noot such a good idea then?), and with large
> projects.

Actually, the filtered branch/tag lists was done partly because of
freedesktop.org. I think it has worked out nicely (but
cgit.freedesktop.org needs to run the latest cgit). Also, the width of
project descriptions is configurable, so it can take up much less
space and leave room for the sidebar.

Thanks for the comments, you've made my day!

-- 
larsh

Re: [ANNOUNCE] cgit v0.7

[Lars Hjemli]

On Nov 5, 2007 1:29 AM, Jakub Narebski <jnareb@gmail.com> wrote:
>  * I'm not sure if it wouln't be beter to provide -n/+m lines changed
>    instead of nn likes changed column.

Agreed, and fixed.

--
larsh

Re: [ANNOUNCE] cgit v0.7

[Peter Karlsson]

Lars Hjemli:

> cgit v0.7 (a fast webinterface for git) is now available at
> 
>      git://hjemli.net/pub/git/cgit

Has anyone managed to compile this for Win32? I cannot get gitweb to
work due to the Perl dependencies. When trying to compile cgit it hangs
checking the git version (or something like that, it keeps printing a
version number over and over again).

-- 
\\// Peter - http://www.softwolves.pp.se/

Re: [ANNOUNCE] cgit v0.7

[Lars Hjemli]

On Nov 6, 2007 9:04 AM, Peter Karlsson <peter@softwolves.pp.se> wrote:
> Lars Hjemli:
>
> > cgit v0.7 (a fast webinterface for git) is now available at
> >
> >      git://hjemli.net/pub/git/cgit
>
> Has anyone managed to compile this for Win32?

If (win32 == cygwin): yes, just did (but it needed a makefile tweak
regarding iconv)

-- 
From: Lars Hjemli <hjemli@gmail.com>
Date: Tue, 6 Nov 2007 09:35:07 +0100
Subject: [PATCH] Makefile: link with libiconv if NEEDS_LIBICONV is defined

This seems to be needed to compile on cygwin.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
---
 Makefile |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/Makefile b/Makefile
index 6abd82a..dbc34a2 100644
--- a/Makefile
+++ b/Makefile
@@ -19,6 +19,11 @@ OBJECTS = shared.o cache.o parsing.o html.o
ui-shared.o ui-repolist.o \
        ui-snapshot.o ui-blob.o ui-tag.o ui-refs.o


+ifdef NEEDS_LIBICONV
+       EXTLIBS += -liconv
+endif
+
+
 .PHONY: all git install clean distclean force-version get-git

 all: cgit git
-- 
1.5.3.4.452.g09149

Re: [ANNOUNCE] cgit v0.7

[Patrick Aljord]

Looks great, thanks for the new release.

It would be great if when a commit message contains something such as
#1234 it would automatically link to the bug tracker page of that bug
(like Trac does) by preconfiguring the bugtracker URL.

Such as "bug #238 test for root-window that XFree86 fixed in their"
would link the "#238"  part to
http://bugs.freedesktop.org/show_bug.cgi?id=238

Re: [ANNOUNCE] cgit v0.7

[Lars Hjemli]

On Nov 6, 2007 7:39 PM, Patrick Aljord <patcito@gmail.com> wrote:
> Looks great, thanks for the new release.

Thanks!

> It would be great if when a commit message contains something such as
> #1234 it would automatically link to the bug tracker page of that bug
> (like Trac does) by preconfiguring the bugtracker URL.

Yeah, it shouldn't be to difficult either. We could just add a couple
of repo-options to cgitrc, maybe something like this:

  repo.bugtracker-regex=(#[0-9]+)
  repo.bugtracker-url=http://bugs.freedesktop.org/show_bug.cgi?id=%1

and then use regexec() combined with the interpolate() function in git
to perform the substitution. There could also be similar global
options, i.e.:

  bugtracker-regex=(#[0-9]+)
  bugtracker-url=http://bugs.freedesktop.org/show_bug.cgi?id=%1

which would be used if the per-repo options weren't specified.

I'll put it on my todo-list (and patches are always appreciated).

--
larsh