[RFCv2 PATCH 0/7] Add ui-blame

John Keeping john at keeping.me.uk
Sat Sep 23 17:53:40 CEST 2017


On Fri, Sep 22, 2017 at 10:38:41PM -0500, Jeff Smith wrote:
> I split git blame functionality into libgit, and the changes were
> accepted upstream and are a part of git 2.14.  Now that the git
> infrastructure is in place, here is what is needed for cgit to make use
> of it.
> 
> Jeff Smith (7):
>   ui-blame: create enable-blame config item
>   ui-blame: create framework
>   ui-blame: create links
>   ui-blame: html_ntxt with no ellipsis
>   ui-blame: pull blame info from libgit
>   ui-blame: begin building
>   ui-blame: generate blame page when requested

I still find the arrangement of this patch series a bit strange, I think
it should be more like:

- html: html_ntxt with no ellipsis
- ui-tree: move set_title_from_path to ui-shared
- ui-blame: add blame UI
	A squashed version of your "begin building" and "pull blame info
	from libgit"; splitting them up just makes it harder to review

	I think it also makes sense to squash the config, cmd and
	Makefile changes into this patch so that the feature springs
	into life fully formed, and then the following patches just link
	it into place
- ui-tree: link to blame UI if enabled


More information about the CGit mailing list