Bug 237342 - games/eduke32: improvement, allow installation of full game data
Summary: games/eduke32: improvement, allow installation of full game data
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Piotr Kubaj
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-04-17 17:36 UTC by Felix Palmen
Modified: 2019-05-29 09:19 UTC (History)
1 user (show)

See Also:


Attachments
patch adding FULL option to build with supplied full game's files (1.87 KB, patch)
2019-04-17 17:36 UTC, Felix Palmen
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Palmen 2019-04-17 17:36:17 UTC
Created attachment 203746 [details]
patch adding FULL option to build with supplied full game's files

If you want to play the full Duke3D version, you have to manually put the files in /usr/local/share/duke3d. I suggest attached patch that adds a port option instead for making the files part of the package.
Comment 1 Bugzilla Automation freebsd_committer 2019-04-17 17:36:17 UTC
Maintainer informed via mail
Comment 2 Piotr Kubaj freebsd_committer 2019-05-28 08:38:10 UTC
Looks good, I will post a review shortly.
Comment 3 Piotr Kubaj freebsd_committer 2019-05-28 09:06:24 UTC
portlint:
WARN: /usr/ports/games/eduke32/pkg-descr: exceeds 24 lines, make it shorter if possible.(currently 27 lines)
Comment 4 Felix Palmen 2019-05-28 09:30:33 UTC
Well, it's a warning after all, but of course it would be nice to have pkg-descr fit on a standard 80x25 terminal screen.

What are your thoughts about this? Is there a better place to put this kind of documentation? Or should the original description be shortened instead?

BR, Felix
Comment 5 Piotr Kubaj freebsd_committer 2019-05-28 09:40:53 UTC
(In reply to Felix Palmen from comment #4)
You could put it option description.

Like:
+FULL_DESC=		Install duke3d.grp and duke.rst from $${DISTDIR}/duke3d
Comment 6 Felix Palmen 2019-05-29 09:19:22 UTC
Thanks for the suggestion. I'll review my change again, with only such a short description, I guess I should add a sanity check, so the port doesn't attempt to build when FULL is selected, but the files aren't there.