Page MenuHomeFreeBSD

games/cockatrice: Add server component
ClosedPublic

Authored by sbruno on May 13 2018, 5:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 23 2023, 10:08 AM
Unknown Object (File)
Nov 6 2023, 9:25 PM
Unknown Object (File)
Oct 5 2023, 8:24 PM
Unknown Object (File)
Sep 26 2023, 2:51 AM
Unknown Object (File)
Sep 5 2023, 12:44 PM
Unknown Object (File)
Sep 5 2023, 12:42 PM
Unknown Object (File)
Sep 5 2023, 12:41 PM
Unknown Object (File)
Sep 5 2023, 12:39 PM
Subscribers

Details

Reviewers
None
Commits
rP470321: games/cockatrice
Summary

I've attempted to add the server component such that the
game can be installed with/without the server and that the server
can be installed with/without the required mysql server backend.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 16638
Build 16543: arc lint + arc unit

Event Timeline

games/cockatrice/Makefile
11

too long

20–21

USES should happen earlier. See Order of Variables in Port Makefiles.

29

Use OPTIONS_SUB instead.

30–32

Use options helpers.

Switch top options helper and OPTIONS_SUB where possible. I still
am not clear on how to make USE_QT5 do the right thing in this
framework.

I can't seem to get USE_QT5 to do anything with the OPTIONS_SUB framework. Does it work?

games/cockatrice/Makefile
27
SERVATRICE_USE=    qt5+="sql sql-mysql websockets"
games/cockatrice/Makefile
27

I guess this is the proper way to do OPTIONS_SUB here. I dunno.

sbruno added inline comments.
games/cockatrice/Makefile
27

It doesn't look like OPTIONS_SUB support "qt5+="

Use the proper syntax for USE_QT5= for the global QT5 requires.

Use the proper syntax for OPTIONS_USE= qt5="" so that different
options pull in their required deps. (thanks mat).

This revision was not accepted when it landed; it landed in state Needs Review.May 18 2018, 7:36 PM
Closed by commit rP470321: games/cockatrice (authored by sbruno). · Explain Why
This revision was automatically updated to reflect the committed changes.