| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <1196728222-17300-1-git-send-email-hjemli@gmail.com> Date: Tue, 4 Dec 2007 01:30:22 +0100 From: Lars Hjemli <hjemli@...il.com> To: Jan Engelhardt <jengelh@...putergmbh.de> Cc: linux-kernel@...r.kernel.org, Lars Hjemli <hjemli@...il.com> Subject: cgit for kernel.org Jan Engelhardt wrote: > Repository overview. > CGIT: http://cgit.freedesktop.org/ > GITWEB: http://git.kernel.org/ Why not compare more equal pages? freedesktop.org also runs gitweb: http://gitweb.freedesktop.org > > Remarks: > cgit is lacking the links "shortlog", "log", "tree" and "git" > which gitweb prints for each repository line. Adding "enable-index-links=1" to /etc/cgitrc will print links to "summary", "log" and "tree" for each repository line, but you still have to open the summary page to get the git (clone) link. It wouldn't be very hard to add another link (see http://hjemli.net/git/cgit/tree/ui-repolist.c#n96), but I'm not sure what actual value such a link would provide. > > Inside a repository. > CGIT: http://cgit.freedesktop.org/hal/ The similar page in gitweb: http://gitweb.freedesktop.org/?p=hal.git;a=summary > GITWEB: >http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=summary The similar page in cgit: http://hjemli.net/git/linux/ > Remarks: > In cgit, the left gray panel went by quite unnoticed IMHO the sidebar makes cgit way easier to navigate (especially the dropdown box to switch between branches is pretty handy). > > Clicking on a commit. > CGIT: >http://cgit.freedesktop.org/hal/commit/?id=b8f72cec53415ebc2d32805b049dcc94cef4a854 Similar page in gitweb: http://gitweb.freedesktop.org/?p=hal.git;a=commit;h=b8f72cec53415ebc2d32805b049dcc94cef4a854 > GITWEB: >http://git.kernel.org/?p=linux/kernel/git/jgarzik/misc-2.6.git;a=commit;h=8c27eba54970c6ebbb408186e5baa2274435e869 Similar page in cgit: http://hjemli.net/git/linux/commit/?id=8c27eba54970c6ebbb408186e5baa2274435e869 > > Remarks: > cgit comes with a diffstat. Not that I look at diffstats all that > often anyway. Still need to click on "(diff)" go get a diff. There's always the link in the sidebar ;-) But seriously, showing the diff on the same page as the commit message would sometimes be nice and sometimes it would be a bit too much: http://hjemli.net/git/linux/commit/?id=b5faa4b89e4d83203b1f44f143a351b518f7cda2 http://git.kernel.org/?p=linux/kernel/git/jgarzik/misc-2.6.git;a=commit;h=b5faa4b89e4d83203b1f44f143a351b518f7cda2 > > Filenames are displayed with monospace font in gitweb :) > Clicking on the commit actually shows its diff in gitweb. I belive cgit and gitweb displays roughly the same information on their commit pages; I don't see a diff in gitweb either unless I click one of the diff- links. > > Final remark: > gitweb has the warmer colors. Yes, gitweb definitly looks more polished. It's a bit slow, though... Another remark - IMHO there are some practical benefits to be had from the virtual url support in cgit, e.g. linking to blobs: http://gitweb.freedesktop.org/?p=xorg/xserver.git;a=blob;h=ace11817054bfb5bcc8d55a31bd51b9dea49bbed;hb=fe25f897c62bb324660217e15dbd3091c808dbba;f=GL/mesa/X/Makefile.am http://cgit.freedesktop.org/xorg/xserver/tree/GL/mesa/X/Makefile.am -- larsh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists