- updated to 0.9.1 auto/th/cgit-0.9.1-1
authorJan Rękorajski <baggins@pld-linux.org>
Fri, 17 May 2013 11:25:59 +0000 (13:25 +0200)
committerJan Rękorajski <baggins@pld-linux.org>
Fri, 17 May 2013 11:25:59 +0000 (13:25 +0200)
- added git patch to update to cgit git master
- added notes patch to fix building with git 1.8
- updated httpd config for apache 2.4

cgit-httpd.conf
cgit-override-cflags.patch
cgit-system-git.patch
cgit.spec
git.patch [new file with mode: 0644]
notes.patch [new file with mode: 0644]

index cc0b45a..84366da 100644 (file)
@@ -1,13 +1,14 @@
 <Directory /usr/share/cgit>
        AllowOverride None
-       Allow from All
+       Require all granted
 </Directory>
 
 # Version 1 (default): under /cgi-bin/cgit.cgi address
 ScriptAlias /cgi-bin/cgit.cgi /usr/lib/cgi-bin/cgit.cgi
 Alias /cgit/ /usr/share/cgit/
+
 <Location /cgi-bin/cgit.cgi>
-       Allow from All
+       Require all granted
 </Location>
 
 # Version 2: cgit as a handler to whole vhost:
@@ -18,6 +19,6 @@ Alias /cgit/ /usr/share/cgit/
 #      DocumentRoot /usr/lib/cgi-bin/cgit.cgi
 #      <Location />
 #              Options ExecCGI
-#              Allow from All
+#              Require all granted
 #      </Location>
 #</VirtualHost>
index 5b401fe..037f5f7 100644 (file)
@@ -1,22 +1,24 @@
 diff -burN cgit-0.8.1.orig/Makefile cgit-0.8.1/Makefile
 --- cgit-0.8.1.orig/Makefile   2008-12-10 09:59:24.000000000 +0100
 +++ cgit-0.8.1/Makefile        2008-12-11 00:02:15.000000000 +0100
-@@ -86,12 +86,12 @@
- -include VERSION
+@@ -86,13 +86,13 @@
+ endif
  
--CFLAGS += -g -Wall -Igit
+ LDFLAGS ?=
+-CFLAGS ?= -g -Wall
+-CFLAGS += -Igit
 -CFLAGS += -DSHA1_HEADER='$(SHA1_HEADER)'
 -CFLAGS += -DCGIT_VERSION='"$(CGIT_VERSION)"'
 -CFLAGS += -DCGIT_CONFIG='"$(CGIT_CONFIG)"'
 -CFLAGS += -DCGIT_SCRIPT_NAME='"$(CGIT_SCRIPT_NAME)"'
 -CFLAGS += -DCGIT_CACHE_ROOT='"$(CACHE_ROOT)"'
-+override CFLAGS += -g -Wall -Igit
++override CFLAGS ?= -g -Wall
++override CFLAGS += -Igit
 +override CFLAGS += -DSHA1_HEADER='$(SHA1_HEADER)'
 +override CFLAGS += -DCGIT_VERSION='"$(CGIT_VERSION)"'
 +override CFLAGS += -DCGIT_CONFIG='"$(CGIT_CONFIG)"'
 +override CFLAGS += -DCGIT_SCRIPT_NAME='"$(CGIT_SCRIPT_NAME)"'
 +override CFLAGS += -DCGIT_CACHE_ROOT='"$(CACHE_ROOT)"'
  
- ifdef NO_ICONV
-       CFLAGS += -DNO_ICONV
+ ifeq ($(uname_O),Cygwin)
+       NO_STRCASESTR = YesPlease
index 359ffe4..8dfe9ac 100644 (file)
@@ -1,37 +1,33 @@
 --- cgit-0.8.3.4/Makefile.orig 2010-09-27 08:00:47.000000000 +0200
 +++ cgit-0.8.3.4/Makefile      2010-10-14 01:24:36.000000000 +0200
 @@ -8,6 +8,7 @@
- GIT_VER = 1.7.3
- GIT_URL = http://www.kernel.org/pub/software/scm/git/git-$(GIT_VER).tar.bz2
+ GIT_VER = 1.7.12.4
+ GIT_URL = https://git-core.googlecode.com/files/git-$(GIT_VER).tar.gz
  INSTALL = install
 +LIBDIR = /usr/lib
+ MAN5_TXT = $(wildcard *.5.txt)
+ MAN_TXT  = $(MAN5_TXT)
+ DOC_MAN5 = $(patsubst %.txt,%,$(MAN5_TXT))
+@@ -1238,7 +123,7 @@
+       LDLIBS += -liconv
+ endif
  
- # Define NO_STRCASESTR if you don't have strcasestr.
- #
-@@ -68,7 +69,7 @@
-       $(QUIET_CC)$(CC) -o $*.o -c $(CFLAGS) $<
+-LDLIBS += git/libgit.a git/xdiff/lib.a -lz -lpthread
++LDLIBS += $(LIBDIR)/libgit.a $(LIBDIR)/libgit_xdiff.a -lpcre -lz -lpthread
  
--EXTLIBS = git/libgit.a git/xdiff/lib.a -lz -lcrypto -lpthread
-+EXTLIBS = $(LIBDIR)/libgit.a $(LIBDIR)/libgit_xdiff.a -lz -lcrypto -lpthread
- OBJECTS =
- OBJECTS += cache.o
  OBJECTS += cgit.o
-@@ -124,17 +125,13 @@
-       CFLAGS += -DNO_STRCASESTR
- endif
--cgit: $(OBJECTS) libgit
-+cgit: $(OBJECTS)
-       $(QUIET_CC)$(CC) $(CFLAGS) $(LDFLAGS) -o cgit $(OBJECTS) $(EXTLIBS)
+ OBJECTS += cache.o
+@@ -174,13 +175,9 @@
  
- cgit.o: VERSION
+ all:: cgit
  
- -include $(OBJECTS:.o=.d)
+-cgit: VERSION $(OBJECTS) libgit
++cgit: VERSION $(OBJECTS)
+       $(QUIET_LINK)$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $(OBJECTS) $(LDLIBS)
  
 -libgit:
--      $(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) NO_CURL=1 libgit.a
--      $(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) NO_CURL=1 xdiff/lib.a
+-      $(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) NO_CURL=1 $(GIT_OPTIONS) libgit.a
+-      $(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) NO_CURL=1 $(GIT_OPTIONS) xdiff/lib.a
 -
  test: all
        $(QUIET_SUBDIR0)tests $(QUIET_SUBDIR1) all
index 0075933..1d8134e 100644 (file)
--- a/cgit.spec
+++ b/cgit.spec
@@ -5,20 +5,22 @@
 Summary:       cgit - a fast webinterface to git
 Summary(pl.UTF-8):     cgit - szybki interfejs WWW do gita
 Name:          cgit
-Version:       0.8.3.4
-Release:       2
+Version:       0.9.1
+Release:       1
 License:       GPL v2
 Group:         Development/Tools
 Source0:       http://hjemli.net/git/cgit/snapshot/%{name}-%{version}.tar.bz2
-# Source0-md5: 712e4d3013d754aa5752e0101188cf32
+# Source0-md5: 78403e6a15a61bb06cb2b8447079139a
 Source1:       %{name}.conf
 Source2:       %{name}-repo.conf
 Source3:       %{name}-apache.conf
 Source4:       %{name}-httpd.conf
 Patch0:                %{name}-system-git.patch
 Patch1:                %{name}-override-cflags.patch
+Patch2:                notes.patch
+Patch100:      git.patch
 URL:           http://hjemli.net/git/cgit
-BuildRequires: git-core-devel >= 1.7.3
+BuildRequires: git-core-devel >= 1.8.0
 BuildRequires: openssl-devel
 BuildConflicts:        zlib-devel = 1.2.5-1
 Requires:      webapps
@@ -46,8 +48,10 @@ HTML zapisany jest na dysku dla kolejnych żądań.
 
 %prep
 %setup -q
+%patch100 -p1
 %patch0 -p1
 %patch1 -p1
+%patch2 -p1
 
 %build
 %{__make} \
@@ -64,6 +68,7 @@ rm -rf $RPM_BUILD_ROOT
 
 %{__make} install \
        DESTDIR=$RPM_BUILD_ROOT \
+       LIBDIR=%{_libdir} \
        CGIT_CONFIG="%{webappdir}/%{webapp.conf}" \
        CGIT_DATA_PATH="%{appdir}" \
        CGIT_SCRIPT_PATH="%{cgibindir}" \
@@ -105,3 +110,7 @@ rm -rf $RPM_BUILD_ROOT
 %attr(755,root,root) %{cgibindir}/cgit.cgi
 %attr(770,root,http) /var/cache/cgit
 %{appdir}
