Page MenuHomeFreeBSD

deskutils/gitte: new port Gitte, a GNOME git desktop client
ClosedPublic

Authored by dch on Thu, May 28, 11:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 22, 12:48 AM
Unknown Object (File)
Sun, Jun 14, 12:21 AM
Unknown Object (File)
Thu, Jun 11, 8:25 AM
Unknown Object (File)
Thu, Jun 11, 4:48 AM
Unknown Object (File)
Mon, Jun 8, 2:38 PM
Unknown Object (File)
Mon, Jun 8, 2:32 PM
Unknown Object (File)
Mon, Jun 8, 2:29 PM
Unknown Object (File)
Mon, Jun 8, 2:23 PM
Subscribers
None

Details

Summary

Sponsored by: SkunkWerks, GmbH

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 73502
Build 70385: arc lint + arc unit

Event Timeline

dch requested review of this revision.Thu, May 28, 11:26 AM
dch created this revision.

Apart from the following stage-qa warning this looks great

Error: /usr/local/bin/gitte is linked to /usr/local/lib/libcairo.so.2 from graphics/cairo but it is not declared as a dependency
Warning: you need USE_GNOME+=cairo
Error: /usr/local/bin/gitte is linked to /usr/local/lib/libgraphene-1.0.so.0 from graphics/graphene but it is not declared as a dependency
Warning: you need LIB_DEPENDS+=libgraphene-1.0.so:graphics/graphene
This revision is now accepted and ready to land.Thu, May 28, 12:59 PM

It seems that desktop-file-validate and appstreamcli utils are only invoked as a part of make test.
Consider adding OPTIONS_DEFINE= TEST and moving these deps to TEST_BUILD_DEPENDS.

use TEST block, add cairo

This revision now requires review to proceed.Thu, May 28, 5:09 PM
arrowd added inline comments.
deskutils/gitte/Makefile
31

Style nitpick: either move TEST_DESC closer to TEST_BUILD_DEPENDS or just remove this empty line altogether.

This revision is now accepted and ready to land.Thu, May 28, 5:32 PM