Bug 226897 - games/openrct2: move title-sequence and objects to the port, update to 0.2.1, fix build with big-endian architectures
Summary: games/openrct2: move title-sequence and objects to the port, update to 0.2.1,...
Status: In Progress
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: 2018-03-24 19:05 UTC by Piotr Kubaj
Modified: 2018-12-05 16:22 UTC (History)
3 users (show)

See Also:
pkubaj: maintainer-feedback+


Attachments
patch (3.38 KB, patch)
2018-03-24 19:05 UTC, Piotr Kubaj
no flags Details | Diff
patch (3.29 KB, patch)
2018-03-26 16:41 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff
patch (3.09 KB, patch)
2018-03-27 13:11 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff
patch (127.52 KB, patch)
2018-06-14 16:40 UTC, Piotr Kubaj
no flags Details | Diff
patch (127.89 KB, patch)
2018-06-16 00:07 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff
patch (125.27 KB, patch)
2018-08-27 18:23 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff
patch (125.27 KB, patch)
2018-08-28 07:29 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff
patch (170.39 KB, patch)
2018-11-22 23:14 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff
patch (40.12 KB, patch)
2018-12-03 17:14 UTC, Piotr Kubaj
pkubaj: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Kubaj 2018-03-24 19:05:08 UTC
Created attachment 191789 [details]
patch

This patch makes the port include title sequences that users were supposed to install manually. Since it installs necessary files and website that had instructions (https://openrct2.website/getting-started/index.html) is down, I'm also removing pkg-message.
Comment 1 janisozaur+freebsd 2018-03-25 17:54:26 UTC
We moved our domain to https://openrct2.io
Comment 2 Piotr Kubaj 2018-03-26 16:41:40 UTC
Created attachment 191843 [details]
patch

I added pkg-message again with the new website.

I also removed CMAKE_ON=      BUILD_SHARED_LIBS, it breaks install target on my CURRENT.
Comment 3 Piotr Kubaj 2018-03-27 13:11:21 UTC
Created attachment 191866 [details]
patch

I solved the issue on my end with shared libraries, so here's a new patch with that restored.
Comment 4 Piotr Kubaj 2018-06-14 16:40:35 UTC
Created attachment 194252 [details]
patch

I'm attaching another patch, which also updates the port to 0.2.0.

Since OpenRCT2 now requires C++17-aware libc++, it doesn't build on 10-STABLE and is as such ignored.

It was tested on 11-STABLE and CURRENT (Poudriere and running).
Comment 5 Piotr Kubaj 2018-06-16 00:07:31 UTC
Created attachment 194285 [details]
patch

New patch with corrected website in pkg-descr.
Comment 6 Piotr Kubaj 2018-08-27 18:23:16 UTC
Created attachment 196608 [details]
patch

Update to objects 1.0.6 and OpenRCT 0.2.1.
Comment 7 janisozaur+freebsd 2018-08-28 05:40:12 UTC
The latest patch is wrong, it changes the project website to https://openrct2.org, while it should point to https://openrct2.io
Comment 8 Piotr Kubaj 2018-08-28 07:29:18 UTC
Created attachment 196630 [details]
patch

Corrected.
Comment 9 Piotr Kubaj 2018-11-22 23:14:42 UTC
Created attachment 199460 [details]
patch

Same as before + add patch from upstream PR to build on big-endian platforms.
Comment 10 Dmitry Marakasov freebsd_committer 2018-11-27 12:07:11 UTC
- Please fetch title-sequence and objects through USE_GITHUB mechanism (e.g. via multiple GH_PROJECT values or GH_TUPLE)
- Please use DATADIR to avoid plist bloat
- I'd be more convenient to provide a "patch from upstream PR to build on big-endian platforms" as a single file with description. E.g. as in `git show` output.
Comment 11 Piotr Kubaj 2018-12-02 14:27:32 UTC
(In reply to Dmitry Marakasov from comment #10)
What do you mean with:
- Please use DATADIR to avoid plist bloat
?
Comment 12 Dmitry Marakasov freebsd_committer 2018-12-03 14:40:23 UTC
(In reply to Piotr Kubaj from comment #11)
> (In reply to Dmitry Marakasov from comment #10)
> What do you mean with:
> - Please use DATADIR to avoid plist bloat
> ?

Sorry, I've meant PORTDATA.

Makefile: add PORTDATA=*
pkg-plist: remove %%DATADIR%%/* lines
Comment 13 Piotr Kubaj 2018-12-03 17:14:08 UTC
Created attachment 199798 [details]
patch

Is that ok?
Comment 14 Dmitry Marakasov freebsd_committer 2018-12-05 16:22:47 UTC
(In reply to Piotr Kubaj from comment #13)

> Is that ok?

Looks good. Needs a few corrections, testing now.