Page MenuHomeFreeBSD

devel/appstream-qt: Create a port for Qt bindings to AppStream library. Sponsored by: Serenity Cybersecurity, LLC
ClosedPublic

Authored by arrowd on Feb 9 2023, 1:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 26, 4:03 AM
Unknown Object (File)
Tue, Dec 24, 1:32 AM
Unknown Object (File)
Sun, Dec 22, 8:45 PM
Unknown Object (File)
Sat, Dec 21, 8:03 AM
Unknown Object (File)
Thu, Dec 19, 10:12 PM
Unknown Object (File)
Nov 22 2024, 3:12 PM
Unknown Object (File)
Nov 18 2024, 11:56 AM
Unknown Object (File)
Nov 18 2024, 10:05 AM
Subscribers

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

arrowd requested review of this revision.Feb 9 2023, 1:40 PM
tcberner added a subscriber: tcberner.

some minor suggestion on line 17, 23 and 30, other wise, I guess it looks fine :)

devel/appstream-qt/Makefile
2

^ I think -qt would be more canonical? -- never mind, the master port is AppStream oO and not appstream :)

devel/appstream/Makefile
20

the ?= and += might be a bit confusing if one does not know about the existence of the slave-port.
Maybe consider to add a comment ala

# overriden by devel/appstream-qt

etc ?

This revision is now accepted and ready to land.Feb 13 2023, 12:18 PM