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, Apr 25, 9:38 PM
Unknown Object (File)
Fri, Apr 12, 10:55 AM
Unknown Object (File)
Fri, Apr 12, 10:37 AM
Unknown Object (File)
Mar 18 2024, 7:42 PM
Unknown Object (File)
Mar 18 2024, 7:42 PM
Unknown Object (File)
Mar 18 2024, 7:41 PM
Unknown Object (File)
Mar 16 2024, 7:13 PM
Unknown Object (File)
Jan 2 2024, 7:40 PM
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