Bug 271545 - games/instead: enable HARFBUZZ by default after sdl2_ttf2 update to 2.0.18
Summary: games/instead: enable HARFBUZZ by default after sdl2_ttf2 update to 2.0.18
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Dmitry Marakasov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-21 01:21 UTC by Jan Beich
Modified: 2023-05-23 22:07 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (amdmi3)


Attachments
v1 (apply via "git am") (1.03 KB, patch)
2023-05-21 01:21 UTC, Jan Beich
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Beich freebsd_committer freebsd_triage 2023-05-21 01:21:29 UTC
Created attachment 242306 [details]
v1 (apply via "git am")

After ports 68d149ef6d4c harfbuzz is indirectly required via sdl2_ttf

$ pkg install portfmt
$ portscan -p . games/instead
[100%] finished in 0s
V       games/instead                            HARFBUZZ_CMAKE_BOOL
V       games/instead                            HARFBUZZ_DESC
V       games/instead                            HARFBUZZ_LIB_DEPENDS
Vc      games/instead                            LUAJIT_LIB_DEPENDS

Note, LUAJIT_LIB_DEPENDS is out of scope but can be fixed by switching to USES=luajit.
Comment 1 Dmitry Marakasov freebsd_committer freebsd_triage 2023-05-22 18:27:56 UTC
> Note, LUAJIT_LIB_DEPENDS is out of scope but can be fixed by switching to USES=luajit.

Not really, `Vc` status denotes variables set multiple times, not unused ones. But yes, we should switch to USES=luajit. I'm also not enabling HARFBUZZ by default to match upstream setting.
Comment 2 commit-hook freebsd_committer freebsd_triage 2023-05-22 18:28:06 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=79e392c15d25a7fac189d22fe2d0c59585306f4f

commit 79e392c15d25a7fac189d22fe2d0c59585306f4f
Author:     Dmitry Marakasov <amdmi3@FreeBSD.org>
AuthorDate: 2023-05-22 15:41:29 +0000
Commit:     Dmitry Marakasov <amdmi3@FreeBSD.org>
CommitDate: 2023-05-22 18:27:10 +0000

    games/instead: enable harfbuzz support, fix lua(jit) deps

    - Enable harfbuzz support which is now available in SDL2_ttf (disabled
      by default according to upstream setting) [1]
    - Switch luajit dependency to USES=luajit (it also should now work on
      powerpc, so a related hack is no longer needed)
    - Fix dependency on lua
    - Remove no longer needed CMake argument

    PR:             271545 [1]
    Reported by:    jbeich [1]

 games/instead/Makefile | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)