Details
- Reviewers
wg kwm - Group Reviewers
gnome Contributor Reviewers (ports)
Diff Detail
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
at a first glance looks good to me but I would get review from someone else as well since I am not familiar with this port.
Like everything related to libgit2. This version is for libgit2 0.23 since it not API stable between 0.x releases.
The whole stack needs to be updated in one go. libgit2, libgit2-glib, gitg, py-pygit2 and rubygem-rugged. Like I did in https://svnweb.freebsd.org/changeset/ports/386103
Oh now I see the other ports. I think this would be more clear if there was 1 patch/review for all the ports instead of spreading them around in 1 review per patch. Since they depend quite closely with each other.
Do you want me to change it to one patch instead? I didn't know what the workflow for this situation and made one per port.
Approved if you add compiler:c11 to USES (at the beginning please, so it stays sorted) to deal with some code that gcc from base doesn't understand.