+%dir %{_prefix}/lib/cgit
+%dir %{_prefix}/lib/cgit/filters
+%attr(755,root,root) %{_prefix}/lib/cgit/filters/commit-links.sh
+%attr(755,root,root) %{_prefix}/lib/cgit/filters/syntax-highlighting.sh
diff --git a/git.patch b/git.patch
new file mode 100644 (file)
index 0000000..24807e0
--- /dev/null
+++ b/git.patch
@@ -0,0 +1,2151 @@
+diff --git a/Makefile b/Makefile
+index bbce29d..1127961 100644
+--- a/Makefile
++++ b/Makefile
+@@ -1,3 +1,5 @@
++all::
++
+ CGIT_VERSION = v0.9.1
+ CGIT_SCRIPT_NAME = cgit.cgi
+ CGIT_SCRIPT_PATH = /var/www/htdocs/cgit
+@@ -12,8 +14,8 @@ htmldir = $(docdir)
+ pdfdir = $(docdir)
+ mandir = $(prefix)/share/man
+ SHA1_HEADER = <openssl/sha.h>
+-GIT_VER = 1.7.4
+-GIT_URL = https://github.com/git/git/archive/v$(GIT_VER).tar.gz
++GIT_VER = 1.7.12.4
++GIT_URL = https://git-core.googlecode.com/files/git-$(GIT_VER).tar.gz
+ INSTALL = install
+ MAN5_TXT = $(wildcard *.5.txt)
+ MAN_TXT  = $(MAN5_TXT)
+@@ -40,22 +42,14 @@ DOC_PDF  = $(patsubst %.txt,%.pdf,$(MAN_TXT))
+ # Platform specific tweaks
+ #
++VERSION: force-version
++      @./gen-version.sh "$(CGIT_VERSION)"
++-include VERSION
++
+ uname_S := $(shell sh -c 'uname -s 2>/dev/null || echo not')
+ uname_O := $(shell sh -c 'uname -o 2>/dev/null || echo not')
+ uname_R := $(shell sh -c 'uname -r 2>/dev/null || echo not')
+-ifeq ($(uname_O),Cygwin)
+-      NO_STRCASESTR = YesPlease
+-      NEEDS_LIBICONV = YesPlease
+-endif
+-
+-ifeq ($(uname_S),$(filter $(uname_S),FreeBSD OpenBSD))
+-      # Apparantly libiconv is installed in /usr/local on BSD
+-      LDFLAGS ?= -L/usr/local/lib
+-      CFLAGS ?= -I/usr/local/include
+-      NEEDS_LIBICONV = yes
+-endif
+-
+ #
+ # Let the user override the above settings.
+ #
+@@ -76,30 +70,62 @@ endif
+ ifndef V
+       QUIET_CC       = @echo '   ' CC $@;
+-      QUIET_MM       = @echo '   ' MM $@;
++      QUIET_LINK     = @echo '   ' LINK $@;
+       QUIET_SUBDIR0  = +@subdir=
+       QUIET_SUBDIR1  = ;$(NO_SUBDIR) echo '   ' SUBDIR $$subdir; \
+                        $(MAKE) $(PRINT_DIR) -C $$subdir
+       QUIET_TAGS     = @echo '   ' TAGS $@;
++      export V
+ endif
+-#
+-# Define a pattern rule for automatic dependency building
+-#
+-%.d: %.c
+-      $(QUIET_MM)$(CC) $(CFLAGS) -MM -MP $< | sed -e 's/\($*\)\.o:/\1.o $@:/g' >$@
++LDFLAGS ?=
++CFLAGS ?= -g -Wall
++CFLAGS += -Igit
++CFLAGS += -DSHA1_HEADER='$(SHA1_HEADER)'
++CFLAGS += -DCGIT_VERSION='"$(CGIT_VERSION)"'
++CFLAGS += -DCGIT_CONFIG='"$(CGIT_CONFIG)"'
++CFLAGS += -DCGIT_SCRIPT_NAME='"$(CGIT_SCRIPT_NAME)"'
++CFLAGS += -DCGIT_CACHE_ROOT='"$(CACHE_ROOT)"'
+-#
+-# Define a pattern rule for silent object building
+-#
+-%.o: %.c
+-      $(QUIET_CC)$(CC) -o $*.o -c $(CFLAGS) $<
++ifeq ($(uname_O),Cygwin)
++      NO_STRCASESTR = YesPlease
++      NEEDS_LIBICONV = YesPlease
++endif
++ifeq ($(uname_S),$(filter $(uname_S),FreeBSD OpenBSD))
++      # Apparantly libiconv is installed in /usr/local on BSD
++      LDFLAGS += -L/usr/local/lib
++      CFLAGS += -I/usr/local/include
++      NEEDS_LIBICONV = yes
++endif
+-EXTLIBS = git/libgit.a git/xdiff/lib.a -lz -lpthread
++GIT_OPTIONS = prefix=/usr NO_GETTEXT=1
+ OBJECTS =
+-OBJECTS += cache.o
++
++ifdef NO_ICONV
++      CFLAGS += -DNO_ICONV
++endif
++ifdef NO_STRCASESTR
++      CFLAGS += -DNO_STRCASESTR
++endif
++ifdef NO_C99_FORMAT
++      CFLAGS += -DNO_C99_FORMAT
++endif
++ifdef NO_OPENSSL
++      CFLAGS += -DNO_OPENSSL
++      GIT_OPTIONS += NO_OPENSSL=1
++else
++      LDLIBS += -lcrypto
++endif
++
++ifdef NEEDS_LIBICONV
++      LDLIBS += -liconv
++endif
++
++LDLIBS += git/libgit.a git/xdiff/lib.a -lz -lpthread
++
+ OBJECTS += cgit.o
++OBJECTS += cache.o
+ OBJECTS += cmd.o
+ OBJECTS += configfile.o
+ OBJECTS += html.o
+@@ -125,55 +151,30 @@ OBJECTS += ui-tag.o
+ OBJECTS += ui-tree.o
+ OBJECTS += vector.o
+-ifdef NEEDS_LIBICONV
+-      EXTLIBS += -liconv
+-endif
+-
+-
+-.PHONY: all libgit test install uninstall clean force-version get-git \
+-      doc clean-doc install-doc install-man install-html install-pdf \
+-      uninstall-doc uninstall-man uninstall-html uninstall-pdf tags
++dep_files := $(foreach f,$(OBJECTS),$(dir $f).deps/$(notdir $f).d)
++dep_dirs := $(addsuffix .deps,$(sort $(dir $OBJECTS)))
+-all: cgit
++$(dep_dirs):
++      @mkdir -p $@
+-VERSION: force-version
+-      @./gen-version.sh "$(CGIT_VERSION)"
+--include VERSION
++missing_dep_dirs := $(filter-out $(wildcard $(dep_dirs)),$(dep_dirs))
++dep_file = $(dir $@).deps/$(notdir $@).d
++dep_args = -MF $(dep_file) -MMD -MP
++.SUFFIXES:
+-CFLAGS += -g -Wall -Igit
+-CFLAGS += -DSHA1_HEADER='$(SHA1_HEADER)'
+-CFLAGS += -DCGIT_VERSION='"$(CGIT_VERSION)"'
+-CFLAGS += -DCGIT_CONFIG='"$(CGIT_CONFIG)"'
+-CFLAGS += -DCGIT_SCRIPT_NAME='"$(CGIT_SCRIPT_NAME)"'
+-CFLAGS += -DCGIT_CACHE_ROOT='"$(CACHE_ROOT)"'
+-
+-GIT_OPTIONS = prefix=/usr
++$(OBJECTS): %.o: %.c $(missing_dep_dirs)
++      $(QUIET_CC)$(CC) -o $*.o -c $(dep_args) $(CFLAGS) $<
+-ifdef NO_ICONV
+-      CFLAGS += -DNO_ICONV
++dep_files_present := $(wildcard $(dep_files))
++ifneq ($(dep_files_present),)
++include $(dep_files_present)
+ endif
+-ifdef NO_STRCASESTR
+-      CFLAGS += -DNO_STRCASESTR
+-endif
+-ifdef NO_C99_FORMAT
+-      CFLAGS += -DNO_C99_FORMAT
+-endif
+-ifdef NO_OPENSSL
+-      CFLAGS += -DNO_OPENSSL
+-      GIT_OPTIONS += NO_OPENSSL=1
+-else
+-      EXTLIBS += -lcrypto
+-endif
+-
+-cgit: $(OBJECTS) libgit
+-      $(QUIET_CC)$(CC) $(CFLAGS) $(LDFLAGS) -o cgit $(OBJECTS) $(EXTLIBS)
+-cgit.o: VERSION
++all:: cgit
+-ifneq "$(MAKECMDGOALS)" "clean"
+-  -include $(OBJECTS:.o=.d)
+-endif
++cgit: VERSION $(OBJECTS) libgit
++      $(QUIET_LINK)$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $(OBJECTS) $(LDLIBS)
+ libgit:
+       $(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) NO_CURL=1 $(GIT_OPTIONS) libgit.a
+@@ -243,13 +244,24 @@ $(DOC_PDF): %.pdf : %.txt
+       a2x -f pdf cgitrc.5.txt
+ clean: clean-doc
+-      rm -f cgit VERSION *.o *.d tags
++      $(RM) cgit VERSION *.o tags
++      $(RM) -r .deps
++
++cleanall: clean
++      $(MAKE) -C git clean
+ clean-doc:
+-      rm -f cgitrc.5 cgitrc.5.html cgitrc.5.pdf cgitrc.5.xml cgitrc.5.fo
++      $(RM) cgitrc.5 cgitrc.5.html cgitrc.5.pdf cgitrc.5.xml cgitrc.5.fo
+ get-git:
+       curl -L $(GIT_URL) | tar -xzf - && rm -rf git && mv git-$(GIT_VER) git
+ tags:
+       $(QUIET_TAGS)find . -name '*.[ch]' | xargs ctags
++
++.PHONY: all cgit get-git libgit force-version
++.PHONY: clean clean-doc cleanall
++.PHONY: doc doc-html doc-man doc-pdf
++.PHONY: install install-doc install-html install-man install-pdf
++.PHONY: tags test
++.PHONY: uninstall uninstall-doc uninstall-html uninstall-man uninstall-pdf
+diff --git a/cache.c b/cache.c
+index d7a8d5a..47cdcb4 100644
+--- a/cache.c
++++ b/cache.c
+@@ -105,7 +105,7 @@ static int is_expired(struct cache_slot *slot)
+       if (slot->ttl < 0)
+               return 0;
+       else
+-              return slot->cache_st.st_mtime + slot->ttl*60 < time(NULL);
++              return slot->cache_st.st_mtime + slot->ttl * 60 < time(NULL);
+ }
+ /* Check if the slot has been modified since we opened it.
+@@ -141,8 +141,8 @@ static int close_lock(struct cache_slot *slot)
+  */
+ static int lock_slot(struct cache_slot *slot)
+ {
+-      slot->lock_fd = open(slot->lock_name, O_RDWR|O_CREAT|O_EXCL,
+-                           S_IRUSR|S_IWUSR);
++      slot->lock_fd = open(slot->lock_name, O_RDWR | O_CREAT | O_EXCL,
++                           S_IRUSR | S_IWUSR);
+       if (slot->lock_fd == -1)
+               return errno;
+       if (xwrite(slot->lock_fd, slot->key, slot->keylen + 1) < 0)
+@@ -214,7 +214,7 @@ unsigned long hash_str(const char *str)
+       if (!s)
+               return h;
+-      while(*s) {
++      while (*s) {
+               h *= FNV_PRIME;
+               h ^= *s++;
+       }
+@@ -342,7 +342,7 @@ int cache_process(int size, const char *path, const char *key, int ttl,
+       strcpy(filename, path);
+       if (filename[len - 1] != '/')
+               filename[len++] = '/';
+-      for(i = 0; i < 8; i++) {
++      for (i = 0; i < 8; i++) {
+               sprintf(filename + len++, "%x",
+                       (unsigned char)(hash & 0xf));
+               hash >>= 4;
+@@ -407,7 +407,7 @@ int cache_ls(const char *path)
+               *name = '\0';
+       }
+       slot.cache_name = fullname;
+-      while((ent = readdir(dir)) != NULL) {
++      while ((ent = readdir(dir)) != NULL) {
+               if (strlen(ent->d_name) != 8)
+                       continue;
+               strcpy(name, ent->d_name);
+diff --git a/cgit.c b/cgit.c
+index a97ed69..2ccf864 100644
+--- a/cgit.c
++++ b/cgit.c
+@@ -172,6 +172,8 @@ void config_cb(const char *name, const char *value)
+               ctx.cfg.enable_http_clone = atoi(value);
+       else if (!strcmp(name, "enable-index-links"))
+               ctx.cfg.enable_index_links = atoi(value);
++      else if (!strcmp(name, "enable-index-owner"))
++              ctx.cfg.enable_index_owner = atoi(value);
+       else if (!strcmp(name, "enable-commit-graph"))
+               ctx.cfg.enable_commit_graph = atoi(value);
+       else if (!strcmp(name, "enable-log-filecount"))
+@@ -313,7 +315,7 @@ static void querystring_cb(const char *name, const char *value)
+               ctx.qry.name = xstrdup(value);
+       } else if (!strcmp(name, "mimetype")) {
+               ctx.qry.mimetype = xstrdup(value);
+-      } else if (!strcmp(name, "s")){
++      } else if (!strcmp(name, "s")) {
+               ctx.qry.sort = xstrdup(value);
+       } else if (!strcmp(name, "showmsg")) {
+               ctx.qry.showmsg = atoi(value);
+@@ -354,6 +356,7 @@ static void prepare_context(struct cgit_context *ctx)
+       ctx->cfg.logo = "/cgit.png";
+       ctx->cfg.local_time = 0;
+       ctx->cfg.enable_http_clone = 1;
++      ctx->cfg.enable_index_owner = 1;
+       ctx->cfg.enable_tree_linenumbers = 1;
+       ctx->cfg.enable_git_config = 0;
+       ctx->cfg.max_repo_count = 50;
+@@ -442,12 +445,12 @@ char *find_default_branch(struct cgit_repo *repo)
+       return ref;
+ }
+-static char *guess_defbranch(const char *repo_path)
++static char *guess_defbranch(void)
+ {
+       const char *ref;
+       unsigned char sha1[20];
+-      ref = resolve_ref("HEAD", sha1, 0, NULL);
++      ref = resolve_ref_unsafe("HEAD", sha1, 0, NULL);
+       if (!ref || prefixcmp(ref, "refs/heads/"))
+               return "master";
+       return xstrdup(ref + 11);
+@@ -480,7 +483,7 @@ static int prepare_repo_cmd(struct cgit_context *ctx)
+       ctx->page.title = fmt("%s - %s", ctx->repo->name, ctx->repo->desc);
+       if (!ctx->repo->defbranch)
+-              ctx->repo->defbranch = guess_defbranch(ctx->repo->path);
++              ctx->repo->defbranch = guess_defbranch();
+       if (!ctx->qry.head) {
+               ctx->qry.nohead = 1;
+@@ -650,7 +653,7 @@ void print_repolist(FILE *f, struct cgit_repolist *list, int start)
+ {
+       int i;
+-      for(i = start; i < list->count; i++)
++      for (i = start; i < list->count; i++)
+               print_repo(f, &list->repos[i]);
+ }
+@@ -738,7 +741,7 @@ static void cgit_parse_args(int argc, const char **argv)
+       for (i = 1; i < argc; i++) {
+               if (!strncmp(argv[i], "--cache=", 8)) {
+-                      ctx.cfg.cache_root = xstrdup(argv[i]+8);
++                      ctx.cfg.cache_root = xstrdup(argv[i] + 8);
+               }
+               if (!strcmp(argv[i], "--nocache")) {
+                       ctx.cfg.nocache = 1;
+@@ -747,24 +750,24 @@ static void cgit_parse_args(int argc, const char **argv)
+                       ctx.env.no_http = "1";
+               }
+               if (!strncmp(argv[i], "--query=", 8)) {
+-                      ctx.qry.raw = xstrdup(argv[i]+8);
++                      ctx.qry.raw = xstrdup(argv[i] + 8);
+               }
+               if (!strncmp(argv[i], "--repo=", 7)) {
+-                      ctx.qry.repo = xstrdup(argv[i]+7);
++                      ctx.qry.repo = xstrdup(argv[i] + 7);
+               }
+               if (!strncmp(argv[i], "--page=", 7)) {
+-                      ctx.qry.page = xstrdup(argv[i]+7);
++                      ctx.qry.page = xstrdup(argv[i] + 7);
+               }
+               if (!strncmp(argv[i], "--head=", 7)) {
+-                      ctx.qry.head = xstrdup(argv[i]+7);
++                      ctx.qry.head = xstrdup(argv[i] + 7);
+                       ctx.qry.has_symref = 1;
+               }
+               if (!strncmp(argv[i], "--sha1=", 7)) {
+-                      ctx.qry.sha1 = xstrdup(argv[i]+7);
++                      ctx.qry.sha1 = xstrdup(argv[i] + 7);
+                       ctx.qry.has_sha1 = 1;
+               }
+               if (!strncmp(argv[i], "--ofs=", 6)) {
+-                      ctx.qry.ofs = atoi(argv[i]+6);
++                      ctx.qry.ofs = atoi(argv[i] + 6);
+               }
+               if (!strncmp(argv[i], "--scan-tree=", 12) ||
+                   !strncmp(argv[i], "--scan-path=", 12)) {
+@@ -831,7 +834,7 @@ int main(int argc, const char **argv)
+               ctx.cfg.virtual_root = trim_end(ctx.cfg.script_name, '/');
+               if (!ctx.cfg.virtual_root)
+                       ctx.cfg.virtual_root = "";
+-        }
++      }
+       /* If no url parameter is specified on the querystring, lets
+        * use PATH_INFO as url. This allows cgit to work with virtual
+@@ -853,7 +856,7 @@ int main(int argc, const char **argv)
+       }
+       ttl = calc_ttl();
+-      ctx.page.expires += ttl*60;
++      ctx.page.expires += ttl * 60;
+       if (ctx.env.request_method && !strcmp(ctx.env.request_method, "HEAD"))
+               ctx.cfg.nocache = 1;
+       if (ctx.cfg.nocache)
+diff --git a/cgit.h b/cgit.h
+index 7a99135..c655bd8 100644
+--- a/cgit.h
++++ b/cgit.h
+@@ -11,6 +11,7 @@
+ #include <tag.h>
+ #include <diff.h>
+ #include <diffcore.h>
++#include <argv-array.h>
+ #include <refs.h>
+ #include <revision.h>
+ #include <log-tree.h>
+@@ -203,6 +204,7 @@ struct cgit_config {
+       int enable_filter_overrides;
+       int enable_http_clone;
+       int enable_index_links;
++      int enable_index_owner;
+       int enable_commit_graph;
+       int enable_log_filecount;
+       int enable_log_linecount;
+@@ -273,6 +275,8 @@ struct cgit_context {
+       struct cgit_page page;
+ };
++typedef int (*write_archive_fn_t)(const char *, const char *);
++
+ struct cgit_snapshot_format {
+       const char *suffix;
+       const char *mimetype;
+diff --git a/cgitrc.5.txt b/cgitrc.5.txt
+index 95a1049..4d27d9f 100644
+--- a/cgitrc.5.txt
++++ b/cgitrc.5.txt
+@@ -120,6 +120,10 @@ enable-index-links::
+       each repo in the repository index (specifically, to the "summary",
+       "commit" and "tree" pages). Default value: "0".
++enable-index-owner::
++      Flag which, when set to "1", will make cgit display the owner of
++      each repo in the repository index. Default value: "1".
++
+ enable-log-filecount::
+       Flag which, when set to "1", will make cgit print the number of
+       modified files for each commit on the repository log page. Default
+@@ -244,8 +248,8 @@ mimetype-file::
+       Specifies the file to use for automatic mimetype lookup. If specified
+       then this field is used as a fallback when no "mimetype.<ext>" match is
+       found. If unspecified then no such lookup is performed. The typical file
+-      to use on a Linux system is /etc/mime.types. Default value: none. See
+-      also: "mimetype.<ext>". The format of the file must comply to:
++      to use on a Linux system is /etc/mime.types. The format of the file must
++      comply to:
+       - a comment line is an empty line or a line starting with a hash (#),
+         optionally preceded by whitespace
+       - a non-comment line starts with the mimetype (like image/png), followed
+diff --git a/cmd.c b/cmd.c
+index 899e913..198bf2f 100644
+--- a/cmd.c
++++ b/cmd.c
+@@ -166,7 +166,7 @@ struct cgit_cmd *cgit_get_cmd(struct cgit_context *ctx)
+                       ctx->qry.page = "repolist";
+       }
+-      for(i = 0; i < sizeof(cmds)/sizeof(*cmds); i++)
++      for (i = 0; i < sizeof(cmds)/sizeof(*cmds); i++)
+               if (!strcmp(ctx->qry.page, cmds[i].name))
+                       return &cmds[i];
+       return NULL;
+diff --git a/configfile.c b/configfile.c
+index 4908058..3fa217f 100644
+--- a/configfile.c
++++ b/configfile.c
+@@ -13,9 +13,9 @@
+ int next_char(FILE *f)
+ {
+       int c = fgetc(f);
+-      if (c=='\r') {
++      if (c == '\r') {
+               c = fgetc(f);
+-              if (c!='\n') {
++              if (c != '\n') {
+                       ungetc(c, f);
+                       c = '\r';
+               }
+@@ -27,7 +27,7 @@ void skip_line(FILE *f)
+ {
+       int c;
+-      while((c=next_char(f)) && c!='\n' && c!=EOF)
++      while ((c = next_char(f)) && c != '\n' && c != EOF)
+               ;
+ }
+@@ -36,31 +36,31 @@ int read_config_line(FILE *f, char *line, const char **value, int bufsize)
+       int i = 0, isname = 0;
+       *value = NULL;
+-      while(i<bufsize-1) {
++      while (i < bufsize - 1) {
+               int c = next_char(f);
+-              if (!isname && (c=='#' || c==';')) {
++              if (!isname && (c == '#' || c == ';')) {
+                       skip_line(f);
+                       continue;
+               }
+               if (!isname && isspace(c))
+                       continue;
+-              if (c=='=' && !*value) {
++              if (c == '=' && !*value) {
+                       line[i] = 0;
+-                      *value = &line[i+1];
+-              } else if (c=='\n' && !isname) {
++                      *value = &line[i + 1];
++              } else if (c == '\n' && !isname) {
+                       i = 0;
+                       continue;
+-              } else if (c=='\n' || c==EOF) {
++              } else if (c == '\n' || c == EOF) {
+                       line[i] = 0;
+                       break;
+               } else {
+-                      line[i]=c;
++                      line[i] = c;
+               }
+               isname = 1;
+               i++;
+       }
+-      line[i+1] = 0;
++      line[i + 1] = 0;
+       return i;
+ }
+@@ -78,7 +78,7 @@ int parse_configfile(const char *filename, configfile_value_fn fn)
+       if (!(f = fopen(filename, "r")))
+               return -1;
+       nesting++;
+-      while((len = read_config_line(f, line, &value, sizeof(line))) > 0)
++      while ((len = read_config_line(f, line, &value, sizeof(line))) > 0)
+               fn(line, value);
+       nesting--;
+       fclose(f);
+diff --git a/html.c b/html.c
+index 8f6e4f6..90cc1c0 100644
+--- a/html.c
++++ b/html.c
+@@ -54,7 +54,7 @@ char *fmt(const char *format, ...)
+       va_start(args, format);
+       len = vsnprintf(buf[bufidx], sizeof(buf[bufidx]), format, args);
+       va_end(args);
+-      if (len>sizeof(buf[bufidx])) {
++      if (len > sizeof(buf[bufidx])) {
+               fprintf(stderr, "[html.c] string truncated: %s\n", format);
+               exit(1);
+       }
+@@ -92,93 +92,93 @@ void html_status(int code, const char *msg, int more_headers)
+ void html_txt(const char *txt)
+ {
+       const char *t = txt;
+-      while(t && *t){
++      while (t && *t) {
+               int c = *t;
+-              if (c=='<' || c=='>' || c=='&') {
++              if (c == '<' || c == '>' || c == '&') {
+                       html_raw(txt, t - txt);
+-                      if (c=='>')
++                      if (c == '>')
+                               html("&gt;");
+-                      else if (c=='<')
++                      else if (c == '<')
+                               html("&lt;");
+-                      else if (c=='&')
++                      else if (c == '&')
+                               html("&amp;");
+-                      txt = t+1;
++                      txt = t + 1;
+               }
+               t++;
+       }
+-      if (t!=txt)
++      if (t != txt)
+               html(txt);
+ }
+ void html_ntxt(int len, const char *txt)
+ {
+       const char *t = txt;
+-      while(t && *t && len--){
++      while (t && *t && len--) {
+               int c = *t;
+-              if (c=='<' || c=='>' || c=='&') {
++              if (c == '<' || c == '>' || c == '&') {
+                       html_raw(txt, t - txt);
+-                      if (c=='>')
++                      if (c == '>')
+                               html("&gt;");
+-                      else if (c=='<')
++                      else if (c == '<')
+                               html("&lt;");
+-                      else if (c=='&')
++                      else if (c == '&')
+                               html("&amp;");
+-                      txt = t+1;
++                      txt = t + 1;
+               }
+               t++;
+       }
+-      if (t!=txt)
++      if (t != txt)
+               html_raw(txt, t - txt);
+-      if (len<0)
++      if (len < 0)
+               html("...");
+ }
+ void html_attr(const char *txt)
+ {
+       const char *t = txt;
+-      while(t && *t){
++      while (t && *t) {
+               int c = *t;
+-              if (c=='<' || c=='>' || c=='\'' || c=='\"' || c=='&') {
++              if (c == '<' || c == '>' || c == '\'' || c == '\"' || c == '&') {
+                       html_raw(txt, t - txt);
+-                      if (c=='>')
++                      if (c == '>')
+                               html("&gt;");
+-                      else if (c=='<')
++                      else if (c == '<')
+                               html("&lt;");
+-                      else if (c=='\'')
++                      else if (c == '\'')
+                               html("&#x27;");
+-                      else if (c=='"')
++                      else if (c == '"')
+                               html("&quot;");
+-                      else if (c=='&')
++                      else if (c == '&')
+                               html("&amp;");
+-                      txt = t+1;
++                      txt = t + 1;
+               }
+               t++;
+       }
+-      if (t!=txt)
++      if (t != txt)
+               html(txt);
+ }
+ void html_url_path(const char *txt)
+ {
+       const char *t = txt;
+-      while(t && *t){
++      while (t && *t) {
+               unsigned char c = *t;
+               const char *e = url_escape_table[c];
+-              if (e && c!='+' && c!='&') {
++              if (e && c != '+' && c != '&') {
+                       html_raw(txt, t - txt);
+                       html(e);
+-                      txt = t+1;
++                      txt = t + 1;
+               }
+               t++;
+       }
+-      if (t!=txt)
++      if (t != txt)
+               html(txt);
+ }
+ void html_url_arg(const char *txt)
+ {
+       const char *t = txt;
+-      while(t && *t){
++      while (t && *t) {
+               unsigned char c = *t;
+               const char *e = url_escape_table[c];
+               if (c == ' ')
+@@ -186,11 +186,11 @@ void html_url_arg(const char *txt)
+               if (e) {
+                       html_raw(txt, t - txt);
+                       html(e);
+-                      txt = t+1;
++                      txt = t + 1;
+               }
+               t++;
+       }
+-      if (t!=txt)
++      if (t != txt)
+               html(txt);
+ }
+@@ -260,7 +260,7 @@ int html_include(const char *filename)
+                       filename, strerror(errno), errno);
+               return -1;
+       }
+-      while((len = fread(buf, 1, 4096, f)) > 0)
++      while ((len = fread(buf, 1, 4096, f)) > 0)
+               html_raw(buf, len);
+       fclose(f);
+       return 0;
+@@ -286,14 +286,14 @@ char *convert_query_hexchar(char *txt)
+               *txt = '\0';
+               return txt-1;
+       }
+-      d1 = hextoint(*(txt+1));
+-      d2 = hextoint(*(txt+2));
+-      if (d1<0 || d2<0) {
+-              memmove(txt, txt+3, n-2);
++      d1 = hextoint(*(txt + 1));
++      d2 = hextoint(*(txt + 2));
++      if (d1 < 0 || d2 < 0) {
++              memmove(txt, txt + 3, n - 2);
+               return txt-1;
+       } else {
+               *txt = d1 * 16 + d2;
+-              memmove(txt+1, txt+3, n-2);
++              memmove(txt + 1, txt + 3, n - 2);
+               return txt;
+       }
+ }
+@@ -310,23 +310,23 @@ int http_parse_querystring(const char *txt_, void (*fn)(const char *name, const
+               printf("Out of memory\n");
+               exit(1);
+       }
+-      while((c=*t) != '\0') {
+-              if (c=='=') {
++      while ((c=*t) != '\0') {
++              if (c == '=') {
+                       *t = '\0';
+-                      value = t+1;
+-              } else if (c=='+') {
++                      value = t + 1;
++              } else if (c == '+') {
+                       *t = ' ';
+-              } else if (c=='%') {
++              } else if (c == '%') {
+                       t = convert_query_hexchar(t);
+-              } else if (c=='&') {
++              } else if (c == '&') {
+                       *t = '\0';
+                       (*fn)(txt, value);
+-                      txt = t+1;
++                      txt = t + 1;
+                       value = NULL;
+               }
+               t++;
+       }
+-      if (t!=txt)
++      if (t != txt)
+               (*fn)(txt, value);
+       free(o);
+       return 0;
+diff --git a/parsing.c b/parsing.c
+index 1b2a551..9b7efb3 100644
+--- a/parsing.c
++++ b/parsing.c
+@@ -112,7 +112,7 @@ const char *reencode(char **txt, const char *src_enc, const char *dst_enc)
+               return *txt;
+       /* no encoding needed if src_enc equals dst_enc */
+-      if(!strcasecmp(src_enc, dst_enc))
++      if (!strcasecmp(src_enc, dst_enc))
+               return *txt;
+       tmp = reencode_string(*txt, dst_enc, src_enc);
+@@ -170,7 +170,7 @@ struct commitinfo *cgit_parse_commit(struct commit *commit)
+       }
+       /* if no special encoding is found, assume UTF-8 */
+-      if(!ret->msg_encoding)
++      if (!ret->msg_encoding)
+               ret->msg_encoding = xstrdup("UTF-8");
+       // skip unknown header fields
+diff --git a/scan-tree.c b/scan-tree.c
+index 6ce8036..10d90f4 100644
+--- a/scan-tree.c
++++ b/scan-tree.c
+@@ -106,7 +106,7 @@ static void add_repo(const char *base, const char *path, repo_config_fn fn)
+       config_fn = fn;
+       if (ctx.cfg.enable_git_config)
+               git_config_from_file(gitconfig_config, fmt("%s/config", path), NULL);
+-      
++
+       if (ctx.cfg.remove_suffix)
+               if ((p = strrchr(repo->url, '.')) && !strcmp(p, ".git"))
+                       *p = '\0';
+@@ -185,7 +185,7 @@ static void scan_path(const char *base, const char *path, repo_config_fn fn)
+               add_repo(base, fmt("%s/.git", path), fn);
+               goto end;
+       }
+-      while((ent = readdir(dir)) != NULL) {
++      while ((ent = readdir(dir)) != NULL) {
+               if (ent->d_name[0] == '.') {
+                       if (ent->d_name[1] == '\0')
+                               continue;
+@@ -222,7 +222,7 @@ void scan_projects(const char *path, const char *projectsfile, repo_config_fn fn
+       char line[MAX_PATH * 2], *z;
+       FILE *projects;
+       int err;
+-      
++
+       projects = fopen(projectsfile, "r");
+       if (!projects) {
+               fprintf(stderr, "Error opening projectsfile %s: %s (%d)\n",
+diff --git a/shared.c b/shared.c
+index 8e5ae48..e732064 100644
+--- a/shared.c
++++ b/shared.c
+@@ -28,8 +28,8 @@ int chk_positive(int result, char *msg)
+ int chk_non_negative(int result, char *msg)
+ {
+-      if (result < 0)
+-              die("%s: %s",msg, strerror(errno));
++      if (result < 0)
++              die("%s: %s", msg, strerror(errno));
+       return result;
+ }
+@@ -80,7 +80,7 @@ struct cgit_repo *cgit_get_repoinfo(const char *url)
+       int i;
+       struct cgit_repo *repo;
+-      for (i=0; i<cgit_repolist.count; i++) {
++      for (i = 0; i < cgit_repolist.count; i++) {
+               repo = &cgit_repolist.repos[i];
+               if (!strcmp(repo->url, url))
+                       return repo;
+@@ -108,7 +108,7 @@ char *trim_end(const char *str, char c)
+       if (str == NULL)
+               return NULL;
+       len = strlen(str);
+-      while(len > 0 && str[len - 1] == c)
++      while (len > 0 && str[len - 1] == c)
+               len--;
+       if (len == 0)
+               return NULL;
+@@ -207,7 +207,7 @@ static int load_mmfile(mmfile_t *file, const unsigned char *sha1)
+               file->ptr = (char *)"";
+               file->size = 0;
+       } else {
+-              file->ptr = read_sha1_file(sha1, &type, 
++              file->ptr = read_sha1_file(sha1, &type,
+                                          (unsigned long *)&file->size);
+       }
+       return 1;
+@@ -307,7 +307,7 @@ void cgit_diff_tree(const unsigned char *old_sha1,
+                   filepair_fn fn, const char *prefix, int ignorews)
+ {
+       struct diff_options opt;
+-      int prefixlen;
++      struct pathspec_item item;
+       diff_setup(&opt);
+       opt.output_format = DIFF_FORMAT_CALLBACK;
+@@ -319,10 +319,10 @@ void cgit_diff_tree(const unsigned char *old_sha1,
+       opt.format_callback = cgit_diff_tree_cb;
+       opt.format_callback_data = fn;
+       if (prefix) {
+-              opt.nr_paths = 1;
+-              opt.paths = &prefix;
+-              prefixlen = strlen(prefix);
+-              opt.pathlens = &prefixlen;
++              item.match = prefix;
++              item.len = strlen(prefix);
++              opt.pathspec.nr = 1;
++              opt.pathspec.items = &item;
+       }
+       diff_setup_done(&opt);
+@@ -351,17 +351,17 @@ int cgit_parse_snapshots_mask(const char *str)
+       int tl, sl, rv = 0;
+       /* favor legacy setting */
+-      if(atoi(str))
++      if (atoi(str))
+               return 1;
+-      for(;;) {
+-              str += strspn(str,delim);
+-              tl = strcspn(str,delim);
++      for (;;) {
++              str += strspn(str, delim);
++              tl = strcspn(str, delim);
+               if (!tl)
+                       break;
+               for (f = cgit_snapshot_formats; f->suffix; f++) {
+                       sl = strlen(f->suffix);
+-                      if((tl == sl && !strncmp(f->suffix, str, tl)) ||
+-                         (tl == sl-1 && !strncmp(f->suffix+1, str, tl-1))) {
++                      if ((tl == sl && !strncmp(f->suffix, str, tl)) ||
++                         (tl == sl - 1 && !strncmp(f->suffix + 1, str, tl - 1))) {
+                               rv |= f->bit;
+                               break;
+                       }
+diff --git a/tests/t0101-index.sh b/tests/t0101-index.sh
+index 573a351..ab63aca 100755
+--- a/tests/t0101-index.sh
++++ b/tests/t0101-index.sh
+@@ -5,14 +5,14 @@
+ prepare_tests "Check content on index page"
+ run_test 'generate index page' 'cgit_url "" >trash/tmp'
+-run_test 'find foo repo' 'grep -e "foo" trash/tmp'
+-run_test 'find foo description' 'grep -e "\[no description\]" trash/tmp'
+-run_test 'find bar repo' 'grep -e "bar" trash/tmp'
+-run_test 'find bar description' 'grep -e "the bar repo" trash/tmp'
+-run_test 'find foo+bar repo' 'grep -e ">foo+bar<" trash/tmp'
+-run_test 'verify foo+bar link' 'grep -e "/foo+bar/" trash/tmp'
+-run_test 'verify "with%20space" link' 'grep -e "/with%20space/" trash/tmp'
+-run_test 'no tree-link' '! grep -e "foo/tree" trash/tmp'
+-run_test 'no log-link' '! grep -e "foo/log" trash/tmp'
++run_test 'find foo repo' 'grep "foo" trash/tmp'
++run_test 'find foo description' 'grep "\[no description\]" trash/tmp'
++run_test 'find bar repo' 'grep "bar" trash/tmp'
++run_test 'find bar description' 'grep "the bar repo" trash/tmp'
++run_test 'find foo+bar repo' 'grep ">foo+bar<" trash/tmp'
++run_test 'verify foo+bar link' 'grep "/foo+bar/" trash/tmp'
++run_test 'verify "with%20space" link' 'grep "/with%20space/" trash/tmp'
++run_test 'no tree-link' '! grep "foo/tree" trash/tmp'
++run_test 'no log-link' '! grep "foo/log" trash/tmp'
+ tests_done
+diff --git a/tests/t0102-summary.sh b/tests/t0102-summary.sh
+index f299c5a..f778cb4 100755
+--- a/tests/t0102-summary.sh
++++ b/tests/t0102-summary.sh
+@@ -5,22 +5,22 @@
+ prepare_tests "Check content on summary page"
+ run_test 'generate foo summary' 'cgit_url "foo" >trash/tmp'
+-run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
+-run_test 'find commit 5' 'grep -e "commit 5" trash/tmp'
+-run_test 'find branch master' 'grep -e "master" trash/tmp'
+-run_test 'no tags' '! grep -e "tags" trash/tmp'
++run_test 'find commit 1' 'grep "commit 1" trash/tmp'
++run_test 'find commit 5' 'grep "commit 5" trash/tmp'
++run_test 'find branch master' 'grep "master" trash/tmp'
++run_test 'no tags' '! grep "tags" trash/tmp'
+ run_test 'clone-url expanded correctly' '
+-      grep -e "git://example.org/foo.git" trash/tmp
++      grep "git://example.org/foo.git" trash/tmp
+ '
+ run_test 'generate bar summary' 'cgit_url "bar" >trash/tmp'
+-run_test 'no commit 45' '! grep -e "commit 45" trash/tmp'
+-run_test 'find commit 46' 'grep -e "commit 46" trash/tmp'
+-run_test 'find commit 50' 'grep -e "commit 50" trash/tmp'
+-run_test 'find branch master' 'grep -e "master" trash/tmp'
+-run_test 'no tags' '! grep -e "tags" trash/tmp'
++run_test 'no commit 45' '! grep "commit 45" trash/tmp'
++run_test 'find commit 46' 'grep "commit 46" trash/tmp'
++run_test 'find commit 50' 'grep "commit 50" trash/tmp'
++run_test 'find branch master' 'grep "master" trash/tmp'
++run_test 'no tags' '! grep "tags" trash/tmp'
+ run_test 'clone-url expanded correctly' '
+-      grep -e "git://example.org/bar.git" trash/tmp
++      grep "git://example.org/bar.git" trash/tmp
+ '
+ tests_done
+diff --git a/tests/t0103-log.sh b/tests/t0103-log.sh
+index 7fa6754..67fcba0 100755
+--- a/tests/t0103-log.sh
++++ b/tests/t0103-log.sh
+@@ -5,21 +5,21 @@
+ prepare_tests "Check content on log page"
+ run_test 'generate foo/log' 'cgit_url "foo/log" >trash/tmp'
+-run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
+-run_test 'find commit 5' 'grep -e "commit 5" trash/tmp'
++run_test 'find commit 1' 'grep "commit 1" trash/tmp'
++run_test 'find commit 5' 'grep "commit 5" trash/tmp'
+ run_test 'generate bar/log' 'cgit_url "bar/log" >trash/tmp'
+-run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
+-run_test 'find commit 50' 'grep -e "commit 50" trash/tmp'
++run_test 'find commit 1' 'grep "commit 1" trash/tmp'
++run_test 'find commit 50' 'grep "commit 50" trash/tmp'
+ run_test 'generate "with%20space/log?qt=grep&q=commit+1"' '
+       cgit_url "with+space/log&qt=grep&q=commit+1" >trash/tmp
+ '
+-run_test 'find commit 1' 'grep -e "commit 1" trash/tmp'
+-run_test 'find link with %20 in path' 'grep -e "/with%20space/log/?qt=grep" trash/tmp'
+-run_test 'find link with + in arg' 'grep -e "/log/?qt=grep&amp;q=commit+1" trash/tmp'
+-run_test 'no links with space in path' '! grep -e "href=./with space/" trash/tmp'
+-run_test 'no links with space in arg' '! grep -e "q=commit 1" trash/tmp'
+-run_test 'commit 2 is not visible' '! grep -e "commit 2" trash/tmp'
++run_test 'find commit 1' 'grep "commit 1" trash/tmp'
++run_test 'find link with %20 in path' 'grep "/with%20space/log/?qt=grep" trash/tmp'
++run_test 'find link with + in arg' 'grep "/log/?qt=grep&amp;q=commit+1" trash/tmp'
++run_test 'no links with space in path' '! grep "href=./with space/" trash/tmp'
++run_test 'no links with space in arg' '! grep "q=commit 1" trash/tmp'
++run_test 'commit 2 is not visible' '! grep "commit 2" trash/tmp'
+ tests_done
+diff --git a/tests/t0104-tree.sh b/tests/t0104-tree.sh
+index 2ce1251..7aa3b8d 100755
+--- a/tests/t0104-tree.sh
++++ b/tests/t0104-tree.sh
+@@ -5,29 +5,29 @@
+ prepare_tests "Check content on tree page"
+ run_test 'generate bar/tree' 'cgit_url "bar/tree" >trash/tmp'
+-run_test 'find file-1' 'grep -e "file-1" trash/tmp'
+-run_test 'find file-50' 'grep -e "file-50" trash/tmp'
++run_test 'find file-1' 'grep "file-1" trash/tmp'
++run_test 'find file-50' 'grep "file-50" trash/tmp'
+ run_test 'generate bar/tree/file-50' 'cgit_url "bar/tree/file-50" >trash/tmp'
+ run_test 'find line 1' '
+-      grep -e "<a class=.no. id=.n1. name=.n1. href=.#n1.>1</a>" trash/tmp
++      grep "<a class=.no. id=.n1. name=.n1. href=.#n1.>1</a>" trash/tmp
+ '
+ run_test 'no line 2' '
+-      ! grep -e "<a class=.no. id=.n2. name=.n2. href=.#n2.>2</a>" trash/tmp
++      ! grep "<a class=.no. id=.n2. name=.n2. href=.#n2.>2</a>" trash/tmp
+ '
+ run_test 'generate foo+bar/tree' 'cgit_url "foo%2bbar/tree" >trash/tmp'
+ run_test 'verify a+b link' '
+-      grep -e "/foo+bar/tree/a+b" trash/tmp
++      grep "/foo+bar/tree/a+b" trash/tmp
+ '
+ run_test 'generate foo+bar/tree?h=1+2' 'cgit_url "foo%2bbar/tree&h=1%2b2" >trash/tmp'
+ run_test 'verify a+b?h=1+2 link' '
+-      grep -e "/foo+bar/tree/a+b?h=1%2b2" trash/tmp
++      grep "/foo+bar/tree/a+b?h=1%2b2" trash/tmp
+ '
+ tests_done
+diff --git a/tests/t0105-commit.sh b/tests/t0105-commit.sh
+index ae794c8..31b554b 100755
+--- a/tests/t0105-commit.sh
++++ b/tests/t0105-commit.sh
+@@ -5,33 +5,33 @@
+ prepare_tests "Check content on commit page"
+ run_test 'generate foo/commit' 'cgit_url "foo/commit" >trash/tmp'
+-run_test 'find tree link' 'grep -e "<a href=./foo/tree/.>" trash/tmp'
++run_test 'find tree link' 'grep "<a href=./foo/tree/.>" trash/tmp'
+ run_test 'find parent link' 'grep -E "<a href=./foo/commit/\?id=.+>" trash/tmp'
+ run_test 'find commit subject' '
+-      grep -e "<div class=.commit-subject.>commit 5<" trash/tmp
++      grep "<div class=.commit-subject.>commit 5<" trash/tmp
+ '
+-run_test 'find commit msg' 'grep -e "<div class=.commit-msg.></div>" trash/tmp'
+-run_test 'find diffstat' 'grep -e "<table summary=.diffstat. class=.diffstat.>" trash/tmp'
++run_test 'find commit msg' 'grep "<div class=.commit-msg.></div>" trash/tmp'
++run_test 'find diffstat' 'grep "<table summary=.diffstat. class=.diffstat.>" trash/tmp'
+ run_test 'find diff summary' '
+-       grep -e "1 files changed, 1 insertions, 0 deletions" trash/tmp
++      grep "1 files changed, 1 insertions, 0 deletions" trash/tmp
+ '
+ run_test 'get root commit' '
+-       root=$(cd trash/repos/foo && git rev-list --reverse HEAD | head -1) &&
+-       cgit_url "foo/commit&id=$root" >trash/tmp &&
+-       grep "</html>" trash/tmp
++      root=$(cd trash/repos/foo && git rev-list --reverse HEAD | head -1) &&
++      cgit_url "foo/commit&id=$root" >trash/tmp &&
++      grep "</html>" trash/tmp
+ '
+ run_test 'root commit contains diffstat' '
+-       grep "<a href=./foo/diff/file-1.id=[0-9a-f]\{40\}.>file-1</a>" trash/tmp
++      grep "<a href=./foo/diff/file-1.id=[0-9a-f]\{40\}.>file-1</a>" trash/tmp
+ '
+ run_test 'root commit contains diff' '
+-       grep ">diff --git a/file-1 b/file-1<" trash/tmp &&
+-       grep -e "<div class=.add.>+1</div>" trash/tmp
++      grep ">diff --git a/file-1 b/file-1<" trash/tmp &&
++      grep "<div class=.add.>+1</div>" trash/tmp
+ '
+ tests_done
+diff --git a/tests/t0106-diff.sh b/tests/t0106-diff.sh
+index e140bcc..eee0c8c 100755
+--- a/tests/t0106-diff.sh
++++ b/tests/t0106-diff.sh
+@@ -5,16 +5,16 @@
+ prepare_tests "Check content on diff page"
+ run_test 'generate foo/diff' 'cgit_url "foo/diff" >trash/tmp'
+-run_test 'find diff header' 'grep -e "a/file-5 b/file-5" trash/tmp'
+-run_test 'find blob link' 'grep -e "<a href=./foo/tree/file-5?id=" trash/tmp'
+-run_test 'find added file' 'grep -e "new file mode 100644" trash/tmp'
++run_test 'find diff header' 'grep "a/file-5 b/file-5" trash/tmp'
++run_test 'find blob link' 'grep "<a href=./foo/tree/file-5?id=" trash/tmp'
++run_test 'find added file' 'grep "new file mode 100644" trash/tmp'
+ run_test 'find hunk header' '
+-      grep -e "<div class=.hunk.>@@ -0,0 +1 @@</div>" trash/tmp
++      grep "<div class=.hunk.>@@ -0,0 +1 @@</div>" trash/tmp
+ '
+ run_test 'find added line' '
+-      grep -e "<div class=.add.>+5</div>" trash/tmp
++      grep "<div class=.add.>+5</div>" trash/tmp
+ '
+ tests_done
+diff --git a/tests/t0107-snapshot.sh b/tests/t0107-snapshot.sh
+index 8ab4912..132d2e9 100755
+--- a/tests/t0107-snapshot.sh
++++ b/tests/t0107-snapshot.sh
+@@ -10,17 +10,20 @@ run_test 'get foo/snapshot/master.tar.gz' '
+ run_test 'check html headers' '
+       head -n 1 trash/tmp |
+-           grep -e "Content-Type: application/x-gzip" &&
++      grep "Content-Type: application/x-gzip" &&
+       head -n 2 trash/tmp |
+-           grep -e "Content-Disposition: inline; filename=.master.tar.gz."
++      grep "Content-Disposition: inline; filename=.master.tar.gz."
+ '
+ run_test 'strip off the header lines' '
+-       tail -n +6 trash/tmp > trash/master.tar.gz
++      tail -n +6 trash/tmp > trash/master.tar.gz
++'
++
++run_test 'verify gzip format' '
++      gunzip --test trash/master.tar.gz
+ '
+-run_test 'verify gzip format' 'gunzip --test trash/master.tar.gz'
+ run_test 'untar' '
+       rm -rf trash/master &&
+       tar -xf trash/master.tar.gz -C trash
+@@ -32,7 +35,42 @@ run_test 'count files' '
+ '
+ run_test 'verify untarred file-5' '
+-       grep -e "^5$" trash/master/file-5 &&
++      grep "^5$" trash/master/file-5 &&
++      test $(cat trash/master/file-5 | wc -l) = 1
++'
++
++run_test 'get foo/snapshot/master.zip' '
++      cgit_url "foo/snapshot/master.zip" >trash/tmp
++'
++
++run_test 'check HTML headers (zip)' '
++      head -n 1 trash/tmp |
++      grep "Content-Type: application/x-zip" &&
++
++      head -n 2 trash/tmp |
++      grep "Content-Disposition: inline; filename=.master.zip."
++'
++
++run_test 'strip off the header lines (zip)' '
++      tail -n +6 trash/tmp >trash/master.zip
++'
++
++run_test 'verify zip format' '
++      unzip -t trash/master.zip
++'
++
++run_test 'unzip' '
++      rm -rf trash/master &&
++      unzip trash/master.zip -d trash
++'
++
++run_test 'count files (zip)' '
++      c=$(ls -1 trash/master/ | wc -l) &&
++      test $c = 5
++'
++
++run_test 'verify unzipped file-5' '
++       grep "^5$" trash/master/file-5 &&
+        test $(cat trash/master/file-5 | wc -l) = 1
+ '
+diff --git a/tests/t0108-patch.sh b/tests/t0108-patch.sh
+index 6ee70b3..f92f69c 100755
+--- a/tests/t0108-patch.sh
++++ b/tests/t0108-patch.sh
+@@ -9,19 +9,19 @@ run_test 'generate foo/patch' '
+ '
+ run_test 'find `From:` line' '
+-      grep -e "^From: " trash/tmp
++      grep "^From: " trash/tmp
+ '
+ run_test 'find `Date:` line' '
+-      grep -e "^Date: " trash/tmp
++      grep "^Date: " trash/tmp
+ '
+ run_test 'find `Subject:` line' '
+-      grep -e "^Subject: commit 5" trash/tmp
++      grep "^Subject: commit 5" trash/tmp
+ '
+ run_test 'find `cgit` signature' '
+-       tail -1 trash/tmp | grep -e "^cgit"
++      tail -1 trash/tmp | grep "^cgit"
+ '
+ run_test 'find initial commit' '
+@@ -33,7 +33,7 @@ run_test 'generate patch for initial commit' '
+ '
+ run_test 'find `cgit` signature' '
+-      tail -1 trash/tmp | grep -e "^cgit"
++      tail -1 trash/tmp | grep "^cgit"
+ '
+ tests_done
+diff --git a/ui-blob.c b/ui-blob.c
+index ec435e1..c59fbcb 100644
+--- a/ui-blob.c
++++ b/ui-blob.c
+@@ -11,17 +11,22 @@
+ #include "html.h"
+ #include "ui-shared.h"
+-static char *match_path;
+-static unsigned char *matched_sha1;
+-static int found_path;
++struct walk_tree_context {
++      char *match_path;
++      unsigned char *matched_sha1;
++      int found_path;
++};
+-static int walk_tree(const unsigned char *sha1, const char *base,int baselen,
+-      const char *pathname, unsigned mode, int stage, void *cbdata) {
+-      if(strncmp(base,match_path,baselen)
+-              || strcmp(match_path+baselen,pathname) )
++static int walk_tree(const unsigned char *sha1, const char *base, int baselen,
++      const char *pathname, unsigned mode, int stage, void *cbdata)
++{
++      struct walk_tree_context *walk_tree_ctx = cbdata;
++
++      if (strncmp(base, walk_tree_ctx->match_path, baselen)
++              || strcmp(walk_tree_ctx->match_path + baselen, pathname))
+               return READ_TREE_RECURSIVE;
+-      memmove(matched_sha1,sha1,20);
+-      found_path = 1;
++      memmove(walk_tree_ctx->matched_sha1, sha1, 20);
++      walk_tree_ctx->found_path = 1;
+       return 0;
+ }
+@@ -32,17 +37,27 @@ int cgit_print_file(char *path, const char *head)
+       char *buf;
+       unsigned long size;
+       struct commit *commit;
+-      const char *paths[] = {path, NULL};
++      struct pathspec_item path_items = {
++              .match = path,
++              .len = strlen(path)
++      };
++      struct pathspec paths = {
++              .nr = 1,
++              .items = &path_items
++      };
++      struct walk_tree_context walk_tree_ctx = {
++              .match_path = path,
++              .matched_sha1 = sha1,
++              .found_path = 0
++      };
++
+       if (get_sha1(head, sha1))
+               return -1;
+       type = sha1_object_info(sha1, &size);
+-      if(type == OBJ_COMMIT && path) {
++      if (type == OBJ_COMMIT && path) {
+               commit = lookup_commit_reference(sha1);
+-              match_path = path;
+-              matched_sha1 = sha1;
+-              found_path = 0;
+-              read_tree_recursive(commit->tree, "", 0, 0, paths, walk_tree, NULL);
+-              if (!found_path)
++              read_tree_recursive(commit->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
++              if (!walk_tree_ctx.found_path)
+                       return -1;
+               type = sha1_object_info(sha1, &size);
+       }
+@@ -63,15 +78,26 @@ void cgit_print_blob(const char *hex, char *path, const char *head)
+       char *buf;
+       unsigned long size;
+       struct commit *commit;
+-      const char *paths[] = {path, NULL};
++      struct pathspec_item path_items = {
++              .match = path,
++              .len = strlen(path)
++      };
++      struct pathspec paths = {
++              .nr = 1,
++              .items = &path_items
++      };
++      struct walk_tree_context walk_tree_ctx = {
++              .match_path = path,
++              .matched_sha1 = sha1,
++      };
+       if (hex) {
+-              if (get_sha1_hex(hex, sha1)){
++              if (get_sha1_hex(hex, sha1)) {
+                       cgit_print_error(fmt("Bad hex value: %s", hex));
+                       return;
+               }
+       } else {
+-              if (get_sha1(head,sha1)) {
++              if (get_sha1(head, sha1)) {
+                       cgit_print_error(fmt("Bad ref: %s", head));
+                       return;
+               }
+@@ -79,11 +105,9 @@ void cgit_print_blob(const char *hex, char *path, const char *head)
+       type = sha1_object_info(sha1, &size);
+-      if((!hex) && type == OBJ_COMMIT && path) {
++      if ((!hex) && type == OBJ_COMMIT && path) {
+               commit = lookup_commit_reference(sha1);
+-              match_path = path;
+-              matched_sha1 = sha1;
+-              read_tree_recursive(commit->tree, "", 0, 0, paths, walk_tree, NULL);
++              read_tree_recursive(commit->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
+               type = sha1_object_info(sha1,&size);
+       }
+diff --git a/ui-clone.c b/ui-clone.c
+index 81e7a4e..fdea24f 100644
+--- a/ui-clone.c
++++ b/ui-clone.c
+@@ -19,12 +19,10 @@ static int print_ref_info(const char *refname, const unsigned char *sha1,
+       if (!(obj = parse_object(sha1)))
+               return 0;
+-      if (!strcmp(refname, "HEAD") || !prefixcmp(refname, "refs/heads/"))
+-              htmlf("%s\t%s\n", sha1_to_hex(sha1), refname);
+-      else if (!prefixcmp(refname, "refs/tags") && obj->type == OBJ_TAG) {
++      htmlf("%s\t%s\n", sha1_to_hex(sha1), refname);
++      if (obj->type == OBJ_TAG) {
+               if (!(obj = deref_tag(obj, refname, 0)))
+                       return 0;
+-              htmlf("%s\t%s\n", sha1_to_hex(sha1), refname);
+               htmlf("%s\t%s^{}\n", sha1_to_hex(obj->sha1), refname);
+       }
+       return 0;
+diff --git a/ui-commit.c b/ui-commit.c
+index 536a8e8..74f37c8 100644
+--- a/ui-commit.c
++++ b/ui-commit.c
+@@ -39,7 +39,7 @@ void cgit_print_commit(char *hex, const char *prefix)
+       format_note(NULL, sha1, &notes, PAGE_ENCODING, 0);
+       load_ref_decorations(DECORATE_FULL_REFS);
+-      
++
+       cgit_print_diff_ctrls();
+       html("<table summary='commit info' class='commit-info'>\n");
+       html("<tr><th>author</th><td>");
+@@ -75,7 +75,7 @@ void cgit_print_commit(char *hex, const char *prefix)
+               cgit_tree_link(prefix, NULL, NULL, ctx.qry.head, tmp, prefix);
+       }
+       html("</td></tr>\n");
+-              for (p = commit->parents; p ; p = p->next) {
++      for (p = commit->parents; p; p = p->next) {
+               parent = lookup_commit_reference(p->item->object.sha1);
+               if (!parent) {
+                       html("<tr><td colspan='3'>");
+diff --git a/ui-diff.c b/ui-diff.c
+index 3d46da2..49e5b46 100644
+--- a/ui-diff.c
++++ b/ui-diff.c
+@@ -184,7 +184,7 @@ void cgit_print_diffstat(const unsigned char *old_sha1,
+       max_changes = 0;
+       cgit_diff_tree(old_sha1, new_sha1, inspect_filepair, prefix,
+                      ctx.qry.ignorews);
+-      for(i = 0; i<files; i++)
++      for (i = 0; i<files; i++)
+               print_fileinfo(&items[i]);
+       html("</table>");
+       html("<div class='diffstat-summary'>");
+diff --git a/ui-plain.c b/ui-plain.c
+index 85877d7..8ef4ec6 100644
+--- a/ui-plain.c
++++ b/ui-plain.c
+@@ -11,8 +11,10 @@
+ #include "html.h"
+ #include "ui-shared.h"
+-int match_baselen;
+-int match;
++struct walk_tree_context {
++      int match_baselen;
++      int match;
++};
+ static char *get_mimetype_from_file(const char *filename, const char *ext)
+ {
+@@ -54,7 +56,7 @@ static char *get_mimetype_from_file(const char *filename, const char *ext)
+       return result;
+ }
+-static void print_object(const unsigned char *sha1, const char *path)
++static int print_object(const unsigned char *sha1, const char *path)
+ {
+       enum object_type type;
+       char *buf, *ext;
+@@ -65,13 +67,13 @@ static void print_object(const unsigned char *sha1, const char *path)
+       type = sha1_object_info(sha1, &size);
+       if (type == OBJ_BAD) {
+               html_status(404, "Not found", 0);
+-              return;
++              return 0;
+       }
+       buf = read_sha1_file(sha1, &type, &size);
+       if (!buf) {
+               html_status(404, "Not found", 0);
+-              return;
++              return 0;
+       }
+       ctx.page.mimetype = NULL;
+       ext = strrchr(path, '.');
+@@ -97,9 +99,9 @@ static void print_object(const unsigned char *sha1, const char *path)
+       ctx.page.etag = sha1_to_hex(sha1);
+       cgit_print_http_headers(&ctx);
+       html_raw(buf, size);
+-      match = 1;
+       if (freemime)
+               free(ctx.page.mimetype);
++      return 1;
+ }
+ static char *buildpath(const char *base, int baselen, const char *path)
+@@ -138,7 +140,6 @@ static void print_dir(const unsigned char *sha1, const char *base,
+                               fullpath);
+               html("</li>\n");
+       }
+-      match = 2;
+ }
+ static void print_dir_entry(const unsigned char *sha1, const char *base,
+@@ -156,7 +157,6 @@ static void print_dir_entry(const unsigned char *sha1, const char *base,
+               cgit_plain_link(path, NULL, NULL, ctx.qry.head, ctx.qry.sha1,
+                               fullpath);
+       html("</li>\n");
+-      match = 2;
+ }
+ static void print_dir_tail(void)
+@@ -168,18 +168,23 @@ static int walk_tree(const unsigned char *sha1, const char *base, int baselen,
+                    const char *pathname, unsigned mode, int stage,
+                    void *cbdata)
+ {
+-      if (baselen == match_baselen) {
+-              if (S_ISREG(mode))
+-                      print_object(sha1, pathname);
+-              else if (S_ISDIR(mode)) {
++      struct walk_tree_context *walk_tree_ctx = cbdata;
++
++      if (baselen == walk_tree_ctx->match_baselen) {
++              if (S_ISREG(mode)) {
++                      if (print_object(sha1, pathname))
++                              walk_tree_ctx->match = 1;
++              } else if (S_ISDIR(mode)) {
+                       print_dir(sha1, base, baselen, pathname);
++                      walk_tree_ctx->match = 2;
+                       return READ_TREE_RECURSIVE;
+               }
+-      }
+-      else if (baselen > match_baselen)
++      } else if (baselen > walk_tree_ctx->match_baselen) {
+               print_dir_entry(sha1, base, baselen, pathname, mode);
+-      else if (S_ISDIR(mode))
++              walk_tree_ctx->match = 2;
++      } else if (S_ISDIR(mode)) {
+               return READ_TREE_RECURSIVE;
++      }
+       return 0;
+ }
+@@ -197,7 +202,17 @@ void cgit_print_plain(struct cgit_context *ctx)
+       const char *rev = ctx->qry.sha1;
+       unsigned char sha1[20];
+       struct commit *commit;
+-      const char *paths[] = {ctx->qry.path, NULL};
++      struct pathspec_item path_items = {
++              .match = ctx->qry.path,
++              .len = ctx->qry.path ? strlen(ctx->qry.path) : 0
++      };
++      struct pathspec paths = {
++              .nr = 1,
++              .items = &path_items
++      };
++      struct walk_tree_context walk_tree_ctx = {
++              .match = 0
++      };
+       if (!rev)
+               rev = ctx->qry.head;
+@@ -211,16 +226,17 @@ void cgit_print_plain(struct cgit_context *ctx)
+               html_status(404, "Not found", 0);
+               return;
+       }
+-      if (!paths[0]) {
+-              paths[0] = "";
+-              match_baselen = -1;
++      if (!path_items.match) {
++              path_items.match = "";
++              walk_tree_ctx.match_baselen = -1;
+               print_dir(commit->tree->object.sha1, "", 0, "");
++              walk_tree_ctx.match = 2;
+       }
+       else
+-              match_baselen = basedir_len(paths[0]);
+-      read_tree_recursive(commit->tree, "", 0, 0, paths, walk_tree, NULL);
+-      if (!match)
++              walk_tree_ctx.match_baselen = basedir_len(path_items.match);
++      read_tree_recursive(commit->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
++      if (!walk_tree_ctx.match)
+               html_status(404, "Not found", 0);
+-      else if (match == 2)
++      else if (walk_tree_ctx.match == 2)
+               print_dir_tail();
+ }
+diff --git a/ui-refs.c b/ui-refs.c
+index caddfbc..ce06b08 100644
+--- a/ui-refs.c
++++ b/ui-refs.c
+@@ -200,7 +200,7 @@ void cgit_print_branches(int maxcount)
+               qsort(list.refs, maxcount, sizeof(*list.refs), cmp_ref_name);
+       }
+-      for(i=0; i<maxcount; i++)
++      for (i = 0; i < maxcount; i++)
+               print_branch(list.refs[i]);
+       if (maxcount < list.count)
+@@ -224,7 +224,7 @@ void cgit_print_tags(int maxcount)
+       else if (maxcount > list.count)
+               maxcount = list.count;
+       print_tag_header();
+-      for(i=0; i<maxcount; i++)
++      for (i = 0; i < maxcount; i++)
+               print_tag(list.refs[i]);
+       if (maxcount < list.count)
+diff --git a/ui-repolist.c b/ui-repolist.c
+index dead1bf..1ae22aa 100644
+--- a/ui-repolist.c
++++ b/ui-repolist.c
+@@ -110,12 +110,13 @@ void print_sort_header(const char *title, const char *sort)
+       htmlf("'>%s</a></th>", title);
+ }
+-void print_header(int columns)
++void print_header()
+ {
+       html("<tr class='nohover'>");
+       print_sort_header("Name", "name");
+       print_sort_header("Description", "desc");
+-      print_sort_header("Owner", "owner");
++      if (ctx.cfg.enable_index_owner)
++              print_sort_header("Owner", "owner");
+       print_sort_header("Idle", "idle");
+       if (ctx.cfg.enable_index_links)
+               html("<th class='left'>Links</th>");
+@@ -128,10 +129,10 @@ void print_pager(int items, int pagelen, char *search, char *sort)
+       int i, ofs;
+       char *class = NULL;
+       html("<div class='pager'>");
+-      for(i = 0, ofs = 0; ofs < items; i++, ofs = i * pagelen) {
++      for (i = 0, ofs = 0; ofs < items; i++, ofs = i * pagelen) {
+               class = (ctx.qry.ofs == ofs) ? "current" : NULL;
+-              cgit_index_link(fmt("[%d]", i+1), fmt("Page %d", i+1), class,
+-                              search, sort, ofs);
++              cgit_index_link(fmt("[%d]", i + 1), fmt("Page %d", i + 1),
++                              class, search, sort, ofs);
+       }
+       html("</div>");
+ }
+@@ -239,13 +240,15 @@ int sort_repolist(char *field)
+ void cgit_print_repolist()
+ {
+-      int i, columns = 4, hits = 0, header = 0;
++      int i, columns = 3, hits = 0, header = 0;
+       char *last_section = NULL;
+       char *section;
+       int sorted = 0;
+       if (ctx.cfg.enable_index_links)
+-              columns++;
++              ++columns;
++      if (ctx.cfg.enable_index_owner)
++              ++columns;
+       ctx.page.title = ctx.cfg.root_title;
+       cgit_print_http_headers(&ctx);
+@@ -255,13 +258,13 @@ void cgit_print_repolist()
+       if (ctx.cfg.index_header)
+               html_include(ctx.cfg.index_header);
+-      if(ctx.qry.sort)
++      if (ctx.qry.sort)
+               sorted = sort_repolist(ctx.qry.sort);
+       else if (ctx.cfg.section_sort)
+               sort_repolist("section");
+       html("<table summary='repository list' class='list nowrap'>");
+-      for (i=0; i<cgit_repolist.count; i++) {
++      for (i = 0; i < cgit_repolist.count; i++) {
+               ctx.repo = &cgit_repolist.repos[i];
+               if (!(is_match(ctx.repo) && is_in_url(ctx.repo)))
+                       continue;
+@@ -271,7 +274,7 @@ void cgit_print_repolist()
+               if (hits > ctx.qry.ofs + ctx.cfg.max_repo_count)
+                       continue;
+               if (!header++)
+-                      print_header(columns);
++                      print_header();
+               section = ctx.repo->section;
+               if (section && !strcmp(section, ""))
+                       section = NULL;
+@@ -294,8 +297,10 @@ void cgit_print_repolist()
+               html_ntxt(ctx.cfg.max_repodesc_len, ctx.repo->desc);
+               html_link_close();
+               html("</td><td>");
+-              html_txt(ctx.repo->owner);
+-              html("</td><td>");
++              if (ctx.cfg.enable_index_owner) {
++                      html_txt(ctx.repo->owner);
++                      html("</td><td>");
++              }
+               print_modtime(ctx.repo);
+               html("</td>");
+               if (ctx.cfg.enable_index_links) {
+diff --git a/ui-shared.c b/ui-shared.c
+index 75b97a1..af5310b 100644
+--- a/ui-shared.c
++++ b/ui-shared.c
+@@ -23,7 +23,7 @@ static char *http_date(time_t t)
+                "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"};
+       struct tm *tm = gmtime(&t);
+       return fmt("%s, %02d %s %04d %02d:%02d:%02d GMT", day[tm->tm_wday],
+-                 tm->tm_mday, month[tm->tm_mon], 1900+tm->tm_year,
++                 tm->tm_mday, month[tm->tm_mon], 1900 + tm->tm_year,
+                  tm->tm_hour, tm->tm_min, tm->tm_sec);
+ }
+@@ -93,7 +93,7 @@ char *cgit_fileurl(const char *reponame, const char *pagename,
+ char *cgit_pageurl(const char *reponame, const char *pagename,
+                  const char *query)
+ {
+-      return cgit_fileurl(reponame,pagename,0,query);
++      return cgit_fileurl(reponame, pagename, 0, query);
+ }
+ const char *cgit_repobasename(const char *reponame)
+@@ -102,21 +102,21 @@ const char *cgit_repobasename(const char *reponame)
+       static char rvbuf[1024];
+       int p;
+       const char *rv;
+-      strncpy(rvbuf,reponame,sizeof(rvbuf));
+-      if(rvbuf[sizeof(rvbuf)-1])
++      strncpy(rvbuf, reponame, sizeof(rvbuf));
++      if (rvbuf[sizeof(rvbuf)-1])
+               die("cgit_repobasename: truncated repository name '%s'", reponame);
+       p = strlen(rvbuf)-1;
+       /* strip trailing slashes */
+-      while(p && rvbuf[p]=='/') rvbuf[p--]=0;
++      while (p && rvbuf[p] == '/') rvbuf[p--] = 0;
+       /* strip trailing .git */
+-      if(p>=3 && !strncmp(&rvbuf[p-3],".git",4)) {
++      if (p >= 3 && !strncmp(&rvbuf[p-3], ".git", 4)) {
+               p -= 3; rvbuf[p--] = 0;
+       }
+       /* strip more trailing slashes if any */
+-      while( p && rvbuf[p]=='/') rvbuf[p--]=0;
++      while ( p && rvbuf[p] == '/') rvbuf[p--] = 0;
+       /* find last slash in the remaining string */
+       rv = strrchr(rvbuf,'/');
+-      if(rv)
++      if (rv)
+               return ++rv;
+       return rvbuf;
+ }
+@@ -499,7 +499,7 @@ void cgit_object_link(struct object *obj)
+       shortrev = xstrdup(fullrev);
+       shortrev[10] = '\0';
+       if (obj->type == OBJ_COMMIT) {
+-                cgit_commit_link(fmt("commit %s...", shortrev), NULL, NULL,
++              cgit_commit_link(fmt("commit %s...", shortrev), NULL, NULL,
+                                ctx.qry.head, fullrev, NULL, 0);
+               return;
+       } else if (obj->type == OBJ_TREE)
+@@ -564,6 +564,7 @@ void cgit_submodule_link(const char *class, char *path, const char *rev)
+       html("'>");
+       html_txt(path);
+       html("</a>");
++      html_txt(fmt(" @ %.7s", rev));
+       if (item && tail)
+               path[len - 1] = tail;
+ }
+@@ -575,7 +576,7 @@ void cgit_print_date(time_t secs, const char *format, int local_time)
+       if (!secs)
+               return;
+-      if(local_time)
++      if (local_time)
+               time = localtime(&secs);
+       else
+               time = gmtime(&secs);
+@@ -735,7 +736,7 @@ int print_archive_ref(const char *refname, const unsigned char *sha1,
+       if (prefixcmp(refname, "refs/archives"))
+               return 0;
+-      strncpy(buf, refname+14, sizeof(buf));
++      strncpy(buf, refname + 14, sizeof(buf));
+       obj = parse_object(sha1);
+       if (!obj)
+               return 1;
+@@ -967,7 +968,7 @@ void cgit_print_snapshot_links(const char *repo, const char *head,
+ {
+       const struct cgit_snapshot_format* f;
+       char *prefix;
+-      char *filename;
++      char *filename;
+       unsigned char sha1[20];
+       if (get_sha1(fmt("refs/tags/%s", hex), sha1) == 0 &&
+diff --git a/ui-snapshot.c b/ui-snapshot.c
+index 47432bd..54e659c 100644
+--- a/ui-snapshot.c
++++ b/ui-snapshot.c
+@@ -11,7 +11,32 @@
+ #include "html.h"
+ #include "ui-shared.h"
+-static int write_compressed_tar_archive(struct archiver_args *args, char *filter_argv[])
++static int write_archive_type(const char *format, const char *hex, const char *prefix)
++{
++      struct argv_array argv = ARGV_ARRAY_INIT;
++      argv_array_push(&argv, "snapshot");
++      argv_array_push(&argv, format);
++      if (prefix) {
++              argv_array_push(&argv, "--prefix");
++              argv_array_push(&argv, fmt("%s/", prefix));
++      }
++      argv_array_push(&argv, hex);
++      return write_archive(argv.argc, argv.argv, NULL, 1, NULL, 0);
++}
++
++static int write_tar_archive(const char *hex, const char *prefix)
++{
++      return write_archive_type("--format=tar", hex, prefix);
++}
++
++static int write_zip_archive(const char *hex, const char *prefix)
++{
++      return write_archive_type("--format=zip", hex, prefix);
++}
++
++static int write_compressed_tar_archive(const char *hex,
++                                      const char *prefix,
++                                      char *filter_argv[])
+ {
+       int rv;
+       struct cgit_filter f;
+@@ -19,27 +44,27 @@ static int write_compressed_tar_archive(struct archiver_args *args, char *filter
+       f.cmd = filter_argv[0];
+       f.argv = filter_argv;
+       cgit_open_filter(&f);
+-      rv = write_tar_archive(args);
++      rv = write_tar_archive(hex, prefix);
+       cgit_close_filter(&f);
+       return rv;
+ }
+-static int write_tar_gzip_archive(struct archiver_args *args)
++static int write_tar_gzip_archive(const char *hex, const char *prefix)
+ {
+       char *argv[] = { "gzip", "-n", NULL };
+-      return write_compressed_tar_archive(args, argv);
++      return write_compressed_tar_archive(hex, prefix, argv);
+ }
+-static int write_tar_bzip2_archive(struct archiver_args *args)
++static int write_tar_bzip2_archive(const char *hex, const char *prefix)
+ {
+       char *argv[] = { "bzip2", NULL };
+-      return write_compressed_tar_archive(args, argv);
++      return write_compressed_tar_archive(hex, prefix, argv);
+ }
+-static int write_tar_xz_archive(struct archiver_args *args)
++static int write_tar_xz_archive(const char *hex, const char *prefix)
+ {
+       char *argv[] = { "xz", NULL };
+-      return write_compressed_tar_archive(args, argv);
++      return write_compressed_tar_archive(hex, prefix, argv);
+ }
+ const struct cgit_snapshot_format cgit_snapshot_formats[] = {
+@@ -48,7 +73,7 @@ const struct cgit_snapshot_format cgit_snapshot_formats[] = {
+       { ".tar.bz2", "application/x-bzip2", write_tar_bzip2_archive, 0x04 },
+       { ".tar", "application/x-tar", write_tar_archive, 0x08 },
+       { ".tar.xz", "application/x-xz", write_tar_xz_archive, 0x10 },
+-      {}
++      { NULL }
+ };
+ static const struct cgit_snapshot_format *get_format(const char *filename)
+@@ -57,7 +82,7 @@ static const struct cgit_snapshot_format *get_format(const char *filename)
+       int fl, sl;
+       fl = strlen(filename);
+-      for(fmt = cgit_snapshot_formats; fmt->suffix; fmt++) {
++      for (fmt = cgit_snapshot_formats; fmt->suffix; fmt++) {
+               sl = strlen(fmt->suffix);
+               if (sl >= fl)
+                       continue;
+@@ -71,34 +96,20 @@ static int make_snapshot(const struct cgit_snapshot_format *format,
+                        const char *hex, const char *prefix,
+                        const char *filename)
+ {
+-      struct archiver_args args;
+-      struct commit *commit;
+       unsigned char sha1[20];
+-      if(get_sha1(hex, sha1)) {
++      if (get_sha1(hex, sha1)) {
+               cgit_print_error(fmt("Bad object id: %s", hex));
+               return 1;
+       }
+-      commit = lookup_commit_reference(sha1);
+-      if(!commit) {
++      if (!lookup_commit_reference(sha1)) {
+               cgit_print_error(fmt("Not a commit reference: %s", hex));
+               return 1;
+       }
+-      memset(&args, 0, sizeof(args));
+-      if (prefix) {
+-              args.base = fmt("%s/", prefix);
+-              args.baselen = strlen(prefix) + 1;
+-      } else {
+-              args.base = "";
+-              args.baselen = 0;
+-      }
+-      args.tree = commit->tree;
+-      args.time = commit->date;
+-      args.compression_level = Z_DEFAULT_COMPRESSION;
+       ctx.page.mimetype = xstrdup(format->mimetype);
+       ctx.page.filename = xstrdup(filename);
+       cgit_print_http_headers(&ctx);
+-      format->write_func(&args);
++      format->write_func(hex, prefix);
+       return 0;
+ }
+diff --git a/ui-ssdiff.c b/ui-ssdiff.c
+index 7108779..3d3dad6 100644
+--- a/ui-ssdiff.c
++++ b/ui-ssdiff.c
+@@ -138,9 +138,8 @@ static char *replace_tabs(char *line)
+                       strcat(result, prev_buf);
+                       break;
+               } else {
+-                      strcat(result, " ");
+-                      strncat(result, spaces, 8 - (strlen(result) % 8));
+                       strncat(result, prev_buf, cur_buf - prev_buf);
++                      strncat(result, spaces, 8 - (strlen(result) % 8));
+               }
+               prev_buf = cur_buf + 1;
+       }
+diff --git a/ui-stats.c b/ui-stats.c
+index 59f4c1e..9cf1dbd 100644
+--- a/ui-stats.c
++++ b/ui-stats.c
+@@ -23,21 +23,21 @@ static void trunc_week(struct tm *tm)
+ {
+       time_t t = timegm(tm);
+       t -= ((tm->tm_wday + 6) % 7) * DAY_SECS;
+-      gmtime_r(&t, tm);       
++      gmtime_r(&t, tm);
+ }
+ static void dec_week(struct tm *tm)
+ {
+       time_t t = timegm(tm);
+       t -= WEEK_SECS;
+-      gmtime_r(&t, tm);       
++      gmtime_r(&t, tm);
+ }
+ static void inc_week(struct tm *tm)
+ {
+       time_t t = timegm(tm);
+       t += WEEK_SECS;
+-      gmtime_r(&t, tm);       
++      gmtime_r(&t, tm);
+ }
+ static char *pretty_week(struct tm *tm)
+@@ -83,7 +83,7 @@ static char *pretty_month(struct tm *tm)
+ static void trunc_quarter(struct tm *tm)
+ {
+       trunc_month(tm);
+-      while(tm->tm_mon % 3 != 0)
++      while (tm->tm_mon % 3 != 0)
+               dec_month(tm);
+ }
+@@ -153,7 +153,7 @@ int cgit_find_stats_period(const char *expr, struct cgit_period **period)
+               if (periods[i].code == code || !strcmp(periods[i].name, expr)) {
+                       if (period)
+                               *period = &periods[i];
+-                      return i+1;
++                      return i + 1;
+               }
+       return 0;
+ }
+@@ -239,7 +239,7 @@ struct string_list collect_stats(struct cgit_context *ctx,
+       init_revisions(&rev, NULL);
+       rev.abbrev = DEFAULT_ABBREV;
+       rev.commit_format = CMIT_FMT_DEFAULT;
+-      rev.no_merges = 1;
++      rev.max_parents = 1;
+       rev.verbose_header = 1;
+       rev.show_root_diff = 0;
+       setup_revisions(argc, argv, &rev, NULL);
+diff --git a/ui-tag.c b/ui-tag.c
+index 39e4cb8..cab96b1 100644
+--- a/ui-tag.c
++++ b/ui-tag.c
+@@ -99,6 +99,6 @@ void cgit_print_tag(char *revname)
+               if (ctx.repo->snapshots)
+                       print_download_links(revname);
+               html("</table>\n");
+-        }
++      }
+       return;
+ }
+diff --git a/ui-tree.c b/ui-tree.c
+index b1adcc7..561f9e7 100644
+--- a/ui-tree.c
++++ b/ui-tree.c
+@@ -11,9 +11,11 @@
+ #include "html.h"
+ #include "ui-shared.h"
+-char *curr_rev;
+-char *match_path;
+-int header = 0;
++struct walk_tree_context {
++      char *curr_rev;
++      char *match_path;
++      int state;
++};
+ static void print_text_buffer(const char *name, char *buf, unsigned long size)
+ {
+@@ -27,10 +29,10 @@ static void print_text_buffer(const char *name, char *buf, unsigned long size)
+               html("<tr><td class='linenumbers'><pre>");
+               idx = 0;
+               lineno = 0;
+-      
++
+               if (size) {
+                       htmlf(numberfmt, ++lineno);
+-                      while(idx < size - 1) { // skip absolute last newline
++                      while (idx < size - 1) { // skip absolute last newline
+                               if (buf[idx] == '\n')
+                                       htmlf(numberfmt, ++lineno);
+                               idx++;
+@@ -84,7 +86,7 @@ static void print_binary_buffer(char *buf, unsigned long size)
+       html("</table>\n");
+ }
+-static void print_object(const unsigned char *sha1, char *path, const char *basename)
++static void print_object(const unsigned char *sha1, char *path, const char *basename, const char *rev)
+ {
+       enum object_type type;
+       char *buf;
+@@ -106,7 +108,7 @@ static void print_object(const unsigned char *sha1, char *path, const char *base
+       htmlf("blob: %s (", sha1_to_hex(sha1));
+       cgit_plain_link("plain", NULL, NULL, ctx.qry.head,
+-                      curr_rev, path);
++                      rev, path);
+       html(")\n");
+       if (ctx.cfg.max_blob_size && size / 1024 > ctx.cfg.max_blob_size) {
+@@ -126,6 +128,7 @@ static int ls_item(const unsigned char *sha1, const char *base, int baselen,
+                  const char *pathname, unsigned int mode, int stage,
+                  void *cbdata)
+ {
++      struct walk_tree_context *walk_tree_ctx = cbdata;
+       char *name;
+       char *fullpath;
+       char *class;
+@@ -153,7 +156,7 @@ static int ls_item(const unsigned char *sha1, const char *base, int baselen,
+               cgit_submodule_link("ls-mod", fullpath, sha1_to_hex(sha1));
+       } else if (S_ISDIR(mode)) {
+               cgit_tree_link(name, NULL, "ls-dir", ctx.qry.head,
+-                             curr_rev, fullpath);
++                             walk_tree_ctx->curr_rev, fullpath);
+       } else {
+               class = strrchr(name, '.');
+               if (class != NULL) {
+@@ -161,19 +164,20 @@ static int ls_item(const unsigned char *sha1, const char *base, int baselen,
+               } else
+                       class = "ls-blob";
+               cgit_tree_link(name, NULL, class, ctx.qry.head,
+-                             curr_rev, fullpath);
++                             walk_tree_ctx->curr_rev, fullpath);
+       }
+       htmlf("</td><td class='ls-size'>%li</td>", size);
+       html("<td>");
+-      cgit_log_link("log", NULL, "button", ctx.qry.head, curr_rev,
+-                    fullpath, 0, NULL, NULL, ctx.qry.showmsg);
++      cgit_log_link("log", NULL, "button", ctx.qry.head,
++                    walk_tree_ctx->curr_rev, fullpath, 0, NULL, NULL,
++                    ctx.qry.showmsg);
+       if (ctx.repo->max_stats)
+               cgit_stats_link("stats", NULL, "button", ctx.qry.head,
+                               fullpath);
+       if (!S_ISGITLINK(mode))
+-              cgit_plain_link("plain", NULL, "button", ctx.qry.head, curr_rev,
+-                              fullpath);
++              cgit_plain_link("plain", NULL, "button", ctx.qry.head,
++                              walk_tree_ctx->curr_rev, fullpath);
+       html("</td></tr>\n");
+       free(name);
+       return 0;
+@@ -188,20 +192,19 @@ static void ls_head()
+       html("<th class='right'>Size</th>");
+       html("<th/>");
+       html("</tr>\n");
+-      header = 1;
+ }
+ static void ls_tail()
+ {
+-      if (!header)
+-              return;
+       html("</table>\n");
+-      header = 0;
+ }
+-static void ls_tree(const unsigned char *sha1, char *path)
++static void ls_tree(const unsigned char *sha1, char *path, struct walk_tree_context *walk_tree_ctx)
+ {
+       struct tree *tree;
++      struct pathspec paths = {
++              .nr = 0
++      };
+       tree = parse_tree_indirect(sha1);
+       if (!tree) {
+@@ -211,7 +214,7 @@ static void ls_tree(const unsigned char *sha1, char *path)
+       }
+       ls_head();
+-      read_tree_recursive(tree, "", 0, 1, NULL, ls_item, NULL);
++      read_tree_recursive(tree, "", 0, 1, &paths, ls_item, walk_tree_ctx);
+       ls_tail();
+ }
+@@ -220,25 +223,25 @@ static int walk_tree(const unsigned char *sha1, const char *base, int baselen,
+                    const char *pathname, unsigned mode, int stage,
+                    void *cbdata)
+ {
+-      static int state;
++      struct walk_tree_context *walk_tree_ctx = cbdata;
+       static char buffer[PATH_MAX];
+-      if (state == 0) {
++      if (walk_tree_ctx->state == 0) {
+               memcpy(buffer, base, baselen);
+-              strcpy(buffer+baselen, pathname);
+-              if (strcmp(match_path, buffer))
++              strcpy(buffer + baselen, pathname);
++              if (strcmp(walk_tree_ctx->match_path, buffer))
+                       return READ_TREE_RECURSIVE;
+               if (S_ISDIR(mode)) {
+-                      state = 1;
++                      walk_tree_ctx->state = 1;
+                       ls_head();
+                       return READ_TREE_RECURSIVE;
+               } else {
+-                      print_object(sha1, buffer, pathname);
++                      print_object(sha1, buffer, pathname, walk_tree_ctx->curr_rev);
+                       return 0;
+               }
+       }
+-      ls_item(sha1, base, baselen, pathname, mode, stage, NULL);
++      ls_item(sha1, base, baselen, pathname, mode, stage, walk_tree_ctx);
+       return 0;
+ }
+@@ -252,12 +255,23 @@ void cgit_print_tree(const char *rev, char *path)
+ {
+       unsigned char sha1[20];
+       struct commit *commit;
+-      const char *paths[] = {path, NULL};
++      struct pathspec_item path_items = {
++              .match = path,
++              .len = path ? strlen(path) : 0
++      };
++      struct pathspec paths = {
++              .nr = path ? 1 : 0,
++              .items = &path_items
++      };
++      struct walk_tree_context walk_tree_ctx = {
++              .match_path = path,
++              .state = 0
++      };
+       if (!rev)
+               rev = ctx.qry.head;
+-      curr_rev = xstrdup(rev);
++      walk_tree_ctx.curr_rev = xstrdup(rev);
+       if (get_sha1(rev, sha1)) {
+               cgit_print_error(fmt("Invalid revision name: %s", rev));
+               return;
+@@ -269,11 +283,11 @@ void cgit_print_tree(const char *rev, char *path)
+       }
+       if (path == NULL) {
+-              ls_tree(commit->tree->object.sha1, NULL);
++              ls_tree(commit->tree->object.sha1, NULL, &walk_tree_ctx);
+               return;
+       }
+-      match_path = path;
+-      read_tree_recursive(commit->tree, "", 0, 0, paths, walk_tree, NULL);
+-      ls_tail();
++      read_tree_recursive(commit->tree, "", 0, 0, &paths, walk_tree, &walk_tree_ctx);
++      if (walk_tree_ctx.state == 1)
++              ls_tail();
+ }
diff --git a/notes.patch b/notes.patch
new file mode 100644 (file)
index 0000000..f54e429
--- /dev/null
@@ -0,0 +1,25 @@
+--- cgit-0.9.1/ui-log.c~       2012-11-15 01:28:59.000000000 +0100
++++ cgit-0.9.1/ui-log.c        2013-05-17 13:15:53.682385508 +0200
+@@ -195,9 +195,9 @@
+                               strbuf_addstr(&msgbuf, info->msg);
+                               strbuf_addch(&msgbuf, '\n');
+                       }
+-                      format_note(NULL, commit->object.sha1, &msgbuf,
++                      format_display_notes(commit->object.sha1, &msgbuf,
+                                   PAGE_ENCODING,
+-                                  NOTES_SHOW_HEADER | NOTES_INDENT);
++                                  0);
+                       strbuf_addch(&msgbuf, '\n');
+                       strbuf_ltrim(&msgbuf);
+               }
+--- cgit-0.9.1/ui-commit.c~    2013-05-17 13:09:43.468524461 +0200
++++ cgit-0.9.1/ui-commit.c     2013-05-17 13:17:12.059843403 +0200
+@@ -36,7 +36,7 @@
+       }
+       info = cgit_parse_commit(commit);
+-      format_note(NULL, sha1, &notes, PAGE_ENCODING, 0);
++      format_display_notes(sha1, &notes, PAGE_ENCODING, 1);
+       load_ref_decorations(DECORATE_FULL_REFS);
This page took 0.252537 seconds and 4 git commands to generate